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-9777] clean testsuite debris in git-trackable directories. #10962
Conversation
retest this please |
@@ -46,7 +47,7 @@ | |||
public static JavaArchive createDeployment() { | |||
final JavaArchive jar = ShrinkWrap.create(JavaArchive.class, MODULE_NAME + ".jar"); | |||
jar.addPackage(UserTransactionAccessTestCase.class.getPackage()); | |||
jar.as(ZipExporter.class).exportTo(new File(".", jar.getName()), true); | |||
jar.as(ZipExporter.class).exportTo(new File(TestSuiteEnvironment.getTmpDir(), jar.getName()), true); |
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.
I think this like should have just been deleted, there is no reason to export the archive
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.
deleted.
@@ -268,7 +269,7 @@ private String getClientJarPaths() throws IOException { | |||
jar.addManifest() | |||
.addAsManifestResource(WSTrustTestCase.class.getPackage(), "META-INF/clientKeystore.properties", "clientKeystore.properties") | |||
.addAsManifestResource(WSTrustTestCase.class.getPackage(), "META-INF/clientstore.jks", "clientstore.jks"); | |||
File jarFile = new File("jaxws-samples-wsse-policy-trust-client.jar"); | |||
File jarFile = new File(TestSuiteEnvironment.getTmpDir(), "jaxws-samples-wsse-policy-trust-client.jar"); |
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.
Same 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.
it's used to create an URLClassLoader with these security files as resource and put into setContextClassLoader by TestWatcher, then later it retrieves back from WSTrustTestUtils static methods for client env setup.
Retest this please |
Retest this please |
failed ConcurrentCoarseWebFailoverTestCase.testGracefulSimpleFailover in Windows CI does not seem related. |
https://issues.jboss.org/browse/WFLY-9777 clean testsuite debris in git-trackable directories.