Merge pull request #177 from johnludwigm/main

doctor/tasks.py

@@ -146,7 +146,7 @@ def get_xray(path):
         PdfReadError,
     ):
         return {"error": True, "msg": "Exception"}
-    except Exception as e:
+    except Exception:
         return {"error": True, "msg": "Exception"}
     # not reached
 

doctor/tests.py

@@ -159,16 +159,6 @@ def test_wpd_format(self):
 class ThumbnailTests(unittest.TestCase):
     """Can we generate thumbnail images from PDF files"""
 
-    # def test_locally(self):
-    #     c = Client()
-    #     with open(f"{asset_path}/image-pdf.pdf", "rb") as fp:
-    #         response = c.post(
-    #             "/convert/pdf/thumbnail/",
-    #             {"max_dimension": 350, "file": fp},
-    #         )
-    #
-    #     self.assertEqual(200, response.status_code)
-
     def test_convert_pdf_to_thumbnail_png(self):
         """Can we generate four thumbanils a pdf?"""
         files = make_file(filename="image-pdf.pdf")
@@ -372,7 +362,6 @@ def test_xray_no_pdf(self):
             direction = re.search("yes", pattern)
             for file in glob.glob(os.path.join(filepath, pattern)):
                 filename = os.path.relpath(file, filepath)
-                filename_sans_ext = filename.split(".")[0]
 
                 with open(file, "rb") as f:
                     files = {"file": (filename, f.read())}

doctor/views.py

@@ -342,7 +342,6 @@ def convert_audio(request) -> Union[FileResponse, HttpResponse]:
 
     :return: Converted audio
     """
-    # try:
     form = AudioForm(request.GET, request.FILES)
     if not form.is_valid():
         return HttpResponse("Failed validation", status=BAD_REQUEST)
@@ -354,8 +353,6 @@ def convert_audio(request) -> Union[FileResponse, HttpResponse]:
     response = FileResponse(open(filepath, "rb"))
     cleanup_form(form)
     return response
-    # except Exception as e:
-    #     return HttpResponse(str(e))
 
 
 def embed_text(request) -> Union[FileResponse, HttpResponse]: