fix: improve use of InputStream in PackageURLTest (#171)

* Fix memory leak with `InputStream`
* `JSONTokener` is able to load the `InputStream` directly, removing
  the test dependency on Commons IO

Author: dwalluck

pom.xml

@@ -123,12 +123,6 @@
             <version>20240303</version>
             <scope>test</scope>
         </dependency>
-        <dependency>
-            <groupId>commons-io</groupId>
-            <artifactId>commons-io</artifactId>
-            <version>2.15.1</version>
-            <scope>test</scope>
-        </dependency>
     </dependencies>
 
     <build>

src/test/java/com/github/packageurl/PackageURLTest.java

@@ -23,16 +23,15 @@
 
 import java.io.IOException;
 import java.io.InputStream;
-import java.nio.charset.StandardCharsets;
 import java.util.HashMap;
 import java.util.Map;
 import java.util.Locale;
 import java.util.TreeMap;
 
-import org.apache.commons.io.IOUtils;
 import org.json.JSONArray;
 import org.json.JSONObject;
 import org.junit.AfterClass;
+import org.json.JSONTokener;
 import org.junit.Assert;
 import org.junit.BeforeClass;
 import org.junit.Rule;
@@ -57,10 +56,11 @@ public class PackageURLTest {
 
     @BeforeClass
     public static void setup() throws IOException {
-        InputStream is = PackageURLTest.class.getResourceAsStream("/test-suite-data.json");
-        Assert.assertNotNull(is);
-        String jsonTxt = IOUtils.toString(is, StandardCharsets.UTF_8);
-        json = new JSONArray(jsonTxt);
+        try (InputStream is = PackageURLTest.class.getResourceAsStream("/test-suite-data.json")) {
+            Assert.assertNotNull(is);
+            json = new JSONArray(new JSONTokener(is));
+        }
+
         defaultLocale = Locale.getDefault();
         Locale.setDefault(new Locale("tr"));
     }