From 1f156692844617a5f70cb56d4f732fa8d4d4829d Mon Sep 17 00:00:00 2001
From: David Bridges <dbridges12@mac.com>
Date: Fri, 3 Feb 2017 12:29:34 -0700
Subject: [PATCH] set new bid.descriptionUrl option to impression_id and
 updated unit test

---
 src/adapters/rubicon.js            |  3 ++-
 test/spec/adapters/rubicon_spec.js | 11 ++++++-----
 2 files changed, 8 insertions(+), 6 deletions(-)

diff --git a/src/adapters/rubicon.js b/src/adapters/rubicon.js
index 8c737238adf..838d8928dfa 100644
--- a/src/adapters/rubicon.js
+++ b/src/adapters/rubicon.js
@@ -280,7 +280,8 @@ function RubiconAdapter() {
       if (bidRequest.mediaType === 'video') {
         bid.width = bidRequest.params.video.playerWidth;
         bid.height = bidRequest.params.video.playerHeight;
-        bid.vastUrl = ad.impression_id;
+        bid.vastUrl = ad.creative_depot_url;
+        bid.descriptionUrl = ad.impression_id;
         bid.impression_id = ad.impression_id;
       } else {
         bid.ad = _renderCreative(ad.script, ad.impression_id);
diff --git a/test/spec/adapters/rubicon_spec.js b/test/spec/adapters/rubicon_spec.js
index 5e76bc3a3da..a875def399f 100644
--- a/test/spec/adapters/rubicon_spec.js
+++ b/test/spec/adapters/rubicon_spec.js
@@ -114,7 +114,7 @@ describe('the rubicon adapter', () => {
       sandbox.stub(rubiconAdapter, 'callBids');
 
       adapterManager.callBids({
-          adUnits: [clone(adUnit)]
+        adUnits: [clone(adUnit)]
       });
 
       let bidderRequest = rubiconAdapter.callBids.getCall(0).args[0];
@@ -172,7 +172,7 @@ describe('the rubicon adapter', () => {
 
       ordering = masSizeOrdering([[120, 600], [320, 50], [160,600], [640, 480],[336, 280], [200, 600], [728, 90]]);
       expect(ordering).to.deep.equal([2, 9, 8, 16, 43, 65, 126]);
-    })
+    });
 
   });
 
@@ -303,7 +303,7 @@ describe('the rubicon adapter', () => {
 
           expect(post).to.have.property('page_url').that.is.a('string');
           expect(post.resolution).to.match(/\d+x\d+/);
-          expect(post.account_id).to.equal('14062')
+          expect(post.account_id).to.equal('14062');
           expect(post.integration).to.equal(INTEGRATION);
           expect(post).to.have.property('timeout').that.is.a('number');
           expect(post.timeout < 5000).to.equal(true);
@@ -346,7 +346,7 @@ describe('the rubicon adapter', () => {
           expect(slot.visitor).to.have.property('ucat').that.equals('new');
           expect(slot.visitor).to.have.property('lastsearch').that.equals('iphone');
 
-        })
+        });
 
       });
 
@@ -639,12 +639,13 @@ describe('the rubicon adapter', () => {
           expect(bids[0].bidderCode).to.equal('rubicon');
           expect(bids[0].creative_id).to.equal(999999);
           expect(bids[0].cpm).to.equal(1);
+          expect(bids[0].descriptionUrl).to.equal('a40fe16e-d08d-46a9-869d-2e1573599e0c');
           expect(bids[0].vastUrl).to.equal(
               'https://optimized-by-adv.rubiconproject.com/v1/creative/a40fe16e-d08d-46a9-869d-2e1573599e0c.xml'
           );
           expect(bids[0].impression_id).to.equal('a40fe16e-d08d-46a9-869d-2e1573599e0c');
 
-        })
+        });
 
       });