Skip to content

Commit

Permalink
Don't exclude tests in trino-tests module to only run on CI
Browse files Browse the repository at this point in the history
The tests excluded to run only on CI take < 3 mins to run in total while
the entire module takes ~30 mins.
Merging the CI runs allows more concurrent jobs to run.
  • Loading branch information
hashhar committed Jul 10, 2021
1 parent 96be8f2 commit bbfca82
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 40 deletions.
1 change: 0 additions & 1 deletion .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -248,7 +248,6 @@ jobs:
modules:
- ":trino-main"
- ":trino-tests"
- ":trino-tests -P ci-only"
- ":trino-raptor-legacy"
- ":trino-accumulo"
- ":trino-cassandra"
Expand Down
39 changes: 0 additions & 39 deletions testing/trino-tests/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -251,43 +251,4 @@
<scope>test</scope>
</dependency>
</dependencies>

<build>
<plugins>
<plugin>
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-surefire-plugin</artifactId>
<configuration>
<!-- these tests take a very long time so only run them in the CI server -->
<excludes>
<exclude>**/TestDistributedQueriesNoHashGeneration.java</exclude>
<exclude>**/TestLocalQueries.java</exclude>
<exclude>**/TestNonIterativeDistributedQueries.java</exclude>
</excludes>
</configuration>
</plugin>
</plugins>
</build>

<profiles>
<profile>
<id>ci-only</id>
<build>
<plugins>
<plugin>
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-surefire-plugin</artifactId>
<configuration>
<excludes combine.self="override" />
<includes>
<include>**/TestDistributedQueriesNoHashGeneration.java</include>
<include>**/TestLocalQueries.java</include>
<include>**/TestNonIterativeDistributedQueries.java</include>
</includes>
</configuration>
</plugin>
</plugins>
</build>
</profile>
</profiles>
</project>

0 comments on commit bbfca82

Please sign in to comment.