Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

--disable-context-mismatch-warnings only works for rule pds4.label #950

Closed
jordanpadams opened this issue Jul 3, 2024 · 0 comments · Fixed by #951
Closed

--disable-context-mismatch-warnings only works for rule pds4.label #950

jordanpadams opened this issue Jul 3, 2024 · 0 comments · Fixed by #951
Assignees
@al-niessner
Labels
B15.0 bug Something isn't working i&t.done s.high sprint-backlog

Comments

@jordanpadams
Copy link
Member

jordanpadams commented Jul 3, 2024 •
edited
Loading

Checked for duplicates

Yes - I've already checked

🐛 Describe the bug

When I used pds4.collection rule and tried to disable context mismatch warnings, it does not disable.

🕵️ Expected behavior

I expected the context mismatch warnings to be disabled.

📜 To Reproduce

See new test added to branch that does not work as expected: https://github.com/NASA-PDS/validate/pull/951/files#diff-f6b32571d9820f6326f656fb2fbf401060bb77d9500a4807392393a46b73d1f2R18

🖥 Environment Info

Mac OSx

📚 Version of Software Used

Latest SNAPSHOT

🩺 Test Data / Additional context

No response

🦄 Related requirements

🦄 #861
🦄 #857

⚙️ Engineering Details

NOTE: New branch and draft PR already created for this that includes a new test that fails and should pass: #951

🎉 Integration & Test

No response
@jordanpadams jordanpadams added bug Something isn't working needs:triage labels Jul 3, 2024
@jordanpadams jordanpadams self-assigned this Jul 3, 2024
jordanpadams added a commit that referenced this issue Jul 3, 2024
@jordanpadams
Add test for disabling context ref mismatch
Loading
Loading status checks…
5a81dee 
Adding test that currently fails per #950
@jordanpadams jordanpadams mentioned this issue Jul 3, 2024
Merged
@github-project-automation github-project-automation bot moved this to Release Backlog in B15.0 Jul 3, 2024
@github-project-automation github-project-automation bot moved this from ToDo to 🏁 Done in EN Portfolio Backlog Aug 9, 2024
@github-project-automation github-project-automation bot moved this from Release Backlog to 🏁 Done in B15.0 Aug 9, 2024
jordanpadams added a commit that referenced this issue Oct 17, 2024
@jordanpadams
Update tests to integrate with current feature file paradigm
Loading
Loading status checks…
383b220 
* Moved tests from integration.feature to v3.6.x.feature file.
* Fixed typo bug in test for #950
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@al-niessner al-niessner

Labels
B15.0 bug Something isn't working i&t.done s.high sprint-backlog
Projects
EN Portfolio Backlog
Status: 🏁 Done
B15.0
Status: 🏁 Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix for disabling context ref mismatch for pds4.collection and pds4.bundle rules NASA-PDS/validate
2 participants
@al-niessner @jordanpadams