Extracting common code in InstallationManagerTestCase

wildfly · Aug 8, 2024 · a5e426e · a5e426e
1 parent 6cb4aaf
commit a5e426e
Show file tree

Hide file tree

Showing 19 changed files with 1,014 additions and 1,502 deletions.
diff --git a/installation-manager/src/main/java/org/wildfly/core/instmgr/InstMgrHistoryHandler.java b/installation-manager/src/main/java/org/wildfly/core/instmgr/InstMgrHistoryHandler.java
@@ -6,6 +6,8 @@
 package org.wildfly.core.instmgr;
 
 import java.nio.file.Path;
+import java.text.DateFormat;
+import java.util.Date;
 import java.util.List;
 import java.util.Locale;
 
@@ -53,7 +55,10 @@ public void execute(OperationContext context, ModelNode operation) throws Operat
                     for (HistoryResult hr : history) {
                         ModelNode entry = new ModelNode();
                         entry.get(InstMgrConstants.HISTORY_RESULT_HASH).set(hr.getName());
-                        entry.get(InstMgrConstants.HISTORY_RESULT_TIMESTAMP).set(hr.timestamp().toString());
+                        if (hr.timestamp() != null) {
+                            final java.util.Date timestamp = Date.from(hr.timestamp());
+                            entry.get(InstMgrConstants.HISTORY_RESULT_TIMESTAMP).set(DateFormat.getInstance().format(timestamp));
+                        }
                         entry.get(InstMgrConstants.HISTORY_RESULT_TYPE).set(hr.getType().toLowerCase(Locale.ENGLISH));
                         if (hr.getVersions() != null && !hr.getVersions().isEmpty()) {
                             final ModelNode versions = entry.get(InstMgrConstants.HISTORY_RESULT_CHANNEL_VERSIONS);

diff --git a/...s/test/integration/domain/installationmanager/InstallationManagerIntegrationTestCase.java b/...s/test/integration/domain/installationmanager/InstallationManagerIntegrationTestCase.java
diff --git a/...rc/test/java/org/jboss/as/test/integration/domain/suites/SimpleRbacProviderTestSuite.java b/...rc/test/java/org/jboss/as/test/integration/domain/suites/SimpleRbacProviderTestSuite.java
@@ -6,6 +6,7 @@
 package org.jboss.as.test.integration.domain.suites;
 
 import java.io.IOException;
+import java.net.URL;
 
 import org.jboss.as.test.integration.domain.extension.ExtensionSetup;
 import org.jboss.as.test.integration.domain.management.util.DomainTestSupport;
@@ -125,10 +126,12 @@ public static boolean isTestSuiteEnabled() {
 
     private static void createTestModule() throws IOException {
         testModule = new TestModule(MODULE_NAME, "org.wildfly.installation-manager.api");
+        final URL serviceLoader = SimpleRbacProviderTestSuite.class.getClassLoader()
+                .getResource("org/wildfly/test/installationmanager/services/org.wildfly.installationmanager.spi.InstallationManagerFactory");
         testModule.addResource("test-mock-installation-manager.jar")
                 .addClass(TestInstallationManager.class)
                 .addClass(TestInstallationManagerFactory.class)
-                .addAsManifestResource("META-INF/services/org.wildfly.installationmanager.spi.InstallationManagerFactory",
+                .addAsManifestResource(serviceLoader,
                         "services/org.wildfly.installationmanager.spi.InstallationManagerFactory");
         testModule.create(true);
     }

diff --git a/.../jboss/as/test/manualmode/installationmanager/InstallationManagerIntegrationTestCase.java b/.../jboss/as/test/manualmode/installationmanager/InstallationManagerIntegrationTestCase.java
diff --git a/...esources/META-INF/services/org.wildfly.installationmanager.spi.InstallationManagerFactory b/...esources/META-INF/services/org.wildfly.installationmanager.spi.InstallationManagerFactory
diff --git a/...sources/org/jboss/as/test/manualmode/installationmanager/test-repo-one/dummy_file_one.txt b/...sources/org/jboss/as/test/manualmode/installationmanager/test-repo-one/dummy_file_one.txt
diff --git a/.../jboss/as/test/manualmode/installationmanager/test-repo-one/maven-repository/artifact-one b/.../jboss/as/test/manualmode/installationmanager/test-repo-one/maven-repository/artifact-one
diff --git a/.../test/manualmode/installationmanager/test-repo-one/other-directory-one/dummy_file_one.txt b/.../test/manualmode/installationmanager/test-repo-one/other-directory-one/dummy_file_one.txt
diff --git a/...sources/org/jboss/as/test/manualmode/installationmanager/test-repo-two/dummy_file_two.txt b/...sources/org/jboss/as/test/manualmode/installationmanager/test-repo-two/dummy_file_two.txt
diff --git a/.../jboss/as/test/manualmode/installationmanager/test-repo-two/maven-repository/artifact-two b/.../jboss/as/test/manualmode/installationmanager/test-repo-two/maven-repository/artifact-two
diff --git a/.../test/manualmode/installationmanager/test-repo-two/other-directory-two/dummy_file_two.txt b/.../test/manualmode/installationmanager/test-repo-two/other-directory-two/dummy_file_two.txt