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

Detected new tests and Opened PR on jinjava project #1150

Merged
merged 2 commits into from
Dec 7, 2023

Conversation

NDR0216
Copy link
Contributor

@NDR0216 NDR0216 commented Dec 2, 2023

I detected new flaky tests on jinjava project, and I opened HubSpot/jinjava#1133 to address the problem.

@darko-marinov
Copy link
Contributor

@zzjas can you please check the logs?

@NDR0216 did you find the commit using git bisect or something else?

@NDR0216
Copy link
Contributor Author

NDR0216 commented Dec 2, 2023

@zzjas can you please check the logs?

@NDR0216 did you find the commit using git bisect or something else?

yes. I find it by using git bisect.

@zzjas
Copy link
Contributor

zzjas commented Dec 2, 2023

The log only has 3 runs. Can you try running NonDex with more runs? I tried 10 on the commit you found and got some test failures.

@darko-marinov
Copy link
Contributor

-DnondexRuns=10

@NDR0216
Copy link
Contributor Author

NDR0216 commented Dec 6, 2023

The log only has 3 runs. Can you try running NonDex with more runs? I tried 10 on the commit you found and got some test failures.

yup. You are right. The test is still flaky.
I opened HubSpot/jinjava#1133 to fix this test and other tests.
Should I close this PR and create a new one, or modify this PR instead?

@zzjas
Copy link
Contributor

zzjas commented Dec 6, 2023

Please just modify this PR.

@NDR0216 NDR0216 changed the title Developer fixed itHandlesImportInDeferredIf Opened PR on jinjava project Dec 6, 2023
@NDR0216 NDR0216 changed the title Opened PR on jinjava project Detected new tests and Opened PR on jinjava project Dec 6, 2023
@NDR0216
Copy link
Contributor Author

NDR0216 commented Dec 6, 2023

Please just modify this PR.

I updated this PR

@zzjas
Copy link
Contributor

zzjas commented Dec 6, 2023

Ok to merge @darko-marinov

@darko-marinov darko-marinov merged commit 0ec270a into TestingResearchIllinois:main Dec 7, 2023
1 check passed
@NDR0216 NDR0216 deleted the patch-1 branch December 7, 2023 23:17
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.

3 participants