fix: more Windows compatiblity issues fixed

 - skip the unit test that depends on making a directory non-writable, since Windows won't let us do it.
 - test_g2p_cli.py: write temp test files in utf8
 - readalongs g2p: open input file in utf8
 - make get_click_file_name() work on both Windows and Linux and in unit testing consistently
 - Failing to delete the failed tempfile is not a fatal error in align.py, just proceed!

readalongs/align.py

@@ -186,7 +186,10 @@ def align_audio(  # noqa: C901
                         save_temps + "_processed" + ext, format=ext[1:]
                     )
                 except CouldntEncodeError:
-                    os.remove(save_temps + "_processed" + ext)
+                    try:
+                        os.remove(save_temps + "_processed" + ext)
+                    except:
+                        pass
                     LOGGER.warning(
                         f"Couldn't find encoder for '{ext[1:]}', defaulting to 'wav'"
                     )

readalongs/cli.py

@@ -63,11 +63,23 @@ def create_app():
 
 
 def get_click_file_name(click_file):
-    """ Return click_file.name, falling back to <stdin> if the .name attribute is missing. """
+    """ Wrapper around click_file.name with consistent handling for stdin
+
+    On Windows, if click_file is stdin, click_file.name == "-".
+    On Linux, if click_file is stdin, click_file.name == "<stdin>".
+    During unit testing, the simulated stdin stream has no .name attribute
+
+    Args:
+        click_file(click.File): the click file whose name we need
+
+    Returns:
+        "-" if click_file represents stdin, click_file.name otherwise
+    """
     try:
-        return click_file.name
-    except Exception:  # For unit testing: simulated stdin stream has no .name attrib
-        return "<stdin>"
+        name = click_file.name
+    except Exception:
+        name = "-"
+    return "-" if name == "<stdin>" else name
 
 
 def parse_g2p_fallback(g2p_fallback_arg):
@@ -344,10 +356,7 @@ def prepare(**kwargs):
     out_file = kwargs["xmlfile"]
     if not out_file:
         out_file = get_click_file_name(input_file)
-        print(f"input_file={out_file}")
-        if out_file in ("<stdin>", "-"):  # intput_file.name is <stdin> on Linux, - on Windows, when cli input is "-"
-            out_file = "-"
-        else:
+        if out_file != "-":
             if out_file.endswith(".txt"):
                 out_file = out_file[:-4]
             out_file += ".xml"
@@ -408,9 +417,7 @@ def tokenize(**kwargs):
 
     if not kwargs["tokfile"]:
         output_path = get_click_file_name(input_file)
-        if output_path == "<stdin>":
-            output_path = "-"
-        else:
+        if output_path != "-":
             if output_path.endswith(".xml"):
                 output_path = output_path[:-4]
             output_path += ".tokenized.xml"
@@ -447,7 +454,7 @@ def tokenize(**kwargs):
     short_help="Apply g2p to a tokenized file, like 'align' does.",
     # NOT TRUE YET: "Apply g2p to a tokenized file, in preparation for alignment."
 )
-@click.argument("tokfile", type=click.File("rb"))
+@click.argument("tokfile", type=click.File("rb", encoding="utf8", lazy=True))
 @click.argument("g2pfile", type=click.Path(), required=False, default="")
 @click.option(
     "--g2p-fallback",
@@ -488,9 +495,7 @@ def g2p(**kwargs):
 
     if not kwargs["g2pfile"]:
         output_path = get_click_file_name(input_file)
-        if output_path == "<stdin>":
-            output_path = "-"
-        else:
+        if output_path != "-":
             if output_path.endswith(".xml"):
                 output_path = output_path[:-4]
             if output_path.endswith(".tokenized"):

test/test_align_cli.py

@@ -167,6 +167,12 @@ def test_invoke_align(self):
         )
 
     def test_permission_denied(self):
+        import platform
+
+        if platform.system() == "Windows":
+            # Cannot change the permission on a directory in Windows though
+            # os.mkdir() or os.chmod(), so skip this test
+            return
         dir = join(self.tempdir, "permission_denied")
         os.mkdir(dir, mode=0o444)
         results = self.runner.invoke(

test/test_g2p_cli.py

@@ -96,7 +96,7 @@ def test_mixed_langs(self):
     # saving the final results into filename.
     # filename is assumed to be inside self.tempdir, so we count on tearDown() to clean up.
     def write_prepare_tokenize(self, text, lang, filename):
-        with open(filename + ".input.txt", "w") as f:
+        with open(filename + ".input.txt", "w", encoding="utf8") as f:
             print(text, file=f)
         self.runner.invoke(
             prepare, ["-l", lang, filename + ".input.txt", filename + ".prepared.xml"]
@@ -197,7 +197,7 @@ def test_three_way_fallback(self):
 
     def test_align_with_error(self):
         text_file = os.path.join(self.tempdir, "input.txt")
-        with io.open(text_file, "w") as f:
+        with io.open(text_file, "w", encoding="utf8") as f:
             print("In French été works but Nunavut ᓄᓇᕗᑦ does not.", file=f)
         empty_wav = os.path.join(self.tempdir, "empty.wav")
         with io.open(empty_wav, "wb"):
@@ -241,7 +241,7 @@ def test_align_with_error(self):
 
     def test_with_stdin(self):
         input_file = os.path.join(self.data_dir, "fra-tokenized.xml")
-        with io.open(input_file) as f:
+        with io.open(input_file, encoding="utf8") as f:
             inputtext = f.read()
         results = self.runner.invoke(g2p, "-", input=inputtext)
         self.assertEqual(results.exit_code, 0)