Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Use the (viewer) parseQueryString helper function in the reference tests #14581

Merged
merged 1 commit into from
Feb 19, 2022

Conversation

Snuffleupagus
Copy link
Collaborator

Rather than re-implementing this functionality in the Driver class, we can simply re-use the existing parseQueryString helper function instead.

…tests

Rather than re-implementing this functionality in the `Driver` class, we can simply re-use the existing `parseQueryString` helper function instead.
@Snuffleupagus
Copy link
Collaborator Author

/botio browsertest

@pdfjsbot
Copy link

From: Bot.io (Windows)


Received

Command cmd_browsertest from @Snuffleupagus received. Current queue size: 0

Live output at: http://54.193.163.58:8877/33794278db129f5/output.txt

@pdfjsbot
Copy link

From: Bot.io (Linux m4)


Received

Command cmd_browsertest from @Snuffleupagus received. Current queue size: 0

Live output at: http://54.241.84.105:8877/1c55e4f7a0cf8d3/output.txt

@pdfjsbot
Copy link

From: Bot.io (Windows)


Success

Full output at http://54.193.163.58:8877/33794278db129f5/output.txt

Total script time: 19.28 mins

  • Regression tests: Passed

@pdfjsbot
Copy link

From: Bot.io (Linux m4)


Failed

Full output at http://54.241.84.105:8877/1c55e4f7a0cf8d3/output.txt

Total script time: 19.66 mins

  • Regression tests: FAILED
  different ref/snapshot: 11
  different first/second rendering: 2

Image differences available at: http://54.241.84.105:8877/1c55e4f7a0cf8d3/reftest-analyzer.html#web=eq.log

@timvandermeij timvandermeij merged commit 964601b into mozilla:master Feb 19, 2022
@timvandermeij
Copy link
Contributor

Thanks!

@Snuffleupagus Snuffleupagus deleted the Driver-parseQueryString branch February 19, 2022 15:12
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants