Removed duplicate, and simplified bids validator tests

[TheChymera]

No description provided.

[codecov]

Codecov Report

Merging #965 (786c22b) into master (00ee4dd) will decrease coverage by 0.03%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #965      +/-   ##
==========================================
- Coverage   87.56%   87.52%   -0.04%     
==========================================
  Files          65       64       -1     
  Lines        8104     8104              
==========================================
- Hits         7096     7093       -3     
- Misses       1008     1011       +3     

Flag	Coverage Δ
unittests	87.52% <100.00%> (-0.04%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
dandi/tests/test_bids_validator_xs.py	100.00% <100.00%> (ø)
dandi/validate.py	75.00% <0.00%> (-25.00%)	⬇️
dandi/tests/test_utils.py	97.29% <0.00%> (-2.03%)	⬇️
dandi/bids_validator_xs.py	93.16% <0.00%> (-0.36%)	⬇️
dandi/tests/test_dandiarchive.py	100.00% <0.00%> (ø)
dandi/utils.py	81.44% <0.00%> (+0.17%)	⬆️
dandi/dandiarchive.py	84.93% <0.00%> (+1.25%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 00ee4dd...786c22b. Read the comment docs.

[TheChymera]

@yarikoptic some minor test simplifications you suggested. Doesn't really affect speed right now, just that the one test that took 30s+ is now broken in two.

[yarikoptic]

coolio, thanks! let's proceed. Please in the future do not forge to label the PRs appropriately for the release process: https://github.com/dandi/dandi-cli/blob/HEAD/DEVELOPMENT.md#labelling-pull-requests

[github-actions]

🚀 PR was released in 0.39.3 🚀