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.

Sign up for GitHub

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

Jump to bottom

https://github.com/google/TestParameterInjector,5d67f8cb66481638bcd117227bd0103c32c84491 #1192

Open
bjchris32 opened this issue Dec 5, 2023 · 0 comments
Open

https://github.com/google/TestParameterInjector,5d67f8cb66481638bcd117227bd0103c32c84491 #1192

bjchris32 opened this issue Dec 5, 2023 · 0 comments

Comments

@bjchris32
Copy link
Contributor

Though the PR google/TestParameterInjector#41 has been merged, there are still 2 sources of non-determinism found which developers think it would change the use cases in clients.

The following are the sources that is not merged in the tentative PR:

  1. https://github.com/bjchris32/TestParameterInjector/pull/2/files#diff-f0f3c84fc90c4da3bd5e1267e7b74d5774029389e86dfe1bba895740b8b886d0R323
  2. https://github.com/bjchris32/TestParameterInjector/pull/2/files#diff-f0f3c84fc90c4da3bd5e1267e7b74d5774029389e86dfe1bba895740b8b886d0R932

For reference, the reproduce steps and logs are left in the original PR: google/TestParameterInjector#41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@bjchris32