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

Framework doesn't handle quoted properties properly #584

Closed
fedinskiy opened this issue Oct 3, 2022 · 2 comments · Fixed by #842
Closed

Framework doesn't handle quoted properties properly #584

fedinskiy opened this issue Oct 3, 2022 · 2 comments · Fixed by #842
Assignees
@jedla97
Labels
good first issue Good for newcomers

Comments

@fedinskiy
Copy link
Contributor

First found here:
quarkusio/quarkus#24763
Supposedly, it should either shield quotes before putting them on the command line
OR
keep them in file and not put them on the command line at all.
@michalvavrik michalvavrik mentioned this issue Apr 11, 2023
Closed
@michalvavrik michalvavrik changed the title Framework doesn't hadnle quoted properties properly Framework doesn't handle quoted properties properly Apr 11, 2023
@michalvavrik
Copy link
Member

If I understand upstream issue correctly, this is just about escaping quotes on OS Windows.

@michalvavrik michalvavrik added the good first issue Good for newcomers label Apr 11, 2023
@rsvoboda
Copy link
Member

@fedinskiy pls update the description to provide better context and what's expected

@jedla97 jedla97 mentioned this issue Jul 26, 2023
Merged
10 tasks
jedla97 added a commit to jedla97/quarkus-test-suite that referenced this issue Jul 26, 2023
@jedla97
Enable tests after fixing quarkus-qe/quarkus-test-framework#584.
73eb9ed
jedla97 added a commit to jedla97/quarkus-test-suite that referenced this issue Jul 26, 2023
@jedla97
Enable tests after fixing quarkus-qe/quarkus-test-framework#584.
d78a58f
@jedla97 jedla97 mentioned this issue Jul 26, 2023
Merged
9 tasks
jedla97 added a commit to jedla97/quarkus-test-suite that referenced this issue Aug 1, 2023
@jedla97
Enable tests after fixing quarkus-qe/quarkus-test-framework#584 and d…
b34d994 
…isable only ReactiveRestClientProxyIT.
jedla97 added a commit to jedla97/quarkus-test-suite that referenced this issue Aug 7, 2023
@jedla97
Enable tests after fixing quarkus-qe/quarkus-test-framework#584 and d…
45f5087 
…isable only ReactiveRestClientProxyIT.
michalvavrik added a commit to quarkus-qe/quarkus-test-suite that referenced this issue Aug 7, 2023
@michalvavrik
Merge pull request #1328 from jedla97/remove-suspended-tests
bc05bdc 
Enable tests after fixing quarkus-qe/quarkus-test-framework#584.
fedinskiy pushed a commit to fedinskiy/quarkus-test-suite that referenced this issue Aug 8, 2023
@jedla97 @fedinskiy
Enable tests after fixing quarkus-qe/quarkus-test-framework#584 and d…
ac88398 
…isable only ReactiveRestClientProxyIT.
gtroitsk pushed a commit to gtroitsk/quarkus-test-suite that referenced this issue Aug 17, 2023
@jedla97 @gtroitsk
Enable tests after fixing quarkus-qe/quarkus-test-framework#584 and d…
e1c77c2 
…isable only ReactiveRestClientProxyIT.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jedla97 jedla97

Labels
good first issue Good for newcomers
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix properties not escape " on windows jedla97/quarkus-test-framework
4 participants
@rsvoboda @fedinskiy @jedla97 @michalvavrik