PhilippRisius / pytest_nbgrader / 23671114911

80%

push

github

test: add 44 tests for cases, dumper, loader + fix dump_subtask append (#30)

* test: add 44 tests for cases, dumper, loader + fix dump_subtask append

New test files:
- test_cases.py (25 tests): Timer, format_result, execute dispatches
  for function/code/type/ModuleSpec/NotImplementedError, dataclass defaults
- test_dumper.py (8 tests): dump_exercise, dump_task hierarchy,
  dump_subtask append mode regression test
- test_loader.py (10 tests): submit(Path) dispatch, print output
  verification for all dispatch paths, generic submit
- test_pytest_nbgrader.py: +1 test for __init__.py re-exports

Bug fix: dump_subtask append mode used "wb+" (truncates) instead of "ab".

Coverage: 45% -> 52%, dumper.py 62% -> 100%, cases.py 28% -> 55%.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

* test: add 31 prerequisites tests + fix writes() bug + strengthen weak tests

- tests/test_prerequisites.py: 31 new tests covering has_signature, writes, writes_file
- src/pytest_nbgrader/prerequisites.py: fix writes() unchecked stream bug
  (comparison loop ran on streams with expected=None, causing false failures)
- tests/test_cases.py: 3 new tests (exception propagation, output count mismatch)
- tests/test_dumper.py: 1 new test (append mode content verification)
- tests/test_loader.py: strengthen print output assertions

Coverage: 52% → 69%, prerequisites.py 11% → 100%

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

---------

Co-authored-by: Philipp Risius <philipp.e.risius@theo.physik.uni-giessen.de>
Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>

3 of 3 new or added lines in 2 files covered. (100.0%)

364 of 533 relevant lines covered (68.29%)

2.05 hits per line