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

Using both generated and configured specs stoped working in 1.6.5 #2213

Conversation

bjornblomqvist
Copy link

Hi, prior to version 1.6.5 I could you use both a spec file and generated spec from annotations. This behavior is something I need to be able to upgrade to a later version. I am not sure if this feature that I have been using was even ment to work but it did and I would very much appreciate if we could bring it back.

You can find in this pull request a test and a fix for my problem. I think I put the test in the wrong place so I would appreciate if you could help me improve the test. Also as I don't know the code I think the fix might also be in the wrong place.

Best regards
Björn Blomqvist

@bjornblomqvist bjornblomqvist changed the title Using both generated and configured specs stoped working in 1 Using both generated and configured specs stoped working in 1.6.5 Apr 21, 2023
@bjornblomqvist
Copy link
Author

bjornblomqvist commented May 2, 2023

@bnasslahsen what do you think about this pull request?

@bjornblomqvist bjornblomqvist force-pushed the using-both-generated-and-configured-specs-no-longer-works branch from 2214fc7 to de848d4 Compare May 4, 2023 07:22
@bjornblomqvist
Copy link
Author

The only change was to rebase on latest master

I don't fully understand the fix but forcing SpringBootTest runner to
not reuse the context solves the issue with SpringDocApp1451Test.java
always failing with the following.

"WebClientRequestException: Connection refused: localhost/127.0.0.1:9086"

BUGFIX
Being able to use both a spec file and generated spec stopped working
in version 1.6.5. The test shows the behavior found 1.6.4 and the fix
brings back that behavior.

BUGFIX
@bjornblomqvist bjornblomqvist force-pushed the using-both-generated-and-configured-specs-no-longer-works branch from de848d4 to 41a988e Compare May 4, 2023 08:12
@bjornblomqvist
Copy link
Author

Last force push was to remove my employers email address.

@bjornblomqvist
Copy link
Author

@bnasslahsen have you had time to look at this yet?

@bnasslahsen
Copy link
Contributor

@bjornblomqvist,

Not yet.
But it will be my next PR in the project to review.

@bnasslahsen bnasslahsen merged commit 41a988e into springdoc:master Jun 2, 2023
1 check passed
@bnasslahsen
Copy link
Contributor

Thank you @bjornblomqvist for your PR.
It's now merged and also committed to 2.x branch as well.

bnasslahsen added a commit that referenced this pull request Jun 2, 2023
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.

None yet

2 participants