keith-turner commented on code in PR #4051:
URL: https://github.com/apache/accumulo/pull/4051#discussion_r1423188603


##########
test/src/main/java/org/apache/accumulo/test/ScanConsistencyIT.java:
##########
@@ -79,12 +79,57 @@
 public class ScanConsistencyIT extends AccumuloClusterHarness {
 
   private static final Logger log = 
LoggerFactory.getLogger(ScanConsistencyIT.class);
+  private static boolean inTestingContext;
+
+  public static void main(String[] args) {
+    /**
+     * @formatter:off
+     * Note: In order to run main,
+     * 1) Build the project
+     * 2) Copy the accumulo test jar (in /test/target/) to your accumulo 
installation's

Review Comment:
   I think we could further minimize changes to the test by requiring copying 
any needed dependencies like junit.  The following may be one way to do this.
   
   ```
   mvn dependency:copy-dependencies -DincludeGroupIds="org.junit.jupiter"
   cp test/target/dependency/junit-jupiter-* $ACCUMULO_HOME/lib/
   ```
   
   If that works, then would not need the wrappers around the junit assert 
method call, which would minimizes the test changes.



-- 
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]

Reply via email to