check API key validity messages in system-test

kjin · kjin · commit 8b7f9bcd6475 · 2017-09-14T13:44:07.000-07:00
diff --git a/packages/error-reporting/src/google-apis/auth-client.js b/packages/error-reporting/src/google-apis/auth-client.js
@@ -120,7 +120,8 @@ class RequestHandler extends common.Service {
         method: 'POST',
         json: {}
       }, (err, body, response) => {
-        if (err && err.message !== 'Message cannot be empty.') {
+        if (err && err.message !== 'Message cannot be empty.' && 
+            response.statusCode === 400) {
           this._logger.error([
             'Encountered an error while attempting to validate the provided',
             'API key'
diff --git a/packages/error-reporting/system-test/error-reporting.js b/packages/error-reporting/system-test/error-reporting.js
@@ -31,6 +31,7 @@ var forEach = require('lodash.foreach');
 var assign = require('lodash.assign');
 var pick = require('lodash.pick');
 var omitBy = require('lodash.omitby');
+var request = require('request');
 var util = require('util');
 var path = require('path');
 
@@ -362,6 +363,41 @@ describe('Expected Behavior', function() {
   });
 });
 
+describe('Error Reporting API', function() {
+  [
+    {
+      name: 'when a valid API key is given',
+      getKey: () => env.apiKey,
+      message: 'Message cannot be empty.'
+    },
+    {
+      name: 'when an empty API key is given',
+      getKey: () => '',
+      message: 'The request is missing a valid API key.'
+    },
+    {
+      name: 'when an invalid API key is given',
+      getKey: () => env.apiKey.slice(1) + env.apiKey[0],
+      message: 'API key not valid. Please pass a valid API key.'
+    }
+  ].forEach(function(testCase) {
+    it(`should return an expected message ${testCase.name}`, function(done) {
+      this.timeout(30000);
+      const API = 'https://clouderrorreporting.googleapis.com/v1beta1';
+      const key = testCase.getKey();
+      request.post({
+        url: `${API}/projects/${env.projectId}/events:report?key=${key}`,
+        json: {},
+      }, (err, response, body) => {
+        assert.ok(!err && body.error);
+        assert.strictEqual(response.statusCode, 400);
+        assert.strictEqual(body.error.message, testCase.message);
+        done();
+      });
+    });
+  });
+});
+
 describe('error-reporting', function() {
   const SRC_ROOT = path.join(__dirname, '..', 'src');
   const TIMESTAMP = Date.now();
