Skip to content

Commit

Permalink
Introduce continuous testing
Browse files Browse the repository at this point in the history
  • Loading branch information
stuartwdouglas committed Apr 14, 2021
1 parent 2dc8bc8 commit 811030b
Show file tree
Hide file tree
Showing 94 changed files with 5,968 additions and 560 deletions.
7 changes: 6 additions & 1 deletion bom/application/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -212,6 +212,7 @@
<avro.version>1.10.2</avro.version>
<jacoco.version>0.8.6</jacoco.version>
<testcontainers.version>1.15.2</testcontainers.version>
<aesh-readline.version>2.1</aesh-readline.version>
</properties>

<dependencyManagement>
Expand Down Expand Up @@ -4734,7 +4735,6 @@
</exclusion>
</exclusions>
</dependency>

<dependency>
<groupId>software.amazon.awssdk</groupId>
<artifactId>apache-client</artifactId>
Expand Down Expand Up @@ -5133,6 +5133,11 @@
<classifier>windows-x86_64</classifier>
<version>${grpc.version}</version>
</dependency>
<dependency>
<groupId>org.aesh</groupId>
<artifactId>readline</artifactId>
<version>${aesh-readline.version}</version>
</dependency>

<!-- Picocli -->
<dependency>
Expand Down
23 changes: 21 additions & 2 deletions build-parent/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -493,13 +493,11 @@
<exclude>io.quarkus:quarkus-test-*</exclude>
<exclude>io.rest-assured:*</exclude>
<exclude>org.assertj:*</exclude>
<exclude>org.junit.jupiter:*</exclude>
</excludes>
<includes>
<include>io.quarkus:quarkus-test-*:*:*:test</include>
<include>io.rest-assured:*:*:*:test</include>
<include>org.assertj:*:*:*:test</include>
<include>org.junit.jupiter:*:*:*:test</include>
</includes>
<message>Found test dependencies with wrong scope:</message>
</bannedDependencies>
Expand All @@ -510,6 +508,27 @@
<goal>enforce</goal>
</goals>
</execution>
<execution>
<id>enforce-test-deps-junit-scope</id>
<configuration>
<rules>
<bannedDependencies>
<searchTransitive>false</searchTransitive>
<excludes>
<exclude>org.junit.jupiter:*</exclude>
</excludes>
<includes>
<include>org.junit.jupiter:*:*:*:test</include>
</includes>
<message>Found JUnit dependencies with wrong scope:</message>
</bannedDependencies>
</rules>
<skip>${enforce-test-deps-scope.skip}</skip>
</configuration>
<goals>
<goal>enforce</goal>
</goals>
</execution>
</executions>
</plugin>
<plugin>
Expand Down
27 changes: 27 additions & 0 deletions core/deployment/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,10 @@
<name>Quarkus - Core - Deployment</name>

<dependencies>
<dependency>
<groupId>org.aesh</groupId>
<artifactId>readline</artifactId>
</dependency>
<dependency>
<groupId>org.wildfly.common</groupId>
<artifactId>wildfly-common</artifactId>
Expand Down Expand Up @@ -106,10 +110,33 @@
</exclusions>
<scope>test</scope>
</dependency>
<dependency>
<groupId>org.junit.platform</groupId>
<artifactId>junit-platform-launcher</artifactId>
</dependency>
<dependency>
<groupId>org.junit.jupiter</groupId>
<artifactId>junit-jupiter</artifactId>
</dependency>
</dependencies>

<build>
<plugins>
<plugin>
<artifactId>maven-enforcer-plugin</artifactId>

<executions>
<execution>
<id>enforce-test-deps-junit-scope</id>
<configuration>
<skip>true</skip>
</configuration>
<goals>
<goal>enforce</goal>
</goals>
</execution>
</executions>
</plugin>
<plugin>
<artifactId>maven-compiler-plugin</artifactId>
<configuration>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -60,6 +60,7 @@ public class QuarkusAugmentor {
private final String baseName;
private final Consumer<ConfigBuilder> configCustomizer;
private final boolean rebuild;
private final boolean auxiliaryApplication;

QuarkusAugmentor(Builder builder) {
this.classLoader = builder.classLoader;
Expand All @@ -78,6 +79,7 @@ public class QuarkusAugmentor {
this.deploymentClassLoader = builder.deploymentClassLoader;
this.rebuild = builder.rebuild;
this.devModeType = builder.devModeType;
this.auxiliaryApplication = builder.auxiliaryApplication;
}

public BuildResult run() throws Exception {
Expand Down Expand Up @@ -142,7 +144,7 @@ public BuildResult run() throws Exception {
.produce(new ShutdownContextBuildItem())
.produce(new RawCommandLineArgumentsBuildItem())
.produce(new LaunchModeBuildItem(launchMode,
devModeType == null ? Optional.empty() : Optional.of(devModeType)))
devModeType == null ? Optional.empty() : Optional.of(devModeType), auxiliaryApplication))
.produce(new BuildSystemTargetBuildItem(targetDir, baseName, rebuild,
buildSystemProperties == null ? new Properties() : buildSystemProperties))
.produce(new DeploymentClassLoaderBuildItem(deploymentClassLoader))
Expand Down Expand Up @@ -197,6 +199,7 @@ public static final class Builder {
Consumer<ConfigBuilder> configCustomizer;
ClassLoader deploymentClassLoader;
DevModeType devModeType;
boolean auxiliaryApplication;

public Builder addBuildChainCustomizer(Consumer<BuildChainBuilder> customizer) {
this.buildChainCustomizers.add(customizer);
Expand All @@ -217,6 +220,11 @@ public Builder excludeFromIndexing(Collection<Path> excludedFromIndexing) {
return this;
}

public Builder setAuxiliaryApplication(boolean auxiliaryApplication) {
this.auxiliaryApplication = auxiliaryApplication;
return this;
}

public LaunchMode getLaunchMode() {
return launchMode;
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -11,12 +11,89 @@
/**
* This is used currently only to suppress warnings about unknown properties
* when the user supplies something like: -Dquarkus.test.profile=someProfile or -Dquarkus.test.native-image-profile=someProfile
*
* <p>
* TODO refactor code to actually use these values
*/
@ConfigRoot
public class TestConfig {

/**
* If continuous testing is enabled.
*
* The default value is 'paused', which will allow you to start testing
* from the console or the Dev UI, but will not run tests on startup.
*
* If this is set to 'enabled' then testing will start as soon as the
* application has started.
*
* If this is 'disabled' then continuous testing is not enabled, and can't
* be enabled without restarting the application.
*
*/
@ConfigItem(defaultValue = "paused")
public Mode continuousTesting;

/**
* If output from the running tests should be displayed in the console.
*/
@ConfigItem(defaultValue = "false")
public boolean displayTestOutput;

/**
* Tags that should be included for continuous testing.
*/
@ConfigItem
public Optional<List<String>> includeTags;

/**
* Tags that should be excluded by default with continuous testing.
*
* This is ignored if include-tags has been set.
*
* Defaults to 'slow'
*/
@ConfigItem(defaultValue = "slow")
public Optional<List<String>> excludeTags;

/**
* Tests that should be included for continuous testing. This is a regular expression.
*/
@ConfigItem
public Optional<String> includePattern;

/**
* Tests that should be excluded with continuous testing. This is a regular expression.
*
* This is ignored if include-pattern has been set.
*
*/
@ConfigItem
public Optional<String> excludePattern;
/**
* Disable the testing status/prompt message at the bottom of the console
* and log these messages to STDOUT instead.
*
* Use this option if your terminal does not support ANSI escape sequences.
*/
@ConfigItem(defaultValue = "false")
public boolean basicConsole;

/**
* Disable color in the testing status and prompt messages.
*
* Use this option if your terminal does not support color.
*/
@ConfigItem(defaultValue = "false")
public boolean disableColor;

/**
* If test results and status should be displayed in the console.
*
* If this is false results can still be viewed in the dev console.
*/
@ConfigItem(defaultValue = "true")
public boolean console;

/**
* Duration to wait for the native image to built during testing
*/
Expand All @@ -35,6 +112,16 @@ public class TestConfig {
@ConfigItem
Profile profile;

/**
* Configures the hang detection in @QuarkusTest. If no activity happens (i.e. no test callbacks are called) over
* this period then QuarkusTest will dump all threads stack traces, to help diagnose a potential hang.
*
* Note that the initial timeout (before Quarkus has started) will only apply if provided by a system property, as
* it is not possible to read all config sources until Quarkus has booted.
*/
@ConfigItem(defaultValue = "10m")
Duration hangDetectionTimeout;

@ConfigGroup
public static class Profile {

Expand All @@ -53,4 +140,11 @@ public static class Profile {
@ConfigItem(defaultValue = "")
Optional<List<String>> tags;
}

public enum Mode {
PAUSED,
ENABLED,
DISABLED

}
}
Original file line number Diff line number Diff line change
Expand Up @@ -15,9 +15,12 @@ public final class LaunchModeBuildItem extends SimpleBuildItem {

private final Optional<DevModeType> devModeType;

public LaunchModeBuildItem(LaunchMode launchMode, Optional<DevModeType> devModeType) {
private final boolean auxiliaryApplication;

public LaunchModeBuildItem(LaunchMode launchMode, Optional<DevModeType> devModeType, boolean auxiliaryApplication) {
this.launchMode = launchMode;
this.devModeType = devModeType;
this.auxiliaryApplication = auxiliaryApplication;
}

public LaunchMode getLaunchMode() {
Expand All @@ -26,11 +29,21 @@ public LaunchMode getLaunchMode() {

/**
* The development mode type.
*
* <p>
* Note that even for NORMAL launch modes this could be generating an application for the local side of remote
* dev mode, so this may be set even for launch mode normal.
*/
public Optional<DevModeType> getDevModeType() {
return devModeType;
}

/**
* An Auxiliary Application is a second application running in the same JVM as a primary application.
* <p>
* Currently this is done to allow running tests in dev mode, while the main dev mode process continues to
* run.
*/
public boolean isAuxiliaryApplication() {
return auxiliaryApplication;
}
}

This file was deleted.

Loading

0 comments on commit 811030b

Please sign in to comment.