Lint

ampproject · May 30, 2017 · 2f83963 · 2f83963
1 parent ee0b983
commit 2f83963
Show file tree

Hide file tree

Showing 6 changed files with 10 additions and 9 deletions.
diff --git a/extensions/amp-analytics/0.1/test/test-amp-analytics.js b/extensions/amp-analytics/0.1/test/test-amp-analytics.js
@@ -94,7 +94,7 @@ describe('amp-analytics', function() {
           }
           return Promise.resolve({
             json() {
-              return Promise.resolve(SON.parse(jsonMockResponses[url]));
+              return Promise.resolve(JSON.parse(jsonMockResponses[url]));
             },
           });
         }};

diff --git a/extensions/amp-auto-ads/0.1/amp-auto-ads.js b/extensions/amp-auto-ads/0.1/amp-auto-ads.js
@@ -79,7 +79,7 @@ export class AmpAutoAds extends AMP.BaseElement {
     };
     return xhrFor(this.win)
         .fetchJson(configUrl, xhrInit)
-        .then(res => res.json());
+        .then(res => res.json())
         .catch(reason => {
           user().error(TAG, 'amp-auto-ads config xhr failed: ' + reason);
           return null;

diff --git a/extensions/amp-list/0.1/test/test-amp-list.js b/extensions/amp-list/0.1/test/test-amp-list.js
@@ -153,7 +153,7 @@ describe('amp-list component', () => {
         json() {
           return Promise.resolve({});
         },
-      });.once();
+      })).once();
     templatesMock.expects('findAndRenderTemplateArray').never();
     return expect(list.layoutCallback()).to.eventually.be
         .rejectedWith(/Response must contain an array/);
@@ -170,7 +170,7 @@ describe('amp-list component', () => {
         json() {
           return Promise.resolve({different});
         },
-      });.once();
+      })).once();
     templatesMock.expects('findAndRenderTemplateArray')
         .withExactArgs(element, different)
         .returns(Promise.resolve([itemElement])).once();

diff --git a/extensions/amp-share-tracking/0.1/amp-share-tracking.js b/extensions/amp-share-tracking/0.1/amp-share-tracking.js
@@ -152,7 +152,8 @@ export class AmpShareTracking extends AMP.BaseElement {
             'have a fragment value.');
           return '';
         }, err => {
-          user().error(TAG, 'The request to share-tracking endpoint failed:' + err);
+          user().error(TAG, 'The request to share-tracking endpoint failed:',
+              err);
           return '';
         });
   }

diff --git a/extensions/amp-share-tracking/0.1/test/test-amp-share-tracking.js b/extensions/amp-share-tracking/0.1/test/test-amp-share-tracking.js
@@ -222,7 +222,7 @@ describes.fakeWin('amp-share-tracking', {
       status: 404,
       json() {
         return Promise.reject('404 bad json');
-      }
+      },
     }));
     const ampShareTracking = getAmpShareTracking('http://foo.bar');
     return shareTrackingForOrNull(ampShareTracking.win).then(fragments => {

diff --git a/test/functional/test-impression.js b/test/functional/test-impression.js
@@ -36,7 +36,7 @@ describe('impression', () => {
     sandbox.stub(viewer, 'getParam');
     xhr = xhrFor(window);
     expect(xhr.fetchJson).to.be.defined;
-    sandbox.stub(xhr, 'fetchJson' () => {
+    sandbox.stub(xhr, 'fetchJson', () => {
       return Promise.resolve({
         json() {
           return Promise.resolve(null);
@@ -101,7 +101,7 @@ describe('impression', () => {
             return Promise.resolve({
               'location': 'test_location?gclid=654321',
             });
-          }
+          },
         });
       }, 5000);
     }));
@@ -125,7 +125,7 @@ describe('impression', () => {
         resolve({
           json() {
             return Promise.resolve(null);
-          }
+          },
         });
       }, 10000);
     }));