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

[camel-main] Junit5-extension-test DoubleRoutesPerClassTest is disabled (possibly would be removed) #6492

Open
JiriOndrusek opened this issue Sep 19, 2024 · 4 comments
Open

[camel-main] Junit5-extension-test DoubleRoutesPerClassTest is disabled (possibly would be removed) #6492

JiriOndrusek opened this issue Sep 19, 2024 · 4 comments
Labels
area/testing release/camel-next test
Milestone
No fix/wont't fix

Comments

@JiriOndrusek
Copy link
Contributor

Bug description

Refactors in Camel (based around AbstractTestSupport) made the scenario covered by DoubleRoutesPerClassTest invalid.
The test are using Lifecycle.PER_CLASS, therefore the routeBuilder from the second test class is not applied and test is failing. Because some of the functionality is marked as deprecated, the whole scenario might not be valid anymore.
@JiriOndrusek JiriOndrusek added the bug Something isn't working label Sep 19, 2024
@JiriOndrusek JiriOndrusek mentioned this issue Sep 19, 2024
Closed
@JiriOndrusek
Copy link
Contributor Author

This is not an issue, I probably used wrong version of Camel or something similar.

@jamesnetherton jamesnetherton added wontfix This will not be worked on and removed bug Something isn't working test area/testing release/camel-next labels Sep 19, 2024
@jamesnetherton jamesnetherton added this to the No fix/wont't fix milestone Sep 19, 2024
@JiriOndrusek
Copy link
Contributor Author

The nightly CI confirmed the error, reopening

@JiriOndrusek JiriOndrusek reopened this Sep 20, 2024
@JiriOndrusek JiriOndrusek removed the wontfix This will not be worked on label Sep 20, 2024
@JiriOndrusek JiriOndrusek removed the wontfix This will not be worked on label Sep 20, 2024
JiriOndrusek added a commit that referenced this issue Sep 20, 2024
@JiriOndrusek
Disabled �[200~DoubleRoutesPerClassTest, see #6492
9959f38
github-actions bot pushed a commit that referenced this issue Sep 21, 2024
@JiriOndrusek @github-actions
Disabled �[200~DoubleRoutesPerClassTest, see #6492
baeb3e5
github-actions bot pushed a commit that referenced this issue Sep 22, 2024
@JiriOndrusek @github-actions
Disabled �[200~DoubleRoutesPerClassTest, see #6492
5841e6e
github-actions bot pushed a commit that referenced this issue Sep 23, 2024
@JiriOndrusek @github-actions
Disabled �[200~DoubleRoutesPerClassTest, see #6492
ebe3c02
@JiriOndrusek
Copy link
Contributor Author

FYI @orpiske

JiriOndrusek added a commit that referenced this issue Sep 24, 2024
@JiriOndrusek
Disabled �[200~DoubleRoutesPerClassTest, see #6492
874cd56
@orpiske
Copy link
Contributor

orpiske commented Sep 24, 2024

@JiriOndrusek thanks for the heads up. Yeah, this test scenario seems like it's invalid from now on

JiriOndrusek added a commit that referenced this issue Sep 27, 2024
@JiriOndrusek
Disabled �[200~DoubleRoutesPerClassTest, see #6492
a30ca11
JiriOndrusek added a commit that referenced this issue Sep 30, 2024
@JiriOndrusek
Disabled �[200~DoubleRoutesPerClassTest, see #6492
0620fdc
JiriOndrusek added a commit that referenced this issue Oct 1, 2024
@JiriOndrusek
Disabled �[200~DoubleRoutesPerClassTest, see #6492
0ce7c35
JiriOndrusek added a commit that referenced this issue Oct 3, 2024
@JiriOndrusek
Disabled �[200~DoubleRoutesPerClassTest, see #6492
4029cd3
JiriOndrusek added a commit to JiriOndrusek/camel-quarkus that referenced this issue Oct 3, 2024
@JiriOndrusek
fixes apache#6492 - removed DoubleRoutesPerClassTest,because it is in…
10e965f 
…valid scenario since Camel 4.9.0
@JiriOndrusek JiriOndrusek mentioned this issue Oct 3, 2024
Open
JiriOndrusek added a commit that referenced this issue Oct 4, 2024
@JiriOndrusek
Disabled �[200~DoubleRoutesPerClassTest, see #6492
d0b4b92
JiriOndrusek added a commit to JiriOndrusek/camel-quarkus that referenced this issue Oct 4, 2024
@JiriOndrusek
fixes apache#6492 - removed DoubleRoutesPerClassTest,because it is in…
24e880e 
…valid scenario since Camel 4.9.0
JiriOndrusek added a commit to JiriOndrusek/camel-quarkus that referenced this issue Oct 7, 2024
@JiriOndrusek
Disabled �[200~DoubleRoutesPerClassTest, see apache#6492
5c2e365
JiriOndrusek added a commit that referenced this issue Oct 9, 2024
@JiriOndrusek
Disabled �[200~DoubleRoutesPerClassTest, see #6492
dc5eb9a
JiriOndrusek added a commit that referenced this issue Oct 10, 2024
@JiriOndrusek
Disabled �[200~DoubleRoutesPerClassTest, see #6492
5344272
jamesnetherton pushed a commit that referenced this issue Oct 15, 2024
@JiriOndrusek @jamesnetherton
Disabled �[200~DoubleRoutesPerClassTest, see #6492
e6a0919
JiriOndrusek added a commit that referenced this issue Oct 17, 2024
@JiriOndrusek
Disabled �[200~DoubleRoutesPerClassTest, see #6492
93f56b8
github-actions bot pushed a commit that referenced this issue Oct 18, 2024
@JiriOndrusek @github-actions
Disabled �[200~DoubleRoutesPerClassTest, see #6492
dccae78
JiriOndrusek added a commit that referenced this issue Oct 25, 2024
@JiriOndrusek
Disabled �[200~DoubleRoutesPerClassTest, see #6492
166e5cc
JiriOndrusek added a commit that referenced this issue Nov 4, 2024
@JiriOndrusek
Disabled �[200~DoubleRoutesPerClassTest, see #6492
717c48e
JiriOndrusek added a commit that referenced this issue Nov 7, 2024
@JiriOndrusek
Disabled �[200~DoubleRoutesPerClassTest, see #6492
e51b28c
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
area/testing release/camel-next test
Projects
None yet
Milestone
No fix/wont't fix
Development

No branches or pull requests
3 participants
@orpiske @jamesnetherton @JiriOndrusek