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

ci: fix flashy tests #220

Closed
wants to merge 1 commit into from
Closed

Conversation

Scott8440
Copy link
Contributor

Summary: This test was failing in CI because the expected output was not what was actually being logged. It was just a simple miss and not indicative of any other problem so this just adjusts the expected result.

Test Plan: CI

Differential Revision: D46232899

Reviewed By: smithscott

Summary:
Pull Request resolved: facebook#220

This test was failing in CI because the expected output was not what was actually being logged. It was just a simple miss and not indicative of any other problem so this just adjusts the expected result.

Test Plan: CI

Differential Revision: D46232899

Reviewed By: smithscott

fbshipit-source-id: 88910d4ea2dae25c521f246bb5c74d5bd3d3dfad
Scott8440 added a commit to Scott8440/openbmc that referenced this pull request May 26, 2023
Summary:
Pull Request resolved: facebook#220

This test was failing in CI because the expected output was not what was actually being logged. It was just a simple miss and not indicative of any other problem so this just adjusts the expected result.

Test Plan: CI

Differential Revision: D46232899

Reviewed By: smithscott

fbshipit-source-id: 116284fe69a282d9a77c1491acfe270a6af8ccc2
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D46232899

@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D46232899

@facebook-github-bot
Copy link
Contributor

This pull request has been merged in 642eda5.

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.

None yet

2 participants