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

PullRequestProcessTest.testRemindReviewer is flaky #984

Closed
nicpuppa opened this issue Nov 17, 2023 · 2 comments · Fixed by #1040
Closed

PullRequestProcessTest.testRemindReviewer is flaky #984

nicpuppa opened this issue Nov 17, 2023 · 2 comments · Fixed by #1040
Assignees
@korthout
Labels
kind/flake Categorizes issue or PR as related to a flaky test

Comments

@nicpuppa
Copy link
Contributor

nicpuppa commented Nov 17, 2023 •
edited
Loading

The test is failing with:

Error:  io.camunda.zeebe.process.test.examples.PullRequestProcessTest.testRemindReviewer -- Time elapsed: 4.264 s <<< FAILURE!
org.opentest4j.AssertionFailedError: Unable to activate 1 jobs, because only 0 were activated.
	at org.junit.jupiter.api.AssertionUtils.fail(AssertionUtils.java:38)
	at org.junit.jupiter.api.Assertions.fail(Assertions.java:138)
	at io.camunda.zeebe.process.test.examples.PullRequestProcessTest.completeServiceTasks(PullRequestProcessTest.java:280)
	at io.camunda.zeebe.process.test.examples.PullRequestProcessTest.completeServiceTask(PullRequestProcessTest.java:269)
	at io.camunda.zeebe.process.test.examples.PullRequestProcessTest.testRemindReviewer(PullRequestProcessTest.java:136)
	at java.base/java.lang.reflect.Method.invoke(Method.java:580)
	at java.base/java.util.ArrayList.forEach(ArrayList.java:1596)
	at java.base/java.util.ArrayList.forEach(ArrayList.java:1596)

More logs here
@nicpuppa nicpuppa added the kind/flake Categorizes issue or PR as related to a flaky test label Nov 17, 2023
@korthout korthout mentioned this issue Dec 1, 2023
Merged
This was referenced Jan 2, 2024
Merged
Merged
Merged
@korthout korthout mentioned this issue Jan 10, 2024
Merged
@ana-vinogradova-camunda
Copy link
Contributor

Just happened to me:

Error:  Tests run: 3, Failures: 1, Errors: 0, Skipped: 0, Time elapsed: 9.016 s <<< FAILURE! -- in io.camunda.zeebe.process.test.examples.PullRequestProcessTest
[zb-actors-0] INFO io.camunda.zeebe.logstreams.impl.log.Sequencer - Closing sequencer for writing
Error:  io.camunda.zeebe.process.test.examples.PullRequestProcessTest.testRemindReviewer -- Time elapsed: 6.121 s <<< FAILURE!
[zb-actors-0] INFO io.camunda.zeebe.logstreams - On closing logstream null close 2 readers
org.opentest4j.AssertionFailedError: Unable to activate 1 jobs, because only 0 were activated.
	at org.junit.jupiter.api.AssertionUtils.fail(AssertionUtils.java:38)
	at org.junit.jupiter.api.Assertions.fail(Assertions.java:138)
	at io.camunda.zeebe.process.test.examples.PullRequestProcessTest.completeServiceTasks(PullRequestProcessTest.java:280)
	at io.camunda.zeebe.process.test.examples.PullRequestProcessTest.completeServiceTask(PullRequestProcessTest.java:269)
	at io.camunda.zeebe.process.test.examples.PullRequestProcessTest.testRemindReviewer(PullRequestProcessTest.java:136)
	at java.base/java.lang.reflect.Method.invoke(Method.java:580)
	at java.base/java.util.ArrayList.forEach(ArrayList.java:1596)

Build here

@koevskinikola
Copy link
Member

Happened again: https://github.com/camunda/zeebe-process-test/actions/runs/7564734652/job/20599341063

@korthout korthout self-assigned this Jan 19, 2024
This was referenced Jan 19, 2024
Merged
Merged
@ghost ghost closed this as completed in 6769a0a Jan 22, 2024
This was referenced Jan 22, 2024
Merged
Merged
Merged
Merged
ghost pushed a commit that referenced this issue Jan 22, 2024
@korthout
merge: #1042
9639158 
1042: [Backport stable/8.2] Unflake tests with published messages r=github-actions[bot] a=backport-action

# Description
Backport of #1040 to `stable/8.2`.

relates to #984

Co-authored-by: Nico Korthout <nico.korthout@camunda.com>
ghost pushed a commit that referenced this issue Jan 22, 2024
@korthout
merge: #1041
4a33ac7 
1041: [Backport stable/8.1] Unflake tests with published messages r=github-actions[bot] a=backport-action

# Description
Backport of #1040 to `stable/8.1`.

relates to #984

Co-authored-by: Nico Korthout <nico.korthout@camunda.com>
ghost pushed a commit that referenced this issue Jan 22, 2024
@korthout
merge: #1043
b395683 
1043: [Backport stable/8.3] Unflake tests with published messages r=github-actions[bot] a=backport-action

# Description
Backport of #1040 to `stable/8.3`.

relates to #984

Co-authored-by: Nico Korthout <nico.korthout@camunda.com>
ghost pushed a commit that referenced this issue Jan 22, 2024
@korthout
merge: #1044
ba01a9a 
1044: [Backport stable/8.4] Unflake tests with published messages r=github-actions[bot] a=backport-action

# Description
Backport of #1040 to `stable/8.4`.

relates to #984

Co-authored-by: Nico Korthout <nico.korthout@camunda.com>
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@korthout korthout

Labels
kind/flake Categorizes issue or PR as related to a flaky test
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Unflake tests with published messages camunda/zeebe-process-test
4 participants
@koevskinikola @korthout @nicpuppa @ana-vinogradova-camunda