Skip to content
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

create stubs for localStorage in widespaceBidAdapter test file #4208

Merged
merged 1 commit into from
Sep 23, 2019

Conversation

jsnellbaker
Copy link
Collaborator

@jsnellbaker jsnellbaker commented Sep 18, 2019

Type of change

  • Refactoring (no functional changes, no api changes)
  • Other

Description of change

This PR is meant to help prevent sporadic browserstack unit test failures related to use of localStorage in the widespaceBidAdapter file.

The error causing the test failures is: Access is denied. Based on some research, this error is due to when some permission settings in IE browsers prevents access to localStorage.

The sporadic nature of the error is likely just timing/initialization of the Browserstack IE 11 browser not having the right setting(s) at the time of the test. Normally rerunning the job causes the tests to pass fine, but the frequency of this error over time has made rerunning the jobs become tedious to do - especially when the related PR isn't about the widespaceBidAdapter.

See example here in recent circleCI test job for a failure and subsequent pass:
https://circleci.com/gh/prebid/Prebid.js/3857 (fail)
https://circleci.com/gh/prebid/Prebid.js/3858 (pass)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants