Skip to content

NASA-PDS/validate#1241: improved patch to use local file instead of attempting to download URL#189

Merged
jordanpadams merged 1 commit intomainfrom
validate_1241
Jan 12, 2026
Merged

NASA-PDS/validate#1241: improved patch to use local file instead of attempting to download URL#189
jordanpadams merged 1 commit intomainfrom
validate_1241

Conversation

@al-niessner
Copy link
Copy Markdown
Contributor

@al-niessner al-niessner commented Jan 12, 2026

🗒️ Summary

The validate unit tests found a small snag. Fixed.

⚙️ Test Data and/or Report

Unit testing done here should pass.

♻️ Related Issues

Part of NASA-PDS/validate#1241

🤓 Reviewer Checklist

Reviewers: Please verify the following before approving this pull request.

Security & Quality

  • SonarCloud: Confirmed no new High or Critical security findings.
  • Secrets Detection: Verified that the Secrets Detection scan passed and no sensitive information (keys, tokens, PII) is exposed.
  • Code Quality: Code follows organization style guidelines and best practices for the specific language (e.g., PEP 8, Google Java Style).

Testing & Validation

  • Test Accuracy: Verified that test data is accurate, representative of real-world PDS4 scenarios, and sufficient for the logic being tested.
  • Coverage: Automated tests cover new logic and edge cases.
  • Local Verification: (If applicable) Successfully built and ran the changes in a local or staging environment.

Documentation

  • Documentation: README, Wiki, or inline documentation (Sphinx, Javadoc, Docstrings) have been updated to reflect these changes.

Maintenance

  • Issue Traceability: The PR is linked to a valid GitHub Issue or Jira Ticket.
  • Backward Compatibility: Confirmed that these changes do not break existing downstream dependencies or API contracts (or that breaking changes are clearly documented).

@al-niessner al-niessner self-assigned this Jan 12, 2026
@al-niessner al-niessner requested a review from a team as a code owner January 12, 2026 19:27
@sonarqubecloud
Copy link
Copy Markdown

@al-niessner
Copy link
Copy Markdown
Contributor Author

@jordanpadams @tloubrieu-jpl @nutjob4life

Seems the local tests let a little bit pass through. Caught it in the validate unit tests however and fixed it here. Please review, merge, and deploy. Let me know when complete and I will once again apply my paranoia and test it in validate as well.

@jordanpadams jordanpadams changed the title validate 1241: improved fix NASA-PDS/validate#1241: improved patch to use local file instead of attempting to download URL Jan 12, 2026
Copy link
Copy Markdown
Member

@nutjob4life nutjob4life left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure how the code delta that changes handling of the count in dst.limit affects how a local file is used instead of a download URL, but that's part of the fun! 😉

Code delta: ✓
Tests: ✓
Approval: ✅
Maven details:

…
[INFO] Tests run: 673, Failures: 0, Errors: 0, Skipped: 0
[INFO] 
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time:  32.939 s
…

@jordanpadams jordanpadams merged commit 6322e2b into main Jan 12, 2026
5 of 6 checks passed
@jordanpadams jordanpadams deleted the validate_1241 branch January 12, 2026 20:59
@jordanpadams
Copy link
Copy Markdown
Member

@al-niessner merged. should be deployed in the next 15-30 minutes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants