fix: ensure packages are excluded correctly and add tests (#414)

Author: ckipp01

Diff for: plugin/src/main/scala/scoverage/ScoverageOptions.scala

@@ -92,7 +92,8 @@ object ScoverageOptions {
 
     scalacOptions.foreach {
       case ExcludedPackages(packages) =>
-        options = options.copy(excludedFiles = parseExclusionOption(packages))
+        options =
+          options.copy(excludedPackages = parseExclusionOption(packages))
       case ExcludedFiles(files) =>
         options = options.copy(excludedFiles = parseExclusionOption(files))
       case ExcludedSymbols(symbols) =>

Diff for: plugin/src/test/scala/scoverage/ScoverageOptionsTest.scala

@@ -0,0 +1,36 @@
+package scoverage
+
+import munit.FunSuite
+
+class ScoverageOptionsTest extends FunSuite {
+
+  val initalOptions = ScoverageOptions.default()
+  val fakeOptions = List(
+    "dataDir:myFakeDir",
+    "sourceRoot:myFakeSourceRoot",
+    "excludedPackages:some.package;another.package*",
+    "excludedFiles:*.proto;iHateThisFile.scala",
+    "excludedSymbols:someSymbol;anotherSymbol;aThirdSymbol",
+    "extraAfterPhase:extarAfter",
+    "extraBeforePhase:extraBefore",
+    "reportTestName"
+  )
+
+  val parsed = ScoverageOptions.parse(fakeOptions, (_) => (), initalOptions)
+
+  test("should be able to parse all options") {
+    assertEquals(
+      parsed.excludedPackages,
+      Seq("some.package", "another.package*")
+    )
+    assertEquals(parsed.excludedFiles, Seq("*.proto", "iHateThisFile.scala"))
+    assertEquals(
+      parsed.excludedSymbols,
+      Seq("someSymbol", "anotherSymbol", "aThirdSymbol")
+    )
+    assertEquals(parsed.dataDir, "myFakeDir")
+    assertEquals(parsed.reportTestName, true)
+    assertEquals(parsed.sourceRoot, "myFakeSourceRoot")
+  }
+
+}