Merge pull request #325 from elautz/coverage-null-path

Added a null check for the htmlDir in PyCoverageTask
linkedin · Dec 5, 2019 · 90e3f5f · 90e3f5f
2 parents 26c6ba2 + cb03227
commit 90e3f5f
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/pygradle-plugin/src/main/groovy/com/linkedin/gradle/python/tasks/PyCoverageTask.groovy b/pygradle-plugin/src/main/groovy/com/linkedin/gradle/python/tasks/PyCoverageTask.groovy
@@ -73,8 +73,10 @@ class PyCoverageTask extends PyTestTask {
         String htmlDir = streamProcessor.htmlDir
         String coverage = streamProcessor.coverage
 
-
-        FileUtils.copyDirectoryToDirectory(project.file(htmlDir), coverageOutputDir)
+        // If there is no coverage to report, then the htmlDir value will be empty
+        if (htmlDir != null) {
+            FileUtils.copyDirectoryToDirectory(project.file(htmlDir), coverageOutputDir)
+        }
 
         CoverageXmlReporter coverageXmlReport = new CoverageXmlReporter(coverage)
         coverageReport.text = coverageXmlReport.generateXML()