Skip to content

add support for clean test output#29

Merged
jdnurme merged 1 commit intomainfrom
update-unit-test-output
Apr 1, 2024
Merged

add support for clean test output#29
jdnurme merged 1 commit intomainfrom
update-unit-test-output

Conversation

@jdnurme
Copy link
Collaborator

@jdnurme jdnurme commented Apr 1, 2024

Add config so that per-test output is cleaner for presubmits.

  • Tests pass
  • Appropriate changes to documentation are included in the PR

@jdnurme jdnurme requested review from MattIrv and bernardhan33 April 1, 2024 20:28
@jdnurme jdnurme self-assigned this Apr 1, 2024
Copy link
Collaborator

@MattIrv MattIrv left a comment

Choose a reason for hiding this comment

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

The linked Kokoro presubmit run doesn't seem to show individual test output and says there was an xml processing error. Would you have expected it to?

@jdnurme
Copy link
Collaborator Author

jdnurme commented Apr 1, 2024

The linked Kokoro presubmit run doesn't seem to show individual test output and says there was an xml processing error. Would you have expected it to?

I think kokoro config changes don't propagate with the CL, but i'm not 100% certain. Plan was to submit this and verify if it works on future runs. Shouldn't cause any harm to submit either way.

@jdnurme jdnurme merged commit 5d4584a into main Apr 1, 2024
@jdnurme jdnurme deleted the update-unit-test-output branch April 5, 2024 17:12
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