Use underscores instead of dashes for spaces

python · Jul 23, 2023 · f98656d · f98656d
1 parent 53ca535
commit f98656d
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 13 deletions.
diff --git a/blurb/blurb.py b/blurb/blurb.py
@@ -114,7 +114,7 @@ def sanitize_section(section):
     """
     Cleans up a section string, making it viable as a directory name.
     """
-    return section.replace("/", "-").replace(" ", "-")
+    return section.replace("/", "-").replace(" ", "_")
 
 
 def sanitize_section_legacy(section):
@@ -125,8 +125,8 @@ def sanitize_section_legacy(section):
 
 
 _unsanitize_section = {
-    "C-API": "C API",
-    "Core-and-Builtins": "Core and Builtins",
+    "C_API": "C API",
+    "Core_and_Builtins": "Core and Builtins",
     "Tools-Demos": "Tools/Demos",
     }
 

diff --git a/blurb/tests/test_blurb.py b/blurb/tests/test_blurb.py
@@ -6,8 +6,8 @@
 @pytest.mark.parametrize(
     "section, expected",
     (
-        ("C API", "C-API"),
-        ("Core and Builtins", "Core-and-Builtins"),
+        ("C API", "C_API"),
+        ("Core and Builtins", "Core_and_Builtins"),
         ("Library", "Library"),
         ("Tools/Demos", "Tools-Demos"),
     ),
@@ -34,9 +34,9 @@ def test_sanitize_section_legacy(section, expected):
 @pytest.mark.parametrize(
     "section, expected",
     (
-        ("C-API", "C API"),
+        ("C_API", "C API"),
         ("C API", "C API"),
-        ("Core-and-Builtins", "Core and Builtins"),
+        ("Core_and_Builtins", "Core and Builtins"),
         ("Core and Builtins", "Core and Builtins"),
         ("Library", "Library"),
         ("Tools-Demos", "Tools/Demos"),
@@ -79,7 +79,7 @@ def test_glob_blurbs_next(fs):
             "Library",
         ),
         (
-            "Misc/NEWS.d/next/Core-and-Builtins/2023-03-17-12-09-45.gh-issue-22222.Pf_BI7.rst",
+            "Misc/NEWS.d/next/Core_and_Builtins/2023-03-17-12-09-45.gh-issue-22222.Pf_BI7.rst",
             "Core and Builtins",
         ),
         (
@@ -91,7 +91,7 @@ def test_glob_blurbs_next(fs):
             "Tools/Demos",
         ),
         (
-            "Misc/NEWS.d/next/C-API/2023-03-27-22-09-07.gh-issue-55555.3SN8Bs.rst",
+            "Misc/NEWS.d/next/C_API/2023-03-27-22-09-07.gh-issue-55555.3SN8Bs.rst",
             "C API",
         ),
         (
@@ -121,16 +121,16 @@ def test_load_next(news_entry, expected_section, fs):
             "root/Misc/NEWS.d/next/Library/2022-04-11-18-34-33.gh-issue-11111.pC7gnM.rst",
         ),
         (
-            "Misc/NEWS.d/next/Core-and-Builtins/2023-03-17-12-09-45.gh-issue-33333.Pf_BI7.rst",
-            "root/Misc/NEWS.d/next/Core-and-Builtins/2023-03-17-12-09-45.gh-issue-33333.Pf_BI7.rst",
+            "Misc/NEWS.d/next/Core_and_Builtins/2023-03-17-12-09-45.gh-issue-33333.Pf_BI7.rst",
+            "root/Misc/NEWS.d/next/Core_and_Builtins/2023-03-17-12-09-45.gh-issue-33333.Pf_BI7.rst",
         ),
         (
             "Misc/NEWS.d/next/Tools-Demos/2023-03-21-01-27-07.gh-issue-44444.2F1Byz.rst",
             "root/Misc/NEWS.d/next/Tools-Demos/2023-03-21-01-27-07.gh-issue-44444.2F1Byz.rst",
         ),
         (
-            "Misc/NEWS.d/next/C-API/2023-03-27-22-09-07.gh-issue-66666.3SN8Bs.rst",
-            "root/Misc/NEWS.d/next/C-API/2023-03-27-22-09-07.gh-issue-66666.3SN8Bs.rst",
+            "Misc/NEWS.d/next/C_API/2023-03-27-22-09-07.gh-issue-66666.3SN8Bs.rst",
+            "root/Misc/NEWS.d/next/C_API/2023-03-27-22-09-07.gh-issue-66666.3SN8Bs.rst",
         ),
     ),
 )