-
Notifications
You must be signed in to change notification settings - Fork 175
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Functional tests for bid.exp in response #3222
Functional tests for bid.exp in response #3222
Conversation
} | ||
|
||
def "PBS auction should resolve exp from request.ext.prebid.cache.vastxml when it have value"() { | ||
given: "Default basic bid with ext.prebid.cache.bids" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ext.prebid.cache.vastXml
} | ||
|
||
def "PBS auction should resolve exp from request.imp[].exp when it have value"() { | ||
given: "Default basic bid with exp" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
bid request
'cache.video-ttl-seconds' : hostVideoTtl as String]) | ||
|
||
def "PBS auction should resolve bid.exp from response that is set by the bidder’s adapter"() { | ||
given: "Default basic bid with exp" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
bid request
assert response.seatbid.bid.first.exp == [hostBannerTtl] | ||
} | ||
|
||
def "PBS auction should resolve exp from global video config for banner request"() { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
maybe for video request?
e05a48e
to
bec8214
Compare
and: "Set bidder response without exp" | ||
def bidResponse = BidResponse.getDefaultBidResponse(bidRequest) | ||
bidder.setResponse(bidRequest.id, bidResponse) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It would be great to see explicitly without exp
, since in the future somebody can change the default bid response and add exp
field. by default
assert response.seatbid.bid.first.exp == [bidRequestExp] | ||
} | ||
|
||
def "PBS auction should resolve exp from request.ext.prebid.cache for request when it have invalid type"() { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe shouldn't resolve exp?
No description provided.