Updates

ampproject · Mar 13, 2018 · a612aa1 · a612aa1
1 parent 64f2bbb
commit a612aa1
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 5 deletions.
diff --git a/extensions/amp-story/0.1/test/test-variable-service.js b/extensions/amp-story/0.1/test/test-variable-service.js
@@ -33,7 +33,8 @@ describes.fakeWin('amp-story variable service', {}, () => {
       },
     });
 
-    expect(variableService.pageIndex).to.equal(123);
-    expect(variableService.pageId).to.equal('my-page-id');
+    const variables = variableService.get();
+    expect(variables.pageIndex).to.equal(123);
+    expect(variables.pageId).to.equal('my-page-id');
   });
 });
diff --git a/extensions/amp-story/0.1/variable-service.js b/extensions/amp-story/0.1/variable-service.js
@@ -37,7 +37,7 @@ export class AmpStoryVariableService {
     };
 
     /** @private {boolean} */
-    this.initialized_ = false;
+    this.isInitialized_ = false;
   }
 
   /**
@@ -48,7 +48,7 @@ export class AmpStoryVariableService {
       case StateChangeType.ACTIVE_PAGE:
         const {pageIndex, pageId} = stateChangeEvent.value;
         const variables = this.variables_;
-        this.initialized_ = true;
+        this.isInitialized_ = true;
         variables.pageIndex = pageIndex;
         variables.pageId = pageId;
         break;
@@ -59,7 +59,7 @@ export class AmpStoryVariableService {
    * @return {!StoryVariableDef}
    */
   get() {
-    dev().assert(this.initialized_);
+    dev().assert(this.isInitialized_);
     // TODO(newmius): You should probably Object.freeze this in development.
     return this.variables_;
   }