Pull test data from upstream instead of maintaining a local copy

.github/workflows/test.yaml

@@ -14,6 +14,8 @@ jobs:
           go-version: ${{ matrix.go-version }}
       - name: Checkout code
         uses: actions/checkout@v2
+      - name: Download test data
+        run: curl -L https://raw.githubusercontent.com/package-url/purl-spec/master/test-suite-data.json -o testdata/test-suite-data.json
       - name: Test go fmt
         run: test -z $(go fmt ./...)
       - name: Golangci-lint

.gitignore

@@ -12,3 +12,5 @@
 
 # Project-local glide cache, RE: https://github.com/Masterminds/glide/issues/736
 .glide/
+
+testdata/test-suite-data.json

Makefile

@@ -1,6 +1,7 @@
 .PHONY: test clean lint
 
 test:
+	curl -Ls https://raw.githubusercontent.com/package-url/purl-spec/master/test-suite-data.json -o testdata/test-suite-data.json
 	go test -v -cover ./...
 
 lint:

README.md

@@ -58,7 +58,7 @@ func main() {
 Testing using the normal ``go test`` command. Using ``make test`` will pull the test fixtures shared between all package-url projects and then execute the tests.
 
 ```
-$ make test
+curl -Ls https://raw.githubusercontent.com/package-url/purl-spec/master/test-suite-data.json -o testdata/test-suite-data.json
 go test -v -cover ./...
 === RUN   TestFromStringExamples
 --- PASS: TestFromStringExamples (0.00s)
@@ -69,6 +69,6 @@ go test -v -cover ./...
 === RUN   TestQualifiersMapConversion
 --- PASS: TestQualifiersMapConversion (0.00s)
 PASS
-coverage: 90.7% of statements
+        github.com/package-url/packageurl-go    coverage: 90.7% of statements
 ok      github.com/package-url/packageurl-go    0.004s  coverage: 90.7% of statements
 ```

packageurl_test.go

@@ -60,7 +60,7 @@ var qualifiersMapPattern = regexp.MustCompile(`^\{.*\}$`)
 // UnmarshalJSON unmarshals the qualifiers field for a TestFixture. The
 // qualifiers field is given as a json object such as:
 //
-//        "qualifiers": {"arch": "i386", "distro": "fedora-25"}
+//	"qualifiers": {"arch": "i386", "distro": "fedora-25"}
 //
 // This function performs in-order parsing of these values into an OrderedMap to
 // preserve items in order of declaration. Note that parsing as a
@@ -199,8 +199,9 @@ func TestToStringExamples(t *testing.T) {
 			continue
 		}
 		instance := packageurl.NewPackageURL(
-			tc.PackageType, tc.Namespace, tc.Name,
-			tc.Version, tc.Qualifiers(), tc.Subpath)
+			tc.PackageType, tc.Namespace, tc.Name, tc.Version,
+			// Use QualifiersFromMap so that the qualifiers have a defined order, which is needed for string comparisons
+			packageurl.QualifiersFromMap(tc.Qualifiers().Map()), tc.Subpath)
 		result := instance.ToString()
 
 		// NOTE: We create a purl with ToString and then load into a PackageURL