Skip to content

[Updated / Fixed / Added] Refactor test cases for updated API definitions - #130

Merged
sachinvodafone merged 19 commits into
camaraproject:mainfrom
eric-murray:eric-murray-patch-1
Oct 8, 2026
Merged

sachinvodafone merged 19 commits into
camaraproject:mainfrom
eric-murray:eric-murray-patch-1

Conversation

@eric-murray

Copy link
Copy Markdown
Contributor

What type of PR is this?

  • correction
  • enhancement/feature
  • cleanup
  • documentation
  • tests

What this PR does / why we need it:

Refactors test cases for updated API definitions:

  • Fix typos in test cases
  • Create separate feature files for each defined operation
  • Add additional test cases
    • Add 429 error test cases for checkDataVolume, retrieveDataVolume and createDeviceDataVolumeSubscription operations
    • Add 401 error test cases for all operations
    • Add 422 error test cases for createDeviceDataVolumeSubscription operations

Which issue(s) this PR fixes:

Fixes #120

Special notes for reviewers:

None

Changelog input

 release-note
 - Refactor test cases for updated API definitions

Additional documentation

None

Removed multiple scenarios related to retrieving and deleting subscriptions, along with error handling for unknown subscription IDs.
Removed synchronous subscription creation scenarios and adjusted feature title for retrieveDeviceDataVolumeSubscription.
Removed multiple synchronous and asynchronous subscription creation scenarios and their examples from the feature file.
Removed 404 error code section and added a placeholder for 429 error code.
Removed commented error codes for 404 and 422, added note for missing test cases.
Added scenarios for error code 429 handling, including 'Too Many Requests' and 'Quota Exceeded'.
Added test scenarios for error code 429 to handle rate limiting and quota exceeded cases.
Added a note regarding test conditions for error code 422 and removed the section for error code 429.
@sachinvodafone
sachinvodafone merged commit e146f15 into camaraproject:main Oct 8, 2026
2 checks passed
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.

Subscriptions API: test definitions reference nonexistent schemas/properties and use inconsistent status wording

2 participants