Use keyword arguments with the open() method

Author: mdmintz

mkdocs_build/prepare.py

@@ -117,7 +117,7 @@ def main(*args, **kwargs):
 
     for file_ in updated_files_to_process:
         readme_file = "./mkdocs_build/" + file_
-        with open(readme_file, "r", encoding="utf-8") as f:
+        with open(readme_file, mode="r", encoding="utf-8") as f:
             all_code = f.read()
         code_lines = all_code.split("\n")
 
@@ -195,6 +195,6 @@ def main(*args, **kwargs):
                 )
             seleniumbase_lines.append(line)
         if changed:
-            out_file = open(readme_file, "w+", encoding="utf-8")
+            out_file = open(readme_file, mode="w+", encoding="utf-8")
             out_file.writelines("\r\n".join(seleniumbase_lines))
             out_file.close()

seleniumbase/behave/behave_sb.py

@@ -1216,24 +1216,24 @@ def _create_dashboard_assets_():
     add_pytest_style_css = True
     if os.path.exists(pytest_style_css):
         existing_pytest_style = None
-        with open(pytest_style_css, "r") as f:
+        with open(pytest_style_css, mode="r") as f:
             existing_pytest_style = f.read()
         if existing_pytest_style == get_pytest_style():
             add_pytest_style_css = False
     if add_pytest_style_css:
-        out_file = open(pytest_style_css, "w+", encoding="utf-8")
+        out_file = open(pytest_style_css, mode="w+", encoding="utf-8")
         out_file.writelines(get_pytest_style())
         out_file.close()
     live_js_file = os.path.join(assets_folder, "live.js")
     add_live_js_file = True
     if os.path.exists(live_js_file):
         existing_live_js = None
-        with open(live_js_file, "r") as f:
+        with open(live_js_file, mode="r") as f:
             existing_live_js = f.read()
         if existing_live_js == live_js:
             add_live_js_file = False
     if add_live_js_file:
-        out_file = open(live_js_file, "w+", encoding="utf-8")
+        out_file = open(live_js_file, mode="w+", encoding="utf-8")
         out_file.writelines(live_js)
         out_file.close()
 
@@ -1306,7 +1306,7 @@ def _perform_behave_unconfigure_():
         # Part 1: Finalizing the dashboard / integrating html report
         if os.path.exists(dashboard_path):
             the_html_d = None
-            with open(dashboard_path, "r", encoding="utf-8") as f:
+            with open(dashboard_path, mode="r", encoding="utf-8") as f:
                 the_html_d = f.read()
             if sb_config._multithreaded and "-c" in sys.argv:
                 # Threads have "-c" in sys.argv, except for the last
@@ -1317,7 +1317,7 @@ def _perform_behave_unconfigure_():
                 if os.path.exists(pie_path):
                     import json
 
-                    with open(pie_path, "r") as f:
+                    with open(pie_path, mode="r") as f:
                         dash_pie = f.read().strip()
                     sb_config._saved_dashboard_pie = json.loads(dash_pie)
             # If the test run doesn't complete by itself, stop refresh
@@ -1326,7 +1326,7 @@ def _perform_behave_unconfigure_():
             the_html_d = the_html_d.replace(find_it_3, swap_with_3)
             the_html_d = the_html_d.replace(find_it_4, swap_with_4)
             the_html_d += stamp
-            with open(dashboard_path, "w", encoding="utf-8") as f:
+            with open(dashboard_path, mode="w", encoding="utf-8") as f:
                 f.write(the_html_d)  # Finalize the dashboard
     except KeyboardInterrupt:
         pass

seleniumbase/console_scripts/sb_caseplans.py

@@ -134,7 +134,7 @@ def generate_case_plan_boilerplates(
         file_name = case_id
         file_path = os.path.join(full_folder_path, file_name)
         if not os.path.exists(file_path):
-            out_file = open(file_path, "w+", "utf-8")
+            out_file = open(file_path, mode="w+", encoding="utf-8")
             out_file.writelines("\r\n".join(data))
             out_file.close()
             new_plans += 1
@@ -182,7 +182,7 @@ def view_summary_of_existing_case_plans(root, tests):
         else:
             case_path = os.path.join(folder_path, "case_plans", case_id)
         if os.path.exists(case_path):
-            f = open(case_path, "r")
+            f = open(case_path, mode="r")
             case_data = f.read()
             f.close()
             case_data_storage.append(case_data)
@@ -315,7 +315,7 @@ def view_summary_of_existing_case_plans(root, tests):
     full_plan = plan_head
 
     file_path = "case_summary.md"
-    file = open(file_path, "w+", "utf-8")
+    file = open(file_path, mode="w+", encoding="utf-8")
     file.writelines("\r\n".join(full_plan))
     file.close()
 

seleniumbase/console_scripts/sb_mkchart.py

@@ -253,7 +253,7 @@ def main():
                 continue
             new_data.append(line)
     data = new_data
-    file = open(file_path, "w+", "utf-8")
+    file = open(file_path, mode="w+", encoding="utf-8")
     file.writelines("\r\n".join(data))
     file.close()
     if " " not in file_name:

seleniumbase/console_scripts/sb_mkdir.py

@@ -113,7 +113,7 @@ def main():
     data.append(seleniumbase_req)
     data.append("")
     file_path = "%s/%s" % (dir_name, "requirements.txt")
-    file = open(file_path, "w+", "utf-8")
+    file = open(file_path, mode="w+", encoding="utf-8")
     file.writelines("\r\n".join(data))
     file.close()
 
@@ -151,7 +151,7 @@ def main():
     data.append("    production: custom marker")
     data.append("")
     file_path = "%s/%s" % (dir_name, "pytest.ini")
-    file = open(file_path, "w+", "utf-8")
+    file = open(file_path, mode="w+", encoding="utf-8")
     file.writelines("\r\n".join(data))
     file.close()
 
@@ -168,14 +168,14 @@ def main():
     data.append("show_skipped=false")
     data.append("show_timings=false")
     file_path = "%s/%s" % (dir_name, "setup.cfg")
-    file = open(file_path, "w+", "utf-8")
+    file = open(file_path, mode="w+", encoding="utf-8")
     file.writelines("\r\n".join(data))
     file.close()
 
     data = []
     data.append("")
     file_path = "%s/%s" % (dir_name, "__init__.py")
-    file = open(file_path, "w+", "utf-8")
+    file = open(file_path, mode="w+", encoding="utf-8")
     file.writelines("\r\n".join(data))
     file.close()
 
@@ -311,7 +311,7 @@ def main():
     data.append("temp_*/")
     data.append("node_modules")
     file_path = "%s/%s" % (dir_name, ".gitignore")
-    file = open(file_path, "w+", "utf-8")
+    file = open(file_path, mode="w+", encoding="utf-8")
     file.writelines("\r\n".join(data))
     file.close()
 
@@ -323,7 +323,7 @@ def main():
         data.append("  ├── requirements.txt")
         data.append("  └── setup.cfg")
         file_path = "%s/%s" % (dir_name, "outline.rst")
-        file = open(file_path, "w+", "utf-8")
+        file = open(file_path, mode="w+", encoding="utf-8")
         file.writelines("\r\n".join(data))
         file.close()
         os.system("sbase print %s -n" % file_path)
@@ -367,7 +367,7 @@ def main():
     data.append('        self.assert_element("div#login_button_container")')
     data.append("")
     file_path = "%s/%s" % (dir_name, "my_first_test.py")
-    file = open(file_path, "w+", "utf-8")
+    file = open(file_path, mode="w+", encoding="utf-8")
     file.writelines("\r\n".join(data))
     file.close()
 
@@ -460,7 +460,7 @@ def main():
     data.append('        self.assert_text("SeleniumBase", "h2")')
     data.append("")
     file_path = "%s/%s" % (dir_name, "test_demo_site.py")
-    file = open(file_path, "w+", "utf-8")
+    file = open(file_path, mode="w+", encoding="utf-8")
     file.writelines("\r\n".join(data))
     file.close()
 
@@ -499,7 +499,7 @@ def main():
     data.append('        self.assert_title_contains(title_text)')
     data.append("")
     file_path = "%s/%s" % (dir_name, "parameterized_test.py")
-    file = open(file_path, "w+", "utf-8")
+    file = open(file_path, mode="w+", encoding="utf-8")
     file.writelines("\r\n".join(data))
     file.close()
 
@@ -509,7 +509,7 @@ def main():
     data = []
     data.append("")
     file_path = "%s/%s" % (dir_name_2, "__init__.py")
-    file = open(file_path, "w+", "utf-8")
+    file = open(file_path, mode="w+", encoding="utf-8")
     file.writelines("\r\n".join(data))
     file.close()
 
@@ -544,7 +544,7 @@ def main():
     data.append("        pass")
     data.append("")
     file_path = "%s/%s" % (dir_name_2, "base_test_case.py")
-    file = open(file_path, "w+", "utf-8")
+    file = open(file_path, mode="w+", encoding="utf-8")
     file.writelines("\r\n".join(data))
     file.close()
 
@@ -553,7 +553,7 @@ def main():
     data.append('    html = "html"')
     data.append("")
     file_path = "%s/%s" % (dir_name_2, "page_objects.py")
-    file = open(file_path, "w+", "utf-8")
+    file = open(file_path, mode="w+", encoding="utf-8")
     file.writelines("\r\n".join(data))
     file.close()
 
@@ -569,7 +569,7 @@ def main():
     data.append("        self.assert_element(Page.html)")
     data.append("")
     file_path = "%s/%s" % (dir_name_2, "boilerplate_test.py")
-    file = open(file_path, "w+", "utf-8")
+    file = open(file_path, mode="w+", encoding="utf-8")
     file.writelines("\r\n".join(data))
     file.close()
 
@@ -593,7 +593,7 @@ def main():
     data.append('        DataPage().add_input_text(self, "Goodbye!")')
     data.append("")
     file_path = "%s/%s" % (dir_name_2, "classic_obj_test.py")
-    file = open(file_path, "w+", "utf-8")
+    file = open(file_path, mode="w+", encoding="utf-8")
     file.writelines("\r\n".join(data))
     file.close()
 
@@ -613,7 +613,7 @@ def main():
     data.append('        DataPage().add_input_text(sb, "Goodbye!")')
     data.append("")
     file_path = "%s/%s" % (dir_name_2, "sb_fixture_test.py")
-    file = open(file_path, "w+", "utf-8")
+    file = open(file_path, mode="w+", encoding="utf-8")
     file.writelines("\r\n".join(data))
     file.close()
 
@@ -623,7 +623,7 @@ def main():
     data = []
     data.append("")
     file_path = "%s/%s" % (dir_name_3, "__init__.py")
-    file = open(file_path, "w+", "utf-8")
+    file = open(file_path, mode="w+", encoding="utf-8")
     file.writelines("\r\n".join(data))
     file.close()
 
@@ -651,7 +651,7 @@ def main():
     )
     data.append("")
     file_path = "%s/%s" % (dir_name_3, "google_test.py")
-    file = open(file_path, "w+", "utf-8")
+    file = open(file_path, mode="w+", encoding="utf-8")
     file.writelines("\r\n".join(data))
     file.close()
 
@@ -669,7 +669,7 @@ def main():
     data.append('    search_results = "div#center_col"')
     data.append("")
     file_path = "%s/%s" % (dir_name_3, "google_objects.py")
-    file = open(file_path, "w+", "utf-8")
+    file = open(file_path, mode="w+", encoding="utf-8")
     file.writelines("\r\n".join(data))
     file.close()
 
@@ -701,7 +701,7 @@ def main():
     data.append('        self.assert_element("div#login_button_container")')
     data.append("")
     file_path = "%s/%s" % (dir_name_3, "swag_labs_test.py")
-    file = open(file_path, "w+", "utf-8")
+    file = open(file_path, mode="w+", encoding="utf-8")
     file.writelines("\r\n".join(data))
     file.close()
 
@@ -728,7 +728,7 @@ def main():
     data.append('        sb.assert_element("div#login_button_container")')
     data.append("")
     file_path = "%s/%s" % (dir_name_3, "sb_swag_test.py")
-    file = open(file_path, "w+", "utf-8")
+    file = open(file_path, mode="w+", encoding="utf-8")
     file.writelines("\r\n".join(data))
     file.close()
 
@@ -755,7 +755,7 @@ def main():
     data.append("          ├── sb_swag_test.py")
     data.append("          └── swag_labs_test.py")
     file_path = "%s/%s" % (dir_name, "outline.rst")
-    file = open(file_path, "w+", "utf-8")
+    file = open(file_path, mode="w+", encoding="utf-8")
     file.writelines("\r\n".join(data))
     file.close()
     if " " not in file_path:

seleniumbase/console_scripts/sb_mkfile.py

@@ -412,7 +412,7 @@ def main():
                 continue
             new_data.append(line)
     data = new_data
-    file = open(file_path, "w+", "utf-8")
+    file = open(file_path, mode="w+", encoding="utf-8")
     file.writelines("\r\n".join(data))
     file.close()
     if " " not in file_name:

seleniumbase/console_scripts/sb_mkpres.py

@@ -272,7 +272,7 @@ def main():
                 continue
             new_data.append(line)
     data = new_data
-    file = open(file_path, "w+", "utf-8")
+    file = open(file_path, mode="w+", encoding="utf-8")
     file.writelines("\r\n".join(data))
     file.close()
     if " " not in file_name:

seleniumbase/console_scripts/sb_mkrec.py

@@ -239,7 +239,7 @@ def main():
         d2.append("")
         data = d2
 
-    file = open(file_path, "w+", "utf-8")
+    file = open(file_path, mode="w+", encoding="utf-8")
     file.writelines("\r\n".join(data))
     file.close()
     success = (

seleniumbase/console_scripts/sb_objectify.py

@@ -135,7 +135,7 @@ def create_objects_file(selector_list_dict=None):
         data.append('    html = "html"')
         data.append("")
     file_path = PAGE_OBJECTS_FILE
-    file = open(file_path, "w+", "utf-8")
+    file = open(file_path, mode="w+", encoding="utf-8")
     file.writelines("\r\n".join(data))
     file.close()
     if not selector_list_dict:
@@ -149,7 +149,7 @@ def scan_objects_file():
         create_objects_file()
 
     page_selectors = {}
-    with open(PAGE_OBJECTS_FILE, "r", encoding="utf-8") as f:
+    with open(PAGE_OBJECTS_FILE, mode="r", encoding="utf-8") as f:
         all_code = f.read()
 
     var_names = []
@@ -3089,7 +3089,7 @@ def main(shell_command):
             "Expecting: %s\n" % (seleniumbase_file, expected_arg)
         )
 
-    with open(seleniumbase_file, "r", encoding="utf-8") as f:
+    with open(seleniumbase_file, mode="r", encoding="utf-8") as f:
         all_code = f.read()
     if "def test_" not in all_code:
         raise Exception(
@@ -3188,7 +3188,7 @@ def main(shell_command):
     # Create SeleniumBase test file
     base_file_name = seleniumbase_file.split(".py")[0]
     converted_file_name = base_file_name + ".py"  # Change end to make a copy
-    out_file = open(converted_file_name, "w+", encoding="utf-8")
+    out_file = open(converted_file_name, mode="w+", encoding="utf-8")
     out_file.writelines(seleniumbase_code)
     out_file.close()
     print('\n>>> ["%s"] was updated!\n' % converted_file_name)

seleniumbase/console_scripts/sb_print.py

@@ -123,7 +123,7 @@ def main():
 
     all_code = None
     with open(
-        file_to_print, "r+", encoding="utf-8", errors="ignore"
+        file_to_print, mode="r+", encoding="utf-8", errors="ignore"
     ) as f:
         all_code = f.read()
     all_code = all_code.replace("\t", "    ")