Merge pull request #6424 from LMFDB/autopep8-patches

Author: edgarcosta

lmfdb/tests/__init__.py

@@ -85,14 +85,14 @@ def check_external(self, homepage, path, text):
             https_handler = HTTPSHandler(context=context)
             opener = build_opener(redirect_handler, http_handler, https_handler)
             response = opener.open(request)
-            
+
             # Check if we were redirected
             final_url = response.geturl()
             if final_url != path:
                 print(f"Redirected from {path} to {final_url}")
-                
+
             response_text = response.read().decode("utf-8")
-            
+
             assert text in response_text, f"Text '{text}' not found in response from {path} (final URL: {final_url})"
         except URLError as e:
             if e.errno in [errno.ETIMEDOUT, errno.ECONNREFUSED, errno.EHOSTDOWN]:

lmfdb/tests/test_homepage.py

@@ -20,14 +20,14 @@ def _get_links_data(cls):
         if cls._links_data is None:
             lmfdb_dir = os.path.dirname(os.path.dirname(os.path.abspath(__file__)))
             yaml_path = os.path.join(lmfdb_dir, "homepage", "index_boxes.yaml")
-            
+
             with open(yaml_path, "r") as f:
                 boxes = list(yaml.load_all(f, Loader=yaml.FullLoader))
-            
+
             links = []
             for box_idx, box in enumerate(boxes):
                 box_title = box.get("title", f"Box {box_idx + 1}")
-                
+
                 if "content" in box:
                     urls = cls._parse_links_from_content(box["content"])
                     for url in urls:
@@ -37,7 +37,7 @@ def _get_links_data(cls):
                             "is_external": url.startswith(('http://', 'https://')),
                             "is_internal": url.startswith('/'),
                         })
-            
+
             cls._links_data = links
         return cls._links_data
 
@@ -52,9 +52,9 @@ def _test_link(self, link_info):
         url = link_info["url"]
         box_title = link_info["box_title"]
         homepage = self._get_homepage()
-        
+
         # Check that link appears in homepage
-        self.assertIn(url, homepage, 
+        self.assertIn(url, homepage,
                      f"Link {url} from {box_title} not found in homepage")
 
         if link_info["is_external"]:
@@ -73,7 +73,7 @@ def test_all_internal_links(self):
         """Test all internal links found in index_boxes.yaml."""
         links = self._get_links_data()
         internal_links = [link for link in links if link["is_internal"]]
-        
+
         for link_info in internal_links:
             with self.subTest(url=link_info["url"], box=link_info["box_title"]):
                 self._test_link(link_info)
@@ -82,23 +82,23 @@ def test_all_external_links(self):
         """Test all external links found in index_boxes.yaml."""
         links = self._get_links_data()
         external_links = [link for link in links if link["is_external"]]
-        
+
         for link_info in external_links:
             with self.subTest(url=link_info["url"], box=link_info["box_title"]):
                 self._test_link(link_info)
 
     def test_all_links_by_box(self):
         """Test all links grouped by box."""
         links = self._get_links_data()
-        
+
         # Group links by box
         boxes = {}
         for link in links:
             box_title = link["box_title"]
             if box_title not in boxes:
                 boxes[box_title] = []
             boxes[box_title].append(link)
-        
+
         for box_title, box_links in boxes.items():
             with self.subTest(box=box_title):
                 for link_info in box_links:
@@ -118,23 +118,23 @@ def test_method(self):
         for link_info in box_links:
             with self.subTest(url=link_info["url"]):
                 self._test_link(link_info)
-    
+
     test_method.__doc__ = f"Test links from box: {box_title}"
     return test_method
 
 
 # Add individual box tests
 try:
     links = HomePageTest._get_links_data()
-    
+
     # Group links by box for dynamic test creation
     boxes = {}
     for link in links:
         box_title = link["box_title"]
         if box_title not in boxes:
             boxes[box_title] = []
         boxes[box_title].append(link)
-    
+
     # Create test methods for each box
     for box_title, box_links in boxes.items():
         safe_name = re.sub(r'[^a-zA-Z0-9_]', '_', box_title.lower())