From e080fdefdf689c1967b73c6a15339dfdb34b2d63 Mon Sep 17 00:00:00 2001 From: ytwei Date: Wed, 26 Aug 2026 21:31:22 +0800 Subject: [PATCH] fix(curriculum): add actionable pathlib assertion messages --- checks/pathlib/pathlib1.py | 8 ++++++-- checks/pathlib/pathlib2.py | 4 +++- checks/pathlib/pathlib3.py | 8 +++++--- checks/pathlib/pathlib4.py | 5 ++++- checks/pathlib/pathlib5.py | 4 +++- checks/pathlib/pathlib6.py | 8 ++++++-- 6 files changed, 27 insertions(+), 10 deletions(-) diff --git a/checks/pathlib/pathlib1.py b/checks/pathlib/pathlib1.py index ed4369d..ac5a715 100644 --- a/checks/pathlib/pathlib1.py +++ b/checks/pathlib/pathlib1.py @@ -1,5 +1,9 @@ from pathlib import Path -assert path == Path("notes/today.txt") -assert filename == "today.txt" +assert path == Path("notes/today.txt"), ( + f"path should be Path('notes/today.txt'), got {path!r}" +) +assert filename == "today.txt", ( + f"filename should be 'today.txt', got {filename!r}" +) print("pathlib1 ok") diff --git a/checks/pathlib/pathlib2.py b/checks/pathlib/pathlib2.py index 87b94a3..eb84ad5 100644 --- a/checks/pathlib/pathlib2.py +++ b/checks/pathlib/pathlib2.py @@ -1,4 +1,6 @@ from pathlib import Path -assert source == Path("pythonlings/src/main.py") +assert source == Path("pythonlings/src/main.py"), ( + f"source should be Path('pythonlings/src/main.py'), got {source!r}" +) print("pathlib2 ok") diff --git a/checks/pathlib/pathlib3.py b/checks/pathlib/pathlib3.py index a6c57dc..298f60e 100644 --- a/checks/pathlib/pathlib3.py +++ b/checks/pathlib/pathlib3.py @@ -1,6 +1,8 @@ from pathlib import Path -assert parent == Path("docs") -assert stem == "guide" -assert suffix == ".md" +assert parent == Path("docs"), ( + f"parent should be Path('docs'), got {parent!r}" +) +assert stem == "guide", f"stem should be 'guide', got {stem!r}" +assert suffix == ".md", f"suffix should be '.md', got {suffix!r}" print("pathlib3 ok") diff --git a/checks/pathlib/pathlib4.py b/checks/pathlib/pathlib4.py index 0f3fbfe..8cf5c1b 100644 --- a/checks/pathlib/pathlib4.py +++ b/checks/pathlib/pathlib4.py @@ -1,4 +1,7 @@ from pathlib import Path -assert python_files == [Path("app.py"), Path("tests.py")] +assert python_files == [Path("app.py"), Path("tests.py")], ( + f"python_files should be [Path('app.py'), Path('tests.py')], " + f"got {python_files!r}" +) print("pathlib4 ok") diff --git a/checks/pathlib/pathlib5.py b/checks/pathlib/pathlib5.py index c6d7e17..2943c7a 100644 --- a/checks/pathlib/pathlib5.py +++ b/checks/pathlib/pathlib5.py @@ -1,4 +1,6 @@ from pathlib import Path -assert final == Path("reports/final.txt") +assert final == Path("reports/final.txt"), ( + f"final should be Path('reports/final.txt'), got {final!r}" +) print("pathlib5 ok") diff --git a/checks/pathlib/pathlib6.py b/checks/pathlib/pathlib6.py index addba6e..ff04253 100644 --- a/checks/pathlib/pathlib6.py +++ b/checks/pathlib/pathlib6.py @@ -1,5 +1,9 @@ from pathlib import Path -assert processed == Path("data/raw.json") -assert parts[-2:] == ("data", "raw.json") +assert processed == Path("data/raw.json"), ( + f"processed should be Path('data/raw.json'), got {processed!r}" +) +assert parts[-2:] == ("data", "raw.json"), ( + f"parts[-2:] should be ('data', 'raw.json'), got {parts[-2:]!r}" +) print("pathlib6 ok")