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

Re-evaluate these tests #1831

Open
Rikkola opened this issue Nov 20, 2023 · 2 comments
Open

Re-evaluate these tests #1831

Rikkola opened this issue Nov 20, 2023 · 2 comments

Comments

@Rikkola
Copy link

Rikkola commented Nov 20, 2023

Description

This ticket will list flaky tests. The tests will be disabled or ignored from the build.

org.acme.travel.tests.multimessaging.springboot.MultiMessagingIT.testProcess
org.acme.travel.tests.messaging.springboot.MessagingIT.testProcess

Implementation ideas

To get a test
back it needs to be fixed. Removing a test is also an option.

@fjtirado
Copy link
Contributor

@Rikkola These test are actually quite important to ensure messaging is working on springboot.

@Rikkola
Copy link
Author

Rikkola commented Jan 24, 2024

@fjtirado I understand. The problem is they set the entire team that is working on the repository back. Especially now that this is under Apache and we are trying to build a bigger community. Failing builds mean a broken project and not getting changes in due to random issues pushes people away.

For this reason if the test is important, we need to find a way to do it better. I don't know if trying to find the solution as a team helps.

The reason for just commenting them out ( this test was just one of many ) is that without that, there is no interest in fixing them. Also each failure they cause, causes trouble for the project. Decreasing the bandwidth to run builds and increasing the healthiness of the project. If they need to be fixed, they can be fixed once the area they affect has changes done on them.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: 📋 Backlog
Development

No branches or pull requests

2 participants