Skip to content

Resolves #1739 and #1740, getOrg() Error Handling and Documentation#1778

Open
afoote-mitre wants to merge 2 commits intov2.7.6_featurefrom
af-1739
Open

Resolves #1739 and #1740, getOrg() Error Handling and Documentation#1778
afoote-mitre wants to merge 2 commits intov2.7.6_featurefrom
af-1739

Conversation

@afoote-mitre
Copy link
Copy Markdown
Collaborator

Closes Issue #1739 and #1740

Summary

Updated the getOrg() error handling to properly separate errors thrown by the repository, from errors through by mongo, and errors thrown when no results are found. Updated inline comments to properly reflect what each case handles.

Important Changes

src/controller/org.controller/org.controller.js

  • Added new error handling for different errors
  • Clarified accompanying documentation

Testing

Steps to manually test updated functionality, if possible

  • 1) Search for an org that doesn't exist
  • 2) Verify that a 404 returns as expected
  • 3) Stop local mongo instance
  • 4) Search for an org
  • 5) Verify that a 500 returns as expected

@afoote-mitre afoote-mitre self-assigned this Apr 23, 2026
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.

1 participant