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

WIP: Reproduce flaky OutboxProcessorIntegrationTest.java #162

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

magro
Copy link
Contributor

@magro magro commented Jul 15, 2023

Try to reproduce flaky test, don't merge.

The PR is opened to trigger the build action.

@github-actions
Copy link

github-actions bot commented Jul 15, 2023

🦙 MegaLinter status: ✅ SUCCESS

Descriptor Linter Files Fixed Errors Elapsed time
✅ EDITORCONFIG editorconfig-checker 3 0 0.02s

See detailed report in MegaLinter reports
Set VALIDATE_ALL_CODEBASE: true in mega-linter.yml to validate all sources, not only the diff

MegaLinter is graciously provided by OX Security

@magro magro force-pushed the OutboxProcessorIntegrationTest.java branch from 9b4c06e to 504c405 Compare November 4, 2024 17:37
@magro
Copy link
Contributor Author

magro commented Nov 4, 2024

OutboxProcessorIntegrationTest > should_processRecordsInOrder_whenKafkaIsTemporarilyNotAvailable() FAILED
    org.opentest4j.AssertionFailedError: OutboxRecord id and x-sequence headers do not match ==> expected: <602> but was: <541>

See https://github.com/tomorrow-one/transactional-outbox/actions/runs/11669616978/job/32499562062?pr=162#step:6:5421

See e.g.
https://github.com/tomorrow-one/transactional-outbox/actions/runs/11669616978/job/32497886155?pr=162#step:6:4777
- where `IOException: [404] toxic not found` is thrown. This happens
with successful and failed tests, i.e. it doesn't seem to be causing the
error we're hunting.
@magro magro force-pushed the OutboxProcessorIntegrationTest.java branch from acea461 to 6b81c93 Compare November 4, 2024 22:18
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.

1 participant