Fix benchmarks

Author: garydgregory

pom.xml

@@ -598,13 +598,45 @@
       </build>
     </profile>
     <profile>
+      <!-- mvn clean test -P benchmark -->
       <id>benchmark</id>
       <properties>
         <skipTests>true</skipTests>
         <benchmark>org.apache</benchmark>
       </properties>
+      <dependencies>
+        <dependency>
+          <groupId>org.openjdk.jmh</groupId>
+          <artifactId>jmh-core</artifactId>
+          <version>${commons.jmh.version}</version>
+          <scope>test</scope>
+        </dependency>
+        <dependency>
+          <groupId>org.openjdk.jmh</groupId>
+          <artifactId>jmh-generator-annprocess</artifactId>
+          <version>${commons.jmh.version}</version>
+          <scope>test</scope>
+        </dependency>
+        <dependency>
+          <groupId>org.apache.commons</groupId>
+          <artifactId>commons-collections4</artifactId>
+          <version>4.5.0</version>
+          <scope>test</scope>
+        </dependency>
+      </dependencies>
       <build>
         <plugins>
+          <!-- Enable the compilation of the benchmarks -->
+          <plugin>
+            <artifactId>maven-compiler-plugin</artifactId>
+            <version>${commons.compiler.version}</version>
+            <configuration combine.self="override">
+              <testIncludes>
+                <testInclude>**/*</testInclude>
+              </testIncludes>
+            </configuration>
+          </plugin>
+          <!-- Hook the benchmarks to the test phase -->
           <plugin>
             <groupId>org.codehaus.mojo</groupId>
             <artifactId>exec-maven-plugin</artifactId>
@@ -620,12 +652,12 @@
                   <executable>java</executable>
                   <arguments>
                     <argument>-classpath</argument>
-                    <classpath/>
+                    <classpath />
                     <argument>org.openjdk.jmh.Main</argument>
                     <argument>-rf</argument>
                     <argument>json</argument>
                     <argument>-rff</argument>
-                    <argument>target/jmh-result.${benchmark}.json</argument>
+                    <argument>target/jmh-result.json</argument>
                     <argument>${benchmark}</argument>
                   </arguments>
                 </configuration>

src/test/java/org/apache/commons/text/jmh/LongestCommonSubsequencePerformance.java

@@ -134,31 +134,31 @@ public void setup() {
     }
 
     @Benchmark
-    void testLCS(final InputData data) {
+    public void testLCS(final InputData data) {
         final LongestCommonSubsequence lcs = new LongestCommonSubsequence();
         for (final Pair<CharSequence, CharSequence> input : data.inputs) {
             lcs.longestCommonSubsequence(input.getLeft(), input.getRight());
         }
     }
 
     @Benchmark
-    void testLCSBaseline(final InputData data) {
+    public void testLCSBaseline(final InputData data) {
         final BaselineLongestCommonSubsequence lcs = new BaselineLongestCommonSubsequence();
         for (final Pair<CharSequence, CharSequence> input : data.inputs) {
             lcs.longestCommonSubsequence(input.getLeft(), input.getRight());
         }
     }
 
     @Benchmark
-    void testLCSLen(final InputData data) {
+    public void testLCSLen(final InputData data) {
         final LongestCommonSubsequence lcs = new LongestCommonSubsequence();
         for (final Pair<CharSequence, CharSequence> input : data.inputs) {
             lcs.apply(input.getLeft(), input.getRight());
         }
     }
 
     @Benchmark
-    void testLCSLenBaseline(final InputData data) {
+    public void testLCSLenBaseline(final InputData data) {
         final BaselineLongestCommonSubsequence lcs = new BaselineLongestCommonSubsequence();
         for (final Pair<CharSequence, CharSequence> input : data.inputs) {
             lcs.apply(input.getLeft(), input.getRight());