Skip to content

Remove obsolete otherPages variable from the scripting integration tests#18567

Merged
timvandermeij merged 1 commit into
mozilla:masterfrom
timvandermeij:integration-test-otherpages
Aug 6, 2024
Merged

Remove obsolete otherPages variable from the scripting integration tests#18567
timvandermeij merged 1 commit into
mozilla:masterfrom
timvandermeij:integration-test-otherpages

Conversation

@timvandermeij

Copy link
Copy Markdown
Collaborator

It looks like this has accidentally been copy/pasted from the Textfields and focus block, which is the only one in which a test actually uses it. We can therefore safely remove it from all other blocks where no test uses it.

…tests

It looks like this has accidentally been copy/pasted from the
`Textfields and focus` block, which is the only one in which a test
actually uses it. We can therefore safely remove it from all other
blocks where no test uses it.
@timvandermeij

Copy link
Copy Markdown
Collaborator Author

/botio integrationtest

@moz-tools-bot

Copy link
Copy Markdown
Collaborator

From: Bot.io (Linux m4)


Received

Command cmd_integrationtest from @timvandermeij received. Current queue size: 0

Live output at: http://54.241.84.105:8877/125e9dc68f63bbb/output.txt

@moz-tools-bot

Copy link
Copy Markdown
Collaborator

From: Bot.io (Windows)


Received

Command cmd_integrationtest from @timvandermeij received. Current queue size: 1

Live output at: http://54.193.163.58:8877/24bc6a459099cac/output.txt

@moz-tools-bot

Copy link
Copy Markdown
Collaborator

From: Bot.io (Linux m4)


Success

Full output at http://54.241.84.105:8877/125e9dc68f63bbb/output.txt

Total script time: 8.44 mins

  • Integration Tests: Passed

@moz-tools-bot

Copy link
Copy Markdown
Collaborator

From: Bot.io (Windows)


Failed

Full output at http://54.193.163.58:8877/24bc6a459099cac/output.txt

Total script time: 17.82 mins

  • Integration Tests: FAILED

@calixteman calixteman left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copy & paste side effects... sorry about that.
Thank you.

@timvandermeij timvandermeij merged commit af012c7 into mozilla:master Aug 6, 2024
@timvandermeij timvandermeij deleted the integration-test-otherpages branch August 6, 2024 15:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants