Deal override fix to resolve #618

[indexexchange]

Type of change

• Bugfix
• Feature
• New bidder adapter
• Code style update (formatting, local variables)
• Refactoring (no functional changes, no api changes)
• Build related changes
• CI related changes
• Other

Description of change

Change the following code to getDealTargeting last

function getAllTargeting() {
    // Get targeting for the winning bid. Add targeting for any bids that have                                                                                                                                                             
    // `alwaysUseBid=true`. If sending all bids is enabled, add targeting for losing bids.                                                                                                                                                 
    var targeting = getDealTargeting()
        .concat(getWinningBidTargeting())
        .concat(getAlwaysUseBidTargeting())
        .concat($$PREBID_GLOBAL$$._sendAllBids ? getBidLandscapeTargeting() : []);

[mkendall07]

@indexexchange
Please include a unit test that asserts this change. Thanks!

[protonate]

Looks good, tests are green. In the case where bid landscape info and deal info collide, we should prefer the deal info. However it may be we need to prevent the key name collision to begin with.