Skip to content

Conversation

@sergseven
Copy link

@sergseven sergseven commented Dec 23, 2024

having the PR here to have the checks running

@sergseven sergseven marked this pull request as ready for review December 23, 2024 15:12
@sergseven
Copy link
Author

}

@Test
public void openrtb2AuctionWithNativeShouldRespondWithBidsFromOpenx() throws IOException, JSONException {
Copy link
Author

@sergseven sergseven Dec 27, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

note: we are considering removing this test case since it seems to be duplicated with the unit tests and it looks like this test level is not dedicated to test the features, but integration only

ref: https://github.com/prebid/prebid-server-java/blob/master/docs/developers/code-style.md#bidder-smoke-tests

Copy link

@bukrain bukrain left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link

@gmiedlar-ox gmiedlar-ox left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

According to our recent conversation, maybe it would be worth to add a test case verifying what will be returned from makeHttpRequest when there is multi-ad-placement imp in bid request?

@sergseven
Copy link
Author

merged to the mainstream

@sergseven sergseven closed this Jan 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants