Skip to content

Update expectations for --product servodriver#39603

Merged
mrobinson merged 1 commit intomainfrom
servodriver-expectations
Oct 1, 2025
Merged

Update expectations for --product servodriver#39603
mrobinson merged 1 commit intomainfrom
servodriver-expectations

Conversation

@delan
Copy link
Copy Markdown
Member

@delan delan commented Oct 1, 2025

this patch updates the expectations for tests that use testdriver.js, which currently only work with --product servodriver (#34683), running the tests with #39587.

expectations for all other tests, which don’t use testdriver.js, are unaffected.

Testing: this patch updates test expectations, though the tests are not yet run in CI

Co-authored-by: Martin Robinson <[email protected]>
Signed-off-by: Delan Azabani <[email protected]>
@delan delan marked this pull request as ready for review October 1, 2025 10:06
@servo-highfive servo-highfive added the S-awaiting-review There is new code that needs to be reviewed. label Oct 1, 2025
@servo-highfive servo-highfive removed the S-awaiting-review There is new code that needs to be reviewed. label Oct 1, 2025
@mrobinson mrobinson added this pull request to the merge queue Oct 1, 2025
@servo-highfive servo-highfive added the S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. label Oct 1, 2025
Merged via the queue into main with commit cd80d27 Oct 1, 2025
30 checks passed
@mrobinson mrobinson deleted the servodriver-expectations branch October 1, 2025 11:18
@servo-highfive servo-highfive removed the S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. label Oct 1, 2025
@yezhizhen
Copy link
Copy Markdown
Member

yezhizhen commented Oct 8, 2025

expectations for all other tests, which don’t use testdriver.js, are unaffected.

Can you show how can I do the same? Run tests in CI and automatically update expectation only for those involve testdriver.js.

@mrobinson
Copy link
Copy Markdown
Member

Can you show how can I do the same? Run tests in CI and automatically update expectation only for those involve testdriver.js.

The process is still pretty manual and there are some scripts involved. I have a new batch of expectations that I hope to upload today. When I do that, I can try to post how I went about generating them.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants