Address review comments and add CHAGELOG

Signed-off-by: Tushar Goel <[email protected]>

Author: TG1999

CHANGELOG.rst

@@ -6,6 +6,7 @@ v0.9.2
 ------
 
 - Make os and python version as mandatory input parameters.
+- Do not return duplicates binaries.
 
 
 v0.9.1

src/python_inspector/package_data.py

@@ -75,9 +75,9 @@ def get_pypi_data_from_purl(
                 python_version=python_version,
             )
         )
-        wheel_urls.sort(reverse=True)
-        if len(wheel_urls) > 0:
-            valid_distribution_urls.append(wheel_urls[0])
+        wheel_url = choose_single_wheel(wheel_urls)
+        if wheel_url:
+            valid_distribution_urls.append(wheel_url)
 
     urls = response.get("urls") or []
     for url in urls:
@@ -111,6 +111,15 @@ def get_pypi_data_from_purl(
         )
 
 
+def choose_single_wheel(wheel_urls):
+    """
+    Sort wheel urls descendingly and return the first one
+    """
+    wheel_urls.sort(reverse=True)
+    if wheel_urls:
+        return wheel_urls[0]
+
+
 def get_pypi_bugtracker_url(project_urls):
     bug_tracking_url = project_urls.get("Tracker")
     if not (bug_tracking_url):

src/python_inspector/resolve_cli.py

@@ -20,7 +20,7 @@
 
 TRACE = False
 
-__version__ = "0.9.1"
+__version__ = "0.9.2"
 
 DEFAULT_PYTHON_VERSION = "38"
 PYPI_SIMPLE_URL = "https://pypi.org/simple"

tests/data/azure-devops.req-310-expected.json

@@ -2,7 +2,7 @@
   "headers": {
     "tool_name": "python-inspector",
     "tool_homepageurl": "https://github.com/nexB/python-inspector",
-    "tool_version": "0.9.1",
+    "tool_version": "0.9.2",
     "options": [
       "--requirement /home/tg1999/Desktop/python-inspector-1/tests/data/azure-devops.req.txt",
       "--index-url https://pypi.org/simple",

tests/data/azure-devops.req-38-expected.json

@@ -2,7 +2,7 @@
   "headers": {
     "tool_name": "python-inspector",
     "tool_homepageurl": "https://github.com/nexB/python-inspector",
-    "tool_version": "0.9.1",
+    "tool_version": "0.9.2",
     "options": [
       "--requirement /home/tg1999/Desktop/python-inspector-1/tests/data/azure-devops.req.txt",
       "--index-url https://pypi.org/simple",

tests/data/default-url-expected.json

@@ -2,7 +2,7 @@
   "headers": {
     "tool_name": "python-inspector",
     "tool_homepageurl": "https://github.com/nexB/python-inspector",
-    "tool_version": "0.9.1",
+    "tool_version": "0.9.2",
     "options": [
       "--specifier zipp==3.8.0",
       "--index-url https://pypi.org/simple",

tests/data/environment-marker-test-requirements.txt-expected.json

@@ -2,7 +2,7 @@
   "headers": {
     "tool_name": "python-inspector",
     "tool_homepageurl": "https://github.com/nexB/python-inspector",
-    "tool_version": "0.9.1",
+    "tool_version": "0.9.2",
     "options": [
       "--requirement /home/tg1999/Desktop/python-inspector-1/tests/data/environment-marker-test-requirements.txt",
       "--index-url https://pypi.org/simple",

tests/data/frozen-requirements.txt-expected.json

@@ -2,7 +2,7 @@
   "headers": {
     "tool_name": "python-inspector",
     "tool_homepageurl": "https://github.com/nexB/python-inspector",
-    "tool_version": "0.9.1",
+    "tool_version": "0.9.2",
     "options": [
       "--requirement /home/tg1999/Desktop/python-inspector-1/tests/data/frozen-requirements.txt",
       "--index-url https://pypi.org/simple",

tests/data/insecure-setup-2/setup.py-expected.json

@@ -2,7 +2,7 @@
   "headers": {
     "tool_name": "python-inspector",
     "tool_homepageurl": "https://github.com/nexB/python-inspector",
-    "tool_version": "0.9.1",
+    "tool_version": "0.9.2",
     "options": [
       "--index-url https://pypi.org/simple",
       "--python-version 27",

tests/data/insecure-setup/setup.py-expected.json

@@ -2,7 +2,7 @@
   "headers": {
     "tool_name": "python-inspector",
     "tool_homepageurl": "https://github.com/nexB/python-inspector",
-    "tool_version": "0.9.1",
+    "tool_version": "0.9.2",
     "options": [
       "--index-url https://pypi.org/simple",
       "--python-version 27",