SIP and Transfer Manifest duplicate files and LIDVIDs #29
Comments
nutjob4life
added a commit
that referenced
this issue
Apr 20, 2020
…empting to catalog them while also finding primaries
- Also: make the pairing of lidvid → xml file unique to avoid redundnant cache entries
- Use `insert or ignore`
- Add a functional test that compares generated SIP with "valid" SIP
- And add framework for unit+functional+integration tests
- Test data accessible via setuptools `pkg_resources`
- Resolve #27: An *attempt* to tag test builds for test PyPI with both a date and timestamp; this requires dropping `--tag-date` and coming up with our own date+time format—and hoping that PyPI doesn't enforce its more capricious requirements on release naming
- Add running tests in the CI/CD GitHub Action
- Add ability to run tests with `setup test` as well as buildout's `bin/test`
- Fix extra newlines in log messages
- Remove need of extra `.strip()` calls by using a smarter regexp
- PEP-8 cleanups: remove needless `+` for string concatenation
- Arg list documentation in function calls
- Remove exec bit from test data files
jordanpadams
pushed a commit
that referenced
this issue
Apr 20, 2020
…empting to catalog them while also finding primaries (#38) - Also: make the pairing of lidvid → xml file unique to avoid redundnant cache entries - Use `insert or ignore` - Add a functional test that compares generated SIP with "valid" SIP - And add framework for unit+functional+integration tests - Test data accessible via setuptools `pkg_resources` - Resolve #27: An *attempt* to tag test builds for test PyPI with both a date and timestamp; this requires dropping `--tag-date` and coming up with our own date+time format—and hoping that PyPI doesn't enforce its more capricious requirements on release naming - Add running tests in the CI/CD GitHub Action - Add ability to run tests with `setup test` as well as buildout's `bin/test` - Fix extra newlines in log messages - Remove need of extra `.strip()` calls by using a smarter regexp - PEP-8 cleanups: remove needless `+` for string concatenation - Arg list documentation in function calls - Remove exec bit from test data files
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Describe the bug
Running against the LADEE test bundle it produces duplicates of the applicable files in the SIP and transfer manifest.
Other thoughts
We should include in this bug fix an regression test that can be part of the CI/CD that tests running
pds-deep-archiveagainst the ladee test bundle and verifies what is output from the software produces what we have intest/data/ladee_test/valid/.The text was updated successfully, but these errors were encountered: