quantranhong1999 commented on code in PR #2813:
URL: https://github.com/apache/james-project/pull/2813#discussion_r2357374879
##########
server/apps/cassandra-app/pom.xml:
##########
@@ -326,6 +326,13 @@
</execution>
</executions>
</plugin>
+ <plugin>
+ <groupId>org.apache.maven.plugins</groupId>
+ <artifactId>maven-surefire-plugin</artifactId>
+ <configuration>
+ <skipTests>true</skipTests>
+ </configuration>
+ </plugin>
Review Comment:
> only keep a test that checks that the app starts successfully for the
bootstrapping part
How about keeping the current Cassandra app test suite, and smoke testing
for a few basic test classes (just like what we do for JMAP IT)?
Given the build time is good now, 1 -> 1.5 hour, keep running the whole
Cassandra app test suite as of today is acceptable though IMO.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
