Use GitHub action to inject Test Optimization (#2674)

* Bump Test Optimization version

* Remove GHA run conditions for debugging purposes

* Revert "Remove GHA run conditions for debugging purposes"

This reverts commit fb04f7c.

---------

Co-authored-by: skarimo <40482491+skarimo@users.noreply.github.com>

Author: nikita-tkachenko-datadog

.github/workflows/test.yml

@@ -128,8 +128,15 @@ jobs:
           java-version: ${{ matrix.java-version }}
           distribution: "temurin"
           cache: "maven"
+      - name: Configure Datadog Test Optimization
+        uses: datadog/test-visibility-github-action@v2
+        with:
+          languages: java
+          api_key: ${{ secrets.DD_API_KEY }}
       - name: Test
         run: ./run-tests.sh
+        env:
+          DD_CIVISIBILITY_COMPILER_PLUGIN_AUTO_CONFIGURATION_ENABLED: "false"
 
   examples:
     runs-on: ubuntu-latest

.github/workflows/test_integration.yml

@@ -65,21 +65,23 @@ jobs:
           java-version: '16'
           distribution: 'temurin'
           cache: 'maven'
+      - name: Configure Datadog Test Optimization
+        uses: datadog/test-visibility-github-action@v2
+        with:
+          languages: java
+          api_key: ${{ secrets.DD_API_KEY }}
       - name: Run integration tests
         run: ./run-tests.sh
         env:
           CI: "true"
-          DD_AGENT_HOST: localhost
           DD_ENV: prod
           DD_HTTP_CLIENT_ERROR_STATUSES: "500-599"
-          DD_INTEGRATION_JUNIT_ENABLED: "true"
-          DD_LOGS_INJECTION: "true"
+          DD_CIVISIBILITY_COMPILER_PLUGIN_AUTO_CONFIGURATION_ENABLED: "false"
           DD_SERVICE: datadog-api-client-java
           DD_TAGS: "team:integrations-tools-and-libraries"
           DD_TEST_CLIENT_API_KEY: ${{ secrets.DD_CLIENT_API_KEY }}
           DD_TEST_CLIENT_APP_KEY: ${{ secrets.DD_CLIENT_APP_KEY }}
           DD_TRACE_ANALYTICS_ENABLED: "true"
-          DD_CIVISIBILITY_ENABLED: "true"
           RECORD: "none"
           SLEEP_AFTER_REQUEST: "${{ vars.SLEEP_AFTER_REQUEST }}"
       - name: Post failure status check

pom.xml

@@ -54,7 +54,7 @@
                         <version>3.0.0-M5</version>
                         <configuration>
                             <runOrder>random</runOrder>
-                            <argLine>-Ddd.integration.junit.enabled=true -Ddd.prioritization.type=ENSURE_TRACE -Ddd.jmxfetch.enabled=false -javaagent:${basedir}/target/dd-java-agent-${dd-java-agent-version}.jar -Xms512m -Xmx3500m -DsurefireForkNumber=${surefire.forkNumber} --add-exports java.base/sun.security.x509=ALL-UNNAMED --add-opens java.base/sun.net.www.protocol.https=ALL-UNNAMED --add-opens java.base/java.net=ALL-UNNAMED</argLine>
+                            <argLine>-Xms512m -Xmx3500m -DsurefireForkNumber=${surefire.forkNumber} --add-exports java.base/sun.security.x509=ALL-UNNAMED --add-opens java.base/sun.net.www.protocol.https=ALL-UNNAMED --add-opens java.base/java.net=ALL-UNNAMED</argLine>
                         </configuration>
                     </plugin>
                 </plugins>
@@ -177,7 +177,7 @@
                 <version>3.0.0-M5</version>
                 <configuration>
                     <runOrder>random</runOrder>
-                    <argLine>-Ddd.integration.junit.enabled=true -Ddd.prioritization.type=ENSURE_TRACE -Ddd.jmxfetch.enabled=false -javaagent:${basedir}/target/dd-java-agent-${dd-java-agent-version}.jar -Xms512m -Xmx3500m -DsurefireForkNumber=${surefire.forkNumber}</argLine>
+                    <argLine>-Xms512m -Xmx3500m -DsurefireForkNumber=${surefire.forkNumber}</argLine>
                 </configuration>
             </plugin>
             <plugin>
@@ -193,23 +193,6 @@
                             <outputDirectory>${project.build.directory}/lib</outputDirectory>
                         </configuration>
                     </execution>
-                    <execution>
-                        <id>copy</id>
-                        <phase>process-resources</phase>
-                        <goals>
-                            <goal>copy</goal>
-                        </goals>
-                        <configuration>
-                            <artifactItems>
-                            <artifactItem>
-                                <groupId>com.datadoghq</groupId>
-                                <artifactId>dd-java-agent</artifactId>
-                                <version>${dd-java-agent-version}</version>
-                                <outputDirectory>${project.build.directory}</outputDirectory>
-                            </artifactItem>
-                            </artifactItems>
-                        </configuration>
-                    </execution>
                 </executions>
             </plugin>
 
@@ -635,7 +618,7 @@
         <jakarta-annotation-version>2.1.0</jakarta-annotation-version>
         <scribejava-version>8.3.1</scribejava-version>
         <junit-version>4.13.2</junit-version>
-        <dd-java-agent-version>1.41.1</dd-java-agent-version>
+        <dd-java-agent-version>1.45.2</dd-java-agent-version>
         <cucumber-version>7.2.3</cucumber-version>
         <project.scm.id>github</project.scm.id>
     </properties>

src/test/java/com/datadog/api/ScenariosTest.java

@@ -8,7 +8,7 @@
 
 @RunWith(Cucumber.class)
 @CucumberOptions(
-    plugin = {"pretty", "summary", "com.datadog.api.TracePlugin", "rerun:target/rerun.txt"},
+    plugin = {"pretty", "summary", "rerun:target/rerun.txt"},
     snippets = CAMELCASE,
     tags = "not @skip and not @skip-java")
 public class ScenariosTest {}