added tests

extensions/amp-video/0.1/test/test-video-cache.js

@@ -88,6 +88,32 @@ describes.realWin('amp-video cached-sources', {amp: true}, (env) => {
     });
   });
 
+  describe('url forming', () => {
+    it('should send the request to the correct address if the video has an absolute url', async () => {
+      const videoEl = createVideo([{'src': 'https://website.com/video.html'}]);
+      const xhrSpy = env.sandbox.spy(xhrService, 'fetch');
+
+      const ampVideo = new AmpVideo(videoEl);
+      await ampVideo.buildCallback();
+
+      expect(xhrSpy).to.have.been.calledWith(
+        'https://website-com.cdn.ampproject.org/mbv/s/website.com/video.html'
+      );
+    });
+
+    it('should send the request to the correct address if the video has a relative url', async () => {
+      const videoEl = createVideo([{'src': 'video.html'}]);
+      const xhrSpy = env.sandbox.spy(xhrService, 'fetch');
+
+      const ampVideo = new AmpVideo(videoEl);
+      await ampVideo.buildCallback();
+
+      expect(xhrSpy).to.have.been.calledWith(
+        'https://example-com.cdn.ampproject.org/mbv/s/example.com/video.html'
+      );
+    });
+  });
+
   describe('end to end', () => {
     it('should create the sources from the request with the correct attributes', async () => {
       env.sandbox.stub(xhrService, 'fetch').resolves({
@@ -106,6 +132,13 @@ describes.realWin('amp-video cached-sources', {amp: true}, (env) => {
 
       expect(videoEl.querySelector('source[data-bitrate]')).to.not.be.null;
     });
+    it('should not create the sources if there is amp-orig-src attribute', async () => {
+      const videoEl = createVideo([{'src': 'video.mp4', 'amp-orig-src': ''}]);
+      const ampVideo = new AmpVideo(videoEl);
+      await ampVideo.buildCallback();
+
+      expect(videoEl.querySelector('source[data-bitrate]')).to.be.null;
+    });
   });
 
   function createVideo(children) {

extensions/amp-video/0.1/video-cache.js

@@ -30,8 +30,8 @@ export function addCacheSources(videoEl, win) {
   if (
     !extensionScriptsInNode(win.document).some(
       (extension) => extension.extensionId === 'amp-cache-url'
-    ) &&
-    videoEl.querySelector('source[src]')
+    ) ||
+    !videoEl.querySelector('source[src]')
   ) {
     return Promise.resolve();
   }