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
[WFLY-16190] FilesystemRealmEncryptedTestCase fails when run from a bootable jar #15699
Conversation
@@ -0,0 +1,150 @@ | |||
/* | |||
* JBoss, Home of Professional Open Source. | |||
* Copyright 2020, Red Hat Middleware LLC, and individual contributors |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
2022 :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed!
import org.junit.runner.RunWith; | ||
|
||
/** | ||
* Full authentication tests for Elytron Encrypted Filesystem Realm. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Since this is testing the wrong secret key case specifically, it would be good to update the description here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated
@RunWith(Arquillian.class) | ||
@RunAsClient | ||
@ServerSetup({FilesystemRealmEncryptedSecretKeyTestCase.SetUpTask.class}) | ||
public class FilesystemRealmEncryptedSecretKeyTestCase { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would also be good to update the class name to something like FilesystemRealmEncryptedWrongSecretKeyTestCase
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated
ce0db63
to
1c0f02a
Compare
a22a0e5
to
97ff84c
Compare
@Ashpan This isn't required for this to be merged, but if you need to update this branch for some other reason please correct the commit message to remove the duplicate text. |
@bstansberry I don't see any duplicated text on the commit message, I only saw it on the title of this PR, which you already fixed. |
https://issues.redhat.com/browse/WFLY-16190