fix: address comments

Author: Bucknalla

test/fluent_api/conftest.py

@@ -34,9 +34,10 @@ def _run_test(fluent_api, notecard_api_name, req_params, rename_key_map=None, re
         # that parameter is actually derived from the value of the 'format'
         # parameter. The rename_value_map allows a test to specify how a fluent
         # API's keyword args map to Notecard API args, in cases where the value
+        # of one arg is derived from the value of another arg.
         if rename_value_map is not None:
-            for old_key, new_value in rename_value_map.items():
-                expected_notecard_api_req[old_key] = new_value
+            for key, new_value in rename_value_map.items():
+                expected_notecard_api_req[key] = new_value
 
         card.Transaction.assert_called_once_with(expected_notecard_api_req)
 

test/test_md5.py

@@ -19,7 +19,7 @@ def tearDown(self):
 
     def test_non_cpython_implementation(self):
         """Test our custom MD5 implementation used in non-CPython environments"""
-        # Set implementation to micropython before importing
+        # Set implementation to non-cpython before importing
         sys.implementation.name = 'non-cpython'
         import notecard.md5