Skip to content

Introduce a helper function to create a freetext editor in the integration tests - #20430

Merged
timvandermeij merged 1 commit into
mozilla:masterfrom
timvandermeij:freetext-helper
Nov 16, 2025
Merged

timvandermeij merged 1 commit into
mozilla:masterfrom
timvandermeij:freetext-helper

Conversation

@timvandermeij

Copy link
Copy Markdown
Collaborator

Doing so has a number of advantages:

  • it removes code duplication, thereby improving readability;
  • it removes hardcoded editor IDs, by using the getNextEditorId helper function that was previously introduced for the highlight editor integration tests, thereby improving readability and reusability;
  • it removes potential for intermittent failures by not proceeding until the freetext editor is fully created and all assertions pass, which didn't happen consistently before because the code wasn't centralized.

Comment thread test/integration/freetext_editor_spec.mjs Fixed
Comment thread test/integration/freetext_editor_spec.mjs Fixed
…ation tests

Doing so has a number of advantages:

- it removes code duplication, thereby improving readability;
- it removes hardcoded editor IDs, by using the `getNextEditorId` helper
  function that was previously introduced for the highlight editor
  integration tests, thereby improving readability and reusability;
- it removes potential for intermittent failures by not proceeding until
  the freetext editor is fully created and all assertions pass, which
  didn't happen consistently before because the code wasn't centralized.
@timvandermeij

Copy link
Copy Markdown
Collaborator Author

/botio integrationtest

@moz-tools-bot

Copy link
Copy Markdown
Collaborator

From: Bot.io (Windows)


Received

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

Live output at: http://54.193.163.58:8877/05a6c203234d05f/output.txt

@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/ab5a6c60a5dc6a3/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/ab5a6c60a5dc6a3/output.txt

Total script time: 20.25 mins

  • Integration Tests: Passed

@moz-tools-bot

Copy link
Copy Markdown
Collaborator

From: Bot.io (Windows)


Success

Full output at http://54.193.163.58:8877/05a6c203234d05f/output.txt

Total script time: 44.46 mins

  • Integration Tests: Passed

@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.

LGTM. Thank you.

@timvandermeij
timvandermeij merged commit 603e3a8 into mozilla:master Nov 16, 2025
7 checks passed
@timvandermeij
timvandermeij deleted the freetext-helper branch November 16, 2025 13:50
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.

4 participants