workaround for #459

testng-team · Sep 15, 2019 · d44725f · d44725f
1 parent 04edccc
commit d44725f
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 0 deletions.
diff --git a/CHANGES.md b/CHANGES.md
@@ -1,6 +1,18 @@
 Change Log
 ====
 
+
+## DEV
+
+Supported Metrics:
+
+| Plugin | Dependency |
+| ------------- | ------------- |
+| TestNG for Eclipse | Eclipse Photon (4.8) or above |
+| TestNG M2E Integration (Optional) | M2E 1.5 or above |
+
+* workaround #459, replace `${argLine}` with empty string
+
 ## 7.0.0
 
 Supported Metrics:

diff --git a/...n-eclipse-plugin/src/org/testng/eclipse/maven/MavenTestNGLaunchConfigurationProvider.java b/...n-eclipse-plugin/src/org/testng/eclipse/maven/MavenTestNGLaunchConfigurationProvider.java
@@ -360,6 +360,11 @@ private Properties collectProperties(Model model, List<Profile> activeProfiles,
     result.put("settings.localRepository", MavenPlugin.getMaven().getLocalRepositoryPath());
     result.put("basedir", project.getProject().getLocation().toOSString());
 
+    // issue #459: quick and dirty workaround for jacoco generated placeholder '${argLine}'
+    // FIXME jacoco configuration support custom property name: https://www.eclemma.org/jacoco/trunk/doc/prepare-agent-mojo.html#propertyName
+    //          should get the actual jacoco propertyName, rather than hard-code below
+    result.put("argLine", "");
+
     //
     // project base properties
     //