Include default option values in JSON

This was removed implicitly when adding a default OS to the CLI.

Signed-off-by: Philippe Ombredanne <[email protected]>

Author: pombredanne

CHANGELOG.rst

@@ -25,6 +25,10 @@ v0.13.2
 
   - python_inspector.utils_pypi.Distribution.download_url is now a method, not a property
 
+  - The command line has again a default OS and Python version set.
+
+  - Default option values are reported in the JSON results. They were skipped before.
+
 - Drop support for running on Python 3.8. You can still resolve dependencies for Python 3.8.
   The default command line tool Python version used for resolution is now 3.9.
 

src/python_inspector/resolve_cli.py

@@ -321,9 +321,6 @@ def get_pretty_options(ctx, generic_paths=False):
         if getattr(param, "hidden", False):
             continue
 
-        if value == param.default:
-            continue
-
         if value in (None, False):
             continue
 

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

@@ -4,7 +4,9 @@
     "tool_homepageurl": "https://github.com/aboutcode-org/python-inspector",
     "tool_version": "0.13.0",
     "options": [
+      "--index-url https://pypi.org/simple",
       "--json <file>",
+      "--operating-system linux",
       "--python-version 310",
       "--requirement tests/data/azure-devops.req.txt"
     ],

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

@@ -4,7 +4,9 @@
     "tool_homepageurl": "https://github.com/aboutcode-org/python-inspector",
     "tool_version": "0.13.0",
     "options": [
+      "--index-url https://pypi.org/simple",
       "--json <file>",
+      "--operating-system linux",
       "--python-version 312",
       "--requirement tests/data/azure-devops.req.txt"
     ],

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

@@ -4,7 +4,9 @@
     "tool_homepageurl": "https://github.com/aboutcode-org/python-inspector",
     "tool_version": "0.13.0",
     "options": [
+      "--index-url https://pypi.org/simple",
       "--json <file>",
+      "--operating-system linux",
       "--python-version 313",
       "--requirement tests/data/azure-devops.req.txt"
     ],

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

@@ -4,7 +4,9 @@
     "tool_homepageurl": "https://github.com/aboutcode-org/python-inspector",
     "tool_version": "0.13.0",
     "options": [
+      "--index-url https://pypi.org/simple",
       "--json <file>",
+      "--operating-system linux",
       "--python-version 38",
       "--requirement tests/data/azure-devops.req.txt"
     ],

tests/data/default-url-expected.json

@@ -4,7 +4,9 @@
     "tool_homepageurl": "https://github.com/aboutcode-org/python-inspector",
     "tool_version": "0.13.0",
     "options": [
+      "--index-url https://pypi.org/simple",
       "--json <file>",
+      "--operating-system linux",
       "--python-version 38",
       "--specifier zipp==3.8.0",
       "--use-pypi-json-api"

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

@@ -4,7 +4,9 @@
     "tool_homepageurl": "https://github.com/aboutcode-org/python-inspector",
     "tool_version": "0.13.0",
     "options": [
+      "--index-url https://pypi.org/simple",
       "--json-pdt <file>",
+      "--operating-system linux",
       "--python-version 37",
       "--requirement tests/data/environment-marker-test-requirements.txt"
     ],

tests/data/example-requirements-ignore-errors-expected.json

@@ -5,7 +5,9 @@
     "tool_version": "0.13.0",
     "options": [
       "--ignore-errors",
+      "--index-url https://pypi.org/simple",
       "--json <file>",
+      "--operating-system linux",
       "--python-version 310",
       "--requirement tests/data/error-requirements.txt"
     ],

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

@@ -4,7 +4,9 @@
     "tool_homepageurl": "https://github.com/aboutcode-org/python-inspector",
     "tool_version": "0.13.0",
     "options": [
+      "--index-url https://pypi.org/simple",
       "--json-pdt <file>",
+      "--operating-system linux",
       "--python-version 38",
       "--requirement tests/data/frozen-requirements.txt"
     ],