Only create coverage reports in a CI environment. Adding forking base…

…d on the guide
AndrewQuijano · Apr 25, 2024 · 508ac18 · 508ac18
1 parent 7fa1b35
commit 508ac18
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 5 deletions.
diff --git a/.github/workflows/build-gradle-project.yml b/.github/workflows/build-gradle-project.yml
@@ -26,8 +26,8 @@ jobs:
         java-version: '17'
         cache: 'gradle'
 
-    - name: Run Gradle Testing
-      run: sh gradlew build
+    - name: Run Gradle Testing and Create Coverage Report
+      run: sh gradlew build -Pcoverage=true
 
     - name: Upload coverage reports to Codecov
       uses: codecov/codecov-action@v4

diff --git a/build.gradle b/build.gradle
@@ -47,11 +47,21 @@ gradle.projectsEvaluated {
     }
 }
 
+tasks.withType(JavaCompile).configureEach {
+    options.fork = true
+}
 
 jacocoTestReport {
     reports {
-        xml.required=true
-        html.required=true
+        // By default do NOT create reports
+        if (project.hasProperty("coverage")) {
+            xml.required=true
+            html.required=true
+        }
+        else {
+            xml.required=false
+            html.required=false
+        }
     }
 }
 

diff --git a/setup.sh b/setup.sh
@@ -45,7 +45,7 @@ if [[ $(id) != *\(docker\)* ]]; then
 	echo '[*] Configuring Docker...'
 	echo '##################################################'
 	sleep 3
-	sudo usermod -aG docker $USER
+	sudo usermod -aG docker "$USER"
 
 	echo '#####################################################################'
 	echo '[*] Almost there! Reboot and run this script one more time to finish.'