FAIRmat-NFDI / nomad-parser-plugins-simulation / 21077212708 / 1
63%
develop: 66%

LAST BUILD BRANCH: rename-annotation.keys
DEFAULT BRANCH: develop
Ran
Files 77
Run time 2s
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
coverage: 62.608% (-0.7%) from 63.286%
21077212708.1

Pull #129

github

ndaelman-hu 
Fix parser test isolation issues by removing annotation contamination

This commit addresses two related test isolation problems:

1. VASP tests failing after exciting/fhiaims/h5md run
   - Removed remove_mapping_annotations() calls from exciting, fhiaims, h5md
   - These were removing annotations from shared parent classes, breaking VASP

2. Quantum Espresso tests failing in full suite due to annotation accumulation
   - Multiple parsers (abinit, ams, crystal, octopus, QE) use key 'out'
   - Annotations accumulated across parsers (~111 annotations before QE)
   - Added remove_annotation_key() utility to selectively clear specific keys
   - QE now clears 'out' key before parsing to get clean slate

Changes:
- parsers/exciting/parser.py: Remove remove_mapping_annotations() call
- parsers/fhiaims/parser.py: Remove remove_mapping_annotations() call
- parsers/h5md/parser.py: Remove remove_mapping_annotations() call
- parsers/quantumespresso/parser.py: Add remove_annotation_key('out') before parsing
- schema_packages/utils.py: Add remove_annotation_key() helper function

Documentation:
- VASP_TEST_ISOLATION_FIX.md: Comprehensive investigation and fix documentation
- PARSER_ANNOTATIONS.md: Analysis of annotation key collisions across parsers

Test results: All 42 parser tests now pass (previously 40 passed, 2 failed)
Pull Request #129: Add pseudopot parsing

4365 of 6972 relevant lines covered (62.61%)

0.63 hits per line

Source Files on job 21077212708.1