more cleanup

ampproject · May 8, 2017 · af057a1 · af057a1
1 parent 9e9bba2
commit af057a1
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/extensions/amp-bind/0.1/test/test-amp-state.js b/extensions/amp-bind/0.1/test/test-amp-state.js
@@ -86,11 +86,13 @@ describes.realWin('AmpState', {
     ampState.build();
 
     // IMPORTANT: No parsing should happen until viewer is visible.
+    expect(fetchSpy).to.not.have.been.called;
     expect(batchedJsonStub).to.not.have.been.called;
     expect(updateStub).to.not.have.been.called;
 
     whenFirstVisiblePromiseResolve();
     return whenFirstVisiblePromise.then(() => {
+      expect(fetchSpy).calledWithExactly(/* isInit */true);
       expect(updateStub).calledWithMatch({foo: 'bar'});
     });
   });
@@ -124,6 +126,7 @@ describes.realWin('AmpState', {
     const isVisibleStub = env.sandbox.stub(viewer, 'isVisible');
     isVisibleStub.returns(false);
     ampState.mutatedAttributesCallback({src: 'https://foo.com/bar?baz=1'});
+    expect(fetchSpy).to.not.have.been.called;
     expect(batchedJsonStub).to.not.have.been.called;
 
     isVisibleStub.returns(true);

diff --git a/extensions/amp-bind/0.1/validator-amp-bind.protoascii b/extensions/amp-bind/0.1/validator-amp-bind.protoascii
@@ -50,7 +50,6 @@ tags: {  # <amp-state>
   html_format: AMP
   tag_name: "AMP-STATE"
   spec_name: "amp-state"
-  satisfies: "amp-state"
   requires: "amp-bind extension .js script"
   requires: "amp-bind extension .json script"
   disallowed_ancestor: "AMP-SIDEBAR"