OpenWeb: updates
Code coverage summary
Note:
- Prebid team doesn't anticipate tests covering code paths that might result in marshal and unmarshal errors
- Coverage summary encompasses all commits leading up to the latest one, a40d8168e3299a122f8137ff24d1c6db14da9923
openweb
Refer here for heat map coverage report
github.com/prebid/prebid-server/v2/adapters/openweb/openweb.go:22: MakeRequests 81.8%
github.com/prebid/prebid-server/v2/adapters/openweb/openweb.go:47: extractOrg 81.8%
github.com/prebid/prebid-server/v2/adapters/openweb/openweb.go:68: MakeBids 83.3%
github.com/prebid/prebid-server/v2/adapters/openweb/openweb.go:105: getMediaTypeForBid 100.0%
github.com/prebid/prebid-server/v2/adapters/openweb/openweb.go:119: Builder 100.0%
total: (statements) 84.8%
@IgorKulemzin There is no description and the title is generic. Could you please give us an idea of the changes to the openweb adapter in this PR?
@SyntaxNode The PR title and the description have been updated. Sorry for inconvenience.
Code coverage summary
Note:
- Prebid team doesn't anticipate tests covering code paths that might result in marshal and unmarshal errors
- Coverage summary encompasses all commits leading up to the latest one, 2d1a0378655197822504daf00427fe8d6abd9f3c
openweb
Refer here for heat map coverage report
github.com/prebid/prebid-server/v2/adapters/openweb/openweb.go:23: MakeRequests 81.8%
github.com/prebid/prebid-server/v2/adapters/openweb/openweb.go:48: extractOrg 76.9%
github.com/prebid/prebid-server/v2/adapters/openweb/openweb.go:73: MakeBids 84.2%
github.com/prebid/prebid-server/v2/adapters/openweb/openweb.go:112: getMediaTypeForBid 100.0%
github.com/prebid/prebid-server/v2/adapters/openweb/openweb.go:126: Builder 100.0%
total: (statements) 83.7%
@onkarvhanumante Can you please take another look, I think all of your comments are addressed 🙏🏻
Code coverage summary
Note:
- Prebid team doesn't anticipate tests covering code paths that might result in marshal and unmarshal errors
- Coverage summary encompasses all commits leading up to the latest one, 03d44e7165abb29668e141977a26092c61f3284f
openweb
Refer here for heat map coverage report
github.com/prebid/prebid-server/v2/adapters/openweb/openweb.go:23: MakeRequests 81.8%
github.com/prebid/prebid-server/v2/adapters/openweb/openweb.go:49: checkExtAndExtractOrg 86.7%
github.com/prebid/prebid-server/v2/adapters/openweb/openweb.go:78: MakeBids 84.2%
github.com/prebid/prebid-server/v2/adapters/openweb/openweb.go:117: getMediaTypeForBid 100.0%
github.com/prebid/prebid-server/v2/adapters/openweb/openweb.go:131: Builder 100.0%
total: (statements) 86.3%
Thanks @onkarvhanumante, can you please check again?
@onkarvhanumante I think I addressed all of your comments, can you please check again?
@onkarvhanumante - just a reminder if you can take a look. @Sonali-More-Xandr already approved, only your approval is needed. thanks in advance.