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

Junit5-extension-test DoubleRoutesPerClassTest is disabled (possibly would be removed) #6492

Closed
JiriOndrusek opened this issue Sep 19, 2024 · 4 comments · Fixed by #6582
Closed

Junit5-extension-test DoubleRoutesPerClassTest is disabled (possibly would be removed) #6492

JiriOndrusek opened this issue Sep 19, 2024 · 4 comments · Fixed by #6582
Labels
area/testing test
Milestone
3.18.0

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 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
Merged
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
JiriOndrusek added a commit that referenced this issue Nov 15, 2024
@JiriOndrusek
Disabled �[200~DoubleRoutesPerClassTest, see #6492
9e07a64
JiriOndrusek added a commit that referenced this issue Nov 19, 2024
@JiriOndrusek
Disabled �[200~DoubleRoutesPerClassTest, see #6492
ae8efdb
JiriOndrusek added a commit that referenced this issue Nov 21, 2024
@JiriOndrusek
Disabled �[200~DoubleRoutesPerClassTest, see #6492
67700e7
JiriOndrusek added a commit that referenced this issue Nov 22, 2024
@JiriOndrusek
Disabled �[200~DoubleRoutesPerClassTest, see #6492
57dc0f7
github-actions bot pushed a commit that referenced this issue Nov 25, 2024
@JiriOndrusek @github-actions
Disabled �[200~DoubleRoutesPerClassTest, see #6492
3f144db
JiriOndrusek added a commit that referenced this issue Nov 28, 2024
@JiriOndrusek
Disabled �[200~DoubleRoutesPerClassTest, see #6492
672473c
JiriOndrusek added a commit that referenced this issue Nov 29, 2024
@JiriOndrusek
Disabled �[200~DoubleRoutesPerClassTest, see #6492
121366e
github-actions bot pushed a commit that referenced this issue Nov 30, 2024
@JiriOndrusek @github-actions
Disabled �[200~DoubleRoutesPerClassTest, see #6492
0ab8a17
github-actions bot pushed a commit that referenced this issue Dec 2, 2024
@JiriOndrusek @github-actions
Disabled �[200~DoubleRoutesPerClassTest, see #6492
4744929
github-actions bot pushed a commit that referenced this issue Dec 4, 2024
@JiriOndrusek @github-actions
Disabled �[200~DoubleRoutesPerClassTest, see #6492
bfddaf6
@jamesnetherton jamesnetherton changed the title [camel-main] Junit5-extension-test DoubleRoutesPerClassTest is disabled (possibly would be removed) Junit5-extension-test DoubleRoutesPerClassTest is disabled (possibly would be removed) Dec 4, 2024
JiriOndrusek added a commit to JiriOndrusek/camel-quarkus that referenced this issue Dec 10, 2024
@JiriOndrusek
fixes apache#6492 - removed DoubleRoutesPerClassTest,because it is in…
3bbdd6e 
…valid scenario since Camel 4.9.0
jamesnetherton pushed a commit that referenced this issue Dec 10, 2024
@JiriOndrusek @jamesnetherton
fixes #6492 - removed DoubleRoutesPerClassTest,because it is invalid …
f854de7 
…scenario since Camel 4.9.0
@github-actions github-actions bot modified the milestones: No fix/wont't fix, 3.18.0 Dec 10, 2024
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 test
Projects
None yet
Milestone
3.18.0
Development

Successfully merging a pull request may close this issue.

[camel-main] fixes #6492 - removed DoubleRoutesPerClassTest JiriOndrusek/camel-quarkus
3 participants
@orpiske @jamesnetherton @JiriOndrusek