prebid-universal-creative 1.14.1 → 1.15.0

package/test/spec/renderingManager_spec.js

@@ -1,4 +1,4 @@
-import { newRenderingManager } from 'src/renderingManager';
+import { renderCrossDomain, renderLegacy } from 'src/renderingManager';
 import * as utils from 'src/utils';
 import * as domHelper from 'src/domHelper';
 import { expect } from 'chai';
@@ -70,251 +70,12 @@ describe('renderingManager', function() {
     xhr.restore();
   });
 
-  describe('mobile creative', function() {
-    let writeHtmlSpy;
-    let sendRequestSpy;
-    let triggerPixelSpy;
-    let mockWin;
-
-    before(function() {
-      writeHtmlSpy = sinon.spy(utils, 'writeAdHtml');
-      sendRequestSpy = sinon.spy(utils, 'sendRequest');
-      triggerPixelSpy = sinon.spy(utils, 'triggerPixel');
-      mockWin = merge(mocks.createFakeWindow('http://example.com'), renderingMocks().getWindowObject());
-    });
-
-    afterEach(function() {
-      writeHtmlSpy.resetHistory();
-      sendRequestSpy.resetHistory();
-      triggerPixelSpy.resetHistory();
-    });
-
-    after(function() {
-      writeHtmlSpy.restore();
-      sendRequestSpy.restore();
-      triggerPixelSpy.restore();
-    });
-
-    const env = {
-      isMobileApp: () => true,
-      isSafeFrame: () => false
-    };
-
-    it('should render mobile app creative', function() {
-      const renderObject = newRenderingManager(mockWin, env);
-      let ucTagData = {
-        cacheHost: 'example.com',
-        cachePath: '/path',
-        uuid: '123',
-        size: '300x250'
-      };
-
-      renderObject.renderAd(mockWin.document, ucTagData);
-
-      let response = {
-        width: 300,
-        height: 250,
-        crid: 123,
-        adm: 'ad-markup',
-        wurl: 'https://test.prebidcache.wurl'
-      };
-      requests[0].respond(200, {}, JSON.stringify(response));
-      expect(writeHtmlSpy.callCount).to.equal(1);
-      expect(sendRequestSpy.args[0][0]).to.equal('https://example.com/path?uuid=123');
-    });
-
-    it('should render mobile app creative with missing cache wurl', function() {
-      const renderObject = newRenderingManager(mockWin, env);
-      let ucTagData = {
-        cacheHost: 'example.com',
-        cachePath: '/path',
-        uuid: '123',
-        size: '300x250'
-      };
-
-      renderObject.renderAd(mockWin.document, ucTagData);
-
-      let response = {
-        width: 300,
-        height: 250,
-        crid: 123,
-        adm: 'ad-markup'
-      };
-      requests[0].respond(200, {}, JSON.stringify(response));
-      expect(writeHtmlSpy.callCount).to.equal(1);
-      expect(sendRequestSpy.args[0][0]).to.equal('https://example.com/path?uuid=123');
-    });
-
-    it('should render mobile app creative using default cacheHost and cachePath', function() {
-      const renderObject = newRenderingManager(mockWin, env);
-      let ucTagData = {
-        uuid: '123',
-        size: '300x250'
-      };
-
-      renderObject.renderAd(mockWin.document, ucTagData);
-
-      let response = {
-        width: 300,
-        height: 250,
-        crid: 123,
-        adm: 'ad-markup'
-      };
-      requests[0].respond(200, {}, JSON.stringify(response));
-      expect(writeHtmlSpy.callCount).to.equal(1);
-      expect(sendRequestSpy.args[0][0]).to.equal('https://prebid.adnxs.com/pbc/v1/cache?uuid=123');
-    });
-
-    // it('should catch errors from creative', function (done) {
-    //   window.addEventListener('error', e => {
-    //     done(e.error);
-    //   });
-
-    //   const consoleErrorSpy = sinon.spy(console, 'error');
-
-    //   const renderObject = newRenderingManager(mockWin, env);
-    //   let ucTagData = {
-    //     cacheHost: 'example.com',
-    //     cachePath: '/path',
-    //     uuid: '123',
-    //     size: '300x250'
-    //   };
-
-    //   renderObject.renderAd(mockWin.document, ucTagData);
-
-    //   let response = {
-    //     width: 300,
-    //     height: 250,
-    //     crid: 123,
-    //     adm: '<script src="notExistingScript.js"></script>'
-    //   };
-    //   requests[0].respond(200, {}, JSON.stringify(response));
-
-    //   setTimeout(()=>{
-    //     expect(consoleErrorSpy.callCount).to.equal(1);
-    //     done();
-    //   }, 10);
-    // });
-  });
-
-  describe('amp creative', function() {
-    let writeHtmlSpy;
-    let sendRequestSpy;
-    let triggerPixelSpy;
-    let mockWin;
-
-    before(function() {
-      writeHtmlSpy = sinon.spy(utils, 'writeAdHtml');
-      sendRequestSpy = sinon.spy(utils, 'sendRequest');
-      triggerPixelSpy = sinon.spy(utils, 'triggerPixel');
-      mockWin = merge(mocks.createFakeWindow('http://example.com'), renderingMocks().getWindowObject());
-    });
-
-    afterEach(function() {
-      writeHtmlSpy.resetHistory();
-      sendRequestSpy.resetHistory();
-      triggerPixelSpy.resetHistory();
-    });
-
-    after(function() {
-      writeHtmlSpy.restore();
-      sendRequestSpy.restore();
-      triggerPixelSpy.restore();
-    });
-
-    const env = {
-      isMobileApp: () => false,
-      isAmp: () => true,
-      isSafeFrame: () => true
-    };
-
-    it('should render amp creative', function() {
-      const renderObject = newRenderingManager(mockWin, env);
-
-      let ucTagData = {
-        cacheHost: 'example.com',
-        cachePath: '/path',
-        uuid: '123',
-        size: '300x250',
-        hbPb: '10.00'
-      };
-
-      renderObject.renderAd(mockWin.document, ucTagData);
-
-      let response = {
-        width: 300,
-        height: 250,
-        crid: 123,
-        adm: 'ad-markup${AUCTION_PRICE}',
-        wurl: 'https://test.prebidcache.wurl'
-      };
-      requests[0].respond(200, {}, JSON.stringify(response));
-      expect(writeHtmlSpy.args[0][0]).to.equal('<!--Creative 123 served by Prebid.js Header Bidding-->ad-markup10.00');
-      expect(sendRequestSpy.args[0][0]).to.equal('https://example.com/path?uuid=123');
-      expect(triggerPixelSpy.args[0][0]).to.equal('https://test.prebidcache.wurl');
-    });
-
-    it('should replace AUCTION_PRICE with response.price over hbPb', function() {
-      const renderObject = newRenderingManager(mockWin, env);
-
-      let ucTagData = {
-        cacheHost: 'example.com',
-        cachePath: '/path',
-        uuid: '123',
-        size: '300x250',
-        hbPb: '10.00'
-      };
-
-      renderObject.renderAd(mockWin.document, ucTagData);
-
-      let response = {
-        width: 300,
-        height: 250,
-        crid: 123,
-        price: 12.50,
-        adm: 'ad-markup${AUCTION_PRICE}',
-        wurl: 'https://test.prebidcache.wurl'
-      };
-      requests[0].respond(200, {}, JSON.stringify(response));
-      expect(writeHtmlSpy.args[0][0]).to.equal('<!--Creative 123 served by Prebid.js Header Bidding-->ad-markup12.5');
-      expect(sendRequestSpy.args[0][0]).to.equal('https://example.com/path?uuid=123');
-      expect(triggerPixelSpy.args[0][0]).to.equal('https://test.prebidcache.wurl');
-    });
-
-    it('should replace AUCTION_PRICE with with empty value when neither price nor hbPb exist', function() {
-      const renderObject = newRenderingManager(mockWin, env);
-
-      let ucTagData = {
-        cacheHost: 'example.com',
-        cachePath: '/path',
-        uuid: '123',
-        size: '300x250'
-      };
-
-      renderObject.renderAd(mockWin.document, ucTagData);
-
-      let response = {
-        width: 300,
-        height: 250,
-        crid: 123,
-        adm: 'ad-markup${AUCTION_PRICE}',
-        wurl: 'https://test.prebidcache.wurl'
-      };
-      requests[0].respond(200, {}, JSON.stringify(response));
-      expect(writeHtmlSpy.args[0][0]).to.equal('<!--Creative 123 served by Prebid.js Header Bidding-->ad-markup');
-      expect(sendRequestSpy.args[0][0]).to.equal('https://example.com/path?uuid=123');
-      expect(triggerPixelSpy.args[0][0]).to.equal('https://test.prebidcache.wurl');
-    });
-  });
-
   describe('cross domain creative', function() {
     const ORIGIN = 'http://example.com';
     let parseStub;
     let iframeStub;
     let triggerPixelSpy;
     let mockWin;
-    let env;
-    let renderObject;
     let ucTagData;
     let mockIframe;
     let eventSource;
@@ -329,22 +90,18 @@ describe('renderingManager', function() {
         host: 'example.com'
       });
       mockWin = merge(mocks.createFakeWindow(ORIGIN), renderingMocks().getWindowObject());
-      env = {
-        isMobileApp: () => false,
-        isAmp: () => false,
-        canLocatePrebid: () => false
-      };
-      renderObject = newRenderingManager(mockWin, env);
       ucTagData = {
         adId: '123',
         adServerDomain: 'mypub.com',
         pubUrl: ORIGIN,
       };
-      renderObject.renderAd(mockWin.document, ucTagData);
+      eventSource = null;
+
+      renderCrossDomain(mockWin, ucTagData.adId, ucTagData.adServerDomain, ucTagData.pubUrl);
 
     });
 
-    afterEach(function() {
+    afterEach(function () {
       parseStub.restore();
       iframeStub.restore();
       triggerPixelSpy.restore();
@@ -353,17 +110,17 @@ describe('renderingManager', function() {
     function mockPrebidResponse(msg)  {
       mockWin.postMessage({
         origin: ORIGIN,
-        message: JSON.stringify(Object.assign({message: 'Prebid Response'}, msg))
+        message: JSON.stringify(Object.assign({ message: 'Prebid Response' }, msg))
       });
     }
 
-    it('should render cross domain creative', function() {
+    it("should render cross domain creative", function () {
       mockPrebidResponse({
-        ad: 'ad',
+        ad: "ad",
         adUrl: ORIGIN,
-        adId: '123',
+        adId: "123",
         width: 300,
-        height: 250
+        height: 250,
       });
       expect(mockIframe.contentDocument.write.args[0][0]).to.equal("ad");
     });
@@ -390,13 +147,13 @@ describe('renderingManager', function() {
             info: {
               reason: 'preventWritingOnMainDocument'
             }
-          })
+          });
         });
 
         it('on ads that have no markup or adUrl', () => {
           mockPrebidResponse({
             adId: '123',
-          })
+          });
           expectEventMessage({
             adId: '123',
             event: RENDER_FAILED,
@@ -408,7 +165,7 @@ describe('renderingManager', function() {
 
         it('on exceptions', () => {
           iframeStub.callsFake(() => {
-            throw new Error()
+            throw new Error();
           });
           mockPrebidResponse({
             adId: '123',
@@ -422,7 +179,7 @@ describe('renderingManager', function() {
               reason: 'exception'
             }
           });
-        })
+        });
       });
       describe('should post AD_RENDER_SUCCEEDED', () => {
         it('on ad with markup', () => {
@@ -452,18 +209,12 @@ describe('renderingManager', function() {
   describe('legacy creative', function() {
     it('should render legacy creative', function() {
       const mockWin = merge(mocks.createFakeWindow('http://example.com'), renderingMocks().getWindowObject());
-      const env = {
-        isMobileApp: () => false,
-        isAmp: () => false,
-        canLocatePrebid: () => true
-      };
-      const renderObject = newRenderingManager(mockWin, env);
-
       let ucTagData = {
         adId: '123'
       };
+      window.parent = mockWin;
 
-      renderObject.renderAd(mockWin.document, ucTagData);
+      renderLegacy(mockWin.document, ucTagData.adId);
       expect(mockWin.parent.$$PREBID_GLOBAL$$.renderAd.callCount).to.equal(1);
     });
   });

package/webpack.conf.js

@@ -1,6 +1,7 @@
 var creative = require('./package.json');
 var StringReplacePlugin = require('string-replace-webpack-plugin');
 var path = require('path');
+const ShakePlugin = require('webpack-common-shake').Plugin;
 
 module.exports = {
   devtool: 'source-map',
@@ -42,5 +43,6 @@ module.exports = {
         })
       }
     ]
-  }
+  },
+  plugins: [new ShakePlugin()],
 };