Apply suggestions from code review

Author: hugovk

Lib/platform.py

@@ -1472,16 +1472,20 @@ def _parse_args(args: list[str] | None):
     parser.add_argument(
         "--terse",
         action="store_true",
-        help=("return only the absolute minimum information needed to identify the "
-            "platform"),
+        help=(
+            "return only the absolute minimum information needed "
+            "to identify the platform"
+        ),
     )
     parser.add_argument(
         "--nonaliased",
         dest="aliased",
         action="store_false",
-        help=("disable system/OS name aliasing. If aliasing is enabled, "
-            "some platforms will report system names which differ from "
-            "their common names, e.g. SunOS will be reported as Solaris"),
+        help=(
+            "disable system/OS name aliasing. If aliasing is enabled, "
+            "some platforms report system names different from "
+            "their common names, e.g. SunOS is reported as Solaris"
+        ),
     )
 
     return parser.parse_args(args)
@@ -1498,4 +1502,3 @@ def _main(args: list[str] | None = None):
 
 if __name__ == "__main__":
     _main()
-    sys.exit(0)

Lib/test/test_platform.py

@@ -773,27 +773,26 @@ def test_invocation(self):
                 self.invoke_platform(*combination)
 
     def test_arg_parsing(self):
-        # Due to backwards compatibility, the `aliased` and `terse` parameters
-        # are computed based on a combination of positional arguments and flags.
+        # For backwards compatibility, the `aliased` and `terse` parameters are
+        # computed based on a combination of positional arguments and flags.
         #
-        # This test tests that the arguments are correctly passed to the underlying
-        # `platform.platform()` call. The parameters are two booleans for `aliased`
-        # and `terse`.
+        # Test that the arguments are correctly passed to the underlying
+        # `platform.platform()` call.
         options = (
-            (["--nonaliased"], (False, False)),
-            (["nonaliased"], (False, False)),
-            (["--terse"], (True, True)),
-            (["terse"], (True, True)),
-            (["nonaliased", "terse"], (False, True)),
-            (["--nonaliased", "terse"], (False, True)),
-            (["--terse", "nonaliased"], (False, True)),
+            (["--nonaliased"], False, False),
+            (["nonaliased"], False, False),
+            (["--terse"], True, True),
+            (["terse"], True, True),
+            (["nonaliased", "terse"], False, True),
+            (["--nonaliased", "terse"], False, True),
+            (["--terse", "nonaliased"], False, True),
         )
 
-        for flags, args in options:
-            with self.subTest(flags=flags, args=args):
+        for flags, aliased, terse in options:
+            with self.subTest(flags=flags, aliased=aliased, terse=terse):
                 with mock.patch.object(platform, 'platform') as obj:
                     self.invoke_platform(*flags)
-                    obj.assert_called_once_with(*args)
+                    obj.assert_called_once_with(aliased, terse)
 
     def test_help(self):
         output = io.StringIO()