[hawkmoth] Re: [PATCH 3/4] test: rename file variables to f

  • From: Jani Nikula <jani@xxxxxxxxxx>
  • To: Bruno Santos <brunomanuelsantos@xxxxxxxxxxxxxxxxxx>, hawkmoth mailing list <hawkmoth@xxxxxxxxxxxxx>
  • Date: Sun, 28 Apr 2019 21:42:25 +0300

On Sun, 28 Apr 2019, Bruno Santos <brunomanuelsantos@xxxxxxxxxxxxxxxxxx> wrote:

'file' is a core name which was being overridden. This was not of any
consequence, but the rename leads to the more common pattern:

      with open(filename) as f:

My bad, thanks for fixing. Pushed.

BR,
Jani.

---
 test/test_cautodoc.py | 6 +++---
 test/testenv.py       | 8 ++++----
 2 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/test/test_cautodoc.py b/test/test_cautodoc.py
index a1aff01..4d7c746 100755
--- a/test/test_cautodoc.py
+++ b/test/test_cautodoc.py
@@ -14,12 +14,12 @@ def _get_output(input_filename, app, status, warning, 
**options):
     shutil.copyfile(input_filename,
                     testenv.modify_filename(input_filename, dir=app.srcdir))
 
-    with open(os.path.join(app.srcdir, 'index.rst'), 'w') as file:
+    with open(os.path.join(app.srcdir, 'index.rst'), 'w') as f:
         fmt = '.. c:autodoc:: {source}\n'
-        file.write(fmt.format(source=os.path.basename(input_filename)))
+        f.write(fmt.format(source=os.path.basename(input_filename)))
         for key in [k for k in options.keys() if k in 
testenv.directive_options]:
             fmt = '   :{key}: {value}\n'
-            file.write(fmt.format(key=key, value=options[key]))
+            f.write(fmt.format(key=key, value=options[key]))
 
     app.build()
 
diff --git a/test/testenv.py b/test/testenv.py
index b6842a8..c86749a 100644
--- a/test/testenv.py
+++ b/test/testenv.py
@@ -37,8 +37,8 @@ def get_testcase_options(testcase):
     # options are optional
     options = {}
     if os.path.isfile(options_filename):
-        with open(options_filename, 'r') as file:
-            for line in file.readlines():
+        with open(options_filename, 'r') as f:
+            for line in f.readlines():
                 opt = [x.strip() for x in line.split('=', 1)]
                 if opt[0] != '':
                     if len(opt) == 2:
@@ -64,8 +64,8 @@ def modify_filename(filename, **kwargs):
 def read_file(filename, **kwargs):
     filename = modify_filename(filename, **kwargs)
 
-    with open(filename, 'r') as file:
-        expected = file.read()
+    with open(filename, 'r') as f:
+        expected = f.read()
 
     return expected
 
-- 
2.21.0

Other related posts: