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

[WFCORE-193] : Move org.jboss.as.test.smoke.mgmt.servermodule tests from full to core. #403

Merged
merged 1 commit into from Dec 23, 2014
Merged

[WFCORE-193] : Move org.jboss.as.test.smoke.mgmt.servermodule tests from full to core. #403

merged 1 commit into from Dec 23, 2014

Conversation

ehsavoie
Copy link
Contributor

Removing sar deployments.
Simplifying test and updating pom with required dependencies.

Jira: https://issues.jboss.org/browse/WFCORE-193

@wildfly-ci
Copy link

Windows Build 683 is now running using a merge of b54a213

@wildfly-ci
Copy link

Core - Full Integration Build 268 is now running using a merge of b54a213

@wildfly-ci
Copy link

Linux Build 977 is now running using a merge of b54a213

@wildfly-ci
Copy link

Windows Build 683 outcome was SUCCESS using a merge of b54a213
Summary: Tests passed: 2920, ignored: 95 Build time: 0:15:48

@wildfly-ci
Copy link

Linux Build 977 outcome was SUCCESS using a merge of b54a213
Summary: Tests passed: 2920, ignored: 95 Build time: 0:19:39

@wildfly-ci
Copy link

Core - Full Integration Build 268 outcome was SUCCESS using a merge of b54a213
Summary: Tests passed: 2841, ignored: 431 Build time: 0:51:21

bstansberry
bstansberry reviewed Dec 17, 2014
View reviewed changes
.../rbac/src/test/java/org/jboss/as/test/integration/mgmt/access/DeploymentScannerTestCase.java
@@ -43,6 +43,7 @@

import org.jboss.as.controller.client.ModelControllerClient;
import org.jboss.as.protocol.StreamUtils;
import org.jboss.as.test.deployment.DeploymentScannerSetupTask;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Unused import

@bstansberry
Copy link
Contributor

This looks fine, other than the comments I made that are really about using this migration as a chance to clean up some really old problems in class names and javadoc.

@wildfly-ci
Copy link

Windows Build 698 is now running using a merge of 666e554

@wildfly-ci
Copy link

Core - Full Integration Build 282 is now running using a merge of 666e554

@wildfly-ci
Copy link

Linux Build 993 is now running using a merge of 666e554

@wildfly-ci
Copy link

Windows Build 698 outcome was FAILURE using a merge of 666e554
Summary: Tests failed: 2 (2 new), passed: 2918, ignored: 95 Build time: 0:16:25

Build problems:

Failed tests detected

Failed tests

org.wildfly.core.test.standalone.mgmt.api.DeploymentTestCase.testDeploymentStreamApi: java.lang.IllegalArgumentException: org/wildfly/core/test/standalone/mgmt/api/marker.txt not found in classloader sun.misc.Launcher$AppClassLoader@15c7850
    at org.jboss.shrinkwrap.api.asset.ClassLoaderAsset.<init>(ClassLoaderAsset.java:70)
    at org.jboss.shrinkwrap.api.asset.ClassLoaderAsset.<init>(ClassLoaderAsset.java:45)

org.wildfly.core.test.standalone.mgmt.api.DeploymentTestCase.testDeploymentFileApi: java.lang.IllegalArgumentException: org/wildfly/core/test/standalone/mgmt/api/marker.txt not found in classloader sun.misc.Launcher$AppClassLoader@15c7850
    at org.jboss.shrinkwrap.api.asset.ClassLoaderAsset.<init>(ClassLoaderAsset.java:70)
    at org.jboss.shrinkwrap.api.asset.ClassLoaderAsset.<init>(ClassLoaderAsset.java:45)

@wildfly-ci
Copy link

Linux Build 993 outcome was FAILURE using a merge of 666e554
Summary: Tests failed: 2 (2 new), passed: 2918, ignored: 95 Build time: 0:19:46

Build problems:

Failed tests detected

Failed tests

org.wildfly.core.test.standalone.mgmt.api.DeploymentTestCase.testDeploymentStreamApi: java.lang.IllegalArgumentException: org/wildfly/core/test/standalone/mgmt/api/marker.txt not found in classloader sun.misc.Launcher$AppClassLoader@868c6d
    at org.jboss.shrinkwrap.api.asset.ClassLoaderAsset.<init>(ClassLoaderAsset.java:70)
    at org.jboss.shrinkwrap.api.asset.ClassLoaderAsset.<init>(ClassLoaderAsset.java:45)

org.wildfly.core.test.standalone.mgmt.api.DeploymentTestCase.testDeploymentFileApi: java.lang.IllegalArgumentException: org/wildfly/core/test/standalone/mgmt/api/marker.txt not found in classloader sun.misc.Launcher$AppClassLoader@868c6d
    at org.jboss.shrinkwrap.api.asset.ClassLoaderAsset.<init>(ClassLoaderAsset.java:70)
    at org.jboss.shrinkwrap.api.asset.ClassLoaderAsset.<init>(ClassLoaderAsset.java:45)

@ehsavoie
[WFCORE-193] : Move org.jboss.as.test.smoke.mgmt.servermodule tests f…
5e36331 
…rom full to core.

Removing sar deployments.
Simplifying test and updating pom with required dependencies.
@wildfly-ci
Copy link

Core - Full Integration Build 283 is now running using a merge of 5e36331

@wildfly-ci
Copy link

Windows Build 699 is now running using a merge of 5e36331

@wildfly-ci
Copy link

Linux Build 994 is now running using a merge of 5e36331

@wildfly-ci
Copy link

Windows Build 699 outcome was SUCCESS using a merge of 5e36331
Summary: Tests passed: 2920, ignored: 95 Build time: 0:16:36

@wildfly-ci
Copy link

Linux Build 994 outcome was SUCCESS using a merge of 5e36331
Summary: Tests passed: 2920, ignored: 95 Build time: 0:20:01

@wildfly-ci
Copy link

Core - Full Integration Build 283 outcome was SUCCESS using a merge of 5e36331
Summary: Tests passed: 2832, ignored: 430 Build time: 0:42:35

@bstansberry bstansberry added the ready-for-merge This PR is ready to be merged and fulfills all requirements label Dec 22, 2014
bstansberry added a commit that referenced this pull request Dec 23, 2014
@bstansberry
Merge pull request #403 from ehsavoie/WFCORE-193
913b671 
[WFCORE-193] : Move org.jboss.as.test.smoke.mgmt.servermodule tests from full to core.
@bstansberry bstansberry merged commit 913b671 into wildfly:master Dec 23, 2014
@ehsavoie ehsavoie deleted the WFCORE-193 branch February 9, 2015 09:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
ready-for-merge This PR is ready to be merged and fulfills all requirements
Projects
None yet
Milestone
No milestone
3 participants
@ehsavoie @wildfly-ci @bstansberry