Skip to content

[tests-only][full-ci] Add test to delete a notification using its id#11530

Merged
anon-pradip merged 1 commit intomasterfrom
test/delete-notification-by-id
Jul 23, 2025
Merged

[tests-only][full-ci] Add test to delete a notification using its id#11530
anon-pradip merged 1 commit intomasterfrom
test/delete-notification-by-id

Conversation

@anon-pradip
Copy link
Contributor

@anon-pradip anon-pradip commented Jul 15, 2025

Description

This PR adds a test coverage for deleting a particular notification using its id.
Listing notifications:

curl "https://localhost:9200/ocs/v2.php/apps/notifications/api/v1/notifications?format=json" -u einstein:relativity -k | jq

Deleting a notification using id:

curl -X DELETE 'https://localhost:9200/ocs/v2.php/apps/notifications/api/v1/notifications/<notification-id>' \
  -u einstein:relativity -k | jq

Related Issue

Motivation and Context

How Has This Been Tested?

  • Locally and CI

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Technical debt
  • Tests only (no source changes)

Checklist:

  • Code changes
  • Unit tests added
  • Acceptance tests added
  • Documentation ticket raised:

@anon-pradip anon-pradip self-assigned this Jul 15, 2025
@anon-pradip anon-pradip force-pushed the test/delete-notification-by-id branch 3 times, most recently from e79d8f3 to 17cf8c5 Compare July 16, 2025 06:31
@anon-pradip anon-pradip marked this pull request as ready for review July 16, 2025 08:24
@anon-pradip anon-pradip force-pushed the test/delete-notification-by-id branch from 17cf8c5 to c6c34cb Compare July 17, 2025 06:37
@anon-pradip anon-pradip force-pushed the test/delete-notification-by-id branch from c6c34cb to 4fe66dc Compare July 18, 2025 04:22
@anon-pradip anon-pradip force-pushed the test/delete-notification-by-id branch 5 times, most recently from 73b234b to 2f165ec Compare July 22, 2025 04:02
@anon-pradip anon-pradip force-pushed the test/delete-notification-by-id branch from 2f165ec to bb8d47e Compare July 23, 2025 11:14
@anon-pradip anon-pradip requested a review from saw-jan July 23, 2025 11:15
@sonarqubecloud
Copy link

@anon-pradip anon-pradip merged commit 4d5fba1 into master Jul 23, 2025
4 checks passed
@anon-pradip anon-pradip deleted the test/delete-notification-by-id branch July 23, 2025 12:31
ownclouders pushed a commit that referenced this pull request Jul 23, 2025
[tests-only][full-ci] Add test to delete a notification using its id
ownclouders pushed a commit that referenced this pull request Jul 24, 2025
[tests-only][full-ci] Add test to delete a notification using its id
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants