Skip to content

Commit f999886

Browse files
feanilclaude
andcommitted
chore: clean up test_root/data/ and ignore generated artifacts
At some point output from a test run got accidentally checked into test_root/data/, which made it hard to fully gitignore the other test-run artifacts that land there. Delete the orphan files and switch the directory to allow-list mode: ignore everything by default and re-include the video/ fixtures we want to keep on purpose. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
1 parent b85f12a commit f999886

4 files changed

Lines changed: 8 additions & 0 deletions

File tree

  • test_root/data
    • Q291cnNlTG9jYXRvcignb3JnLjIzMTYnLCAnY291cnNlXzIzMTYnLCAnUnVuXzIzMTYnLCBOb25lLCBOb25lKQ==
    • Q291cnNlTG9jYXRvcignb3JnLjIzMTknLCAnY291cnNlXzIzMTknLCAnUnVuXzIzMTknLCBOb25lLCBOb25lKQ==

.gitignore

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -89,6 +89,14 @@ pytest_task*.txt
8989
common/test/data/test_unicode/static/
9090
test_root/courses/
9191
test_root/data/test_bare.git/
92+
# `test_root/data/` is `settings.GITHUB_REPO_ROOT` in the test settings;
93+
# course-import code (`cms/djangoapps/contentstore/.../tasks.py::import_olx`)
94+
# writes per-course subdirectories here named `urlsafe_b64encode(repr(course_key))`.
95+
# Ignore everything in that directory by default and explicitly re-include the
96+
# fixtures we actually want tracked. New fixture additions need an extra `!` line
97+
# below.
98+
test_root/data/*
99+
!test_root/data/video
92100
test_root/export_course_repos/
93101
test_root/paver_logs/
94102
test_root/uploads/

test_root/data/.gitkeep

Whitespace-only changes.

0 commit comments

Comments
 (0)