Skip to content

test(webpush): Add integration test for webpush that shows the mozart…#2886

Merged
nickvergessen merged 2 commits intomasterfrom
tests/noid/integration-webpush
Mar 31, 2026
Merged

test(webpush): Add integration test for webpush that shows the mozart…#2886
nickvergessen merged 2 commits intomasterfrom
tests/noid/integration-webpush

Conversation

@nickvergessen
Copy link
Copy Markdown
Member

… 1.2.1 bug

Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen nickvergessen self-assigned this Mar 31, 2026
@nickvergessen nickvergessen added this to the Nextcloud 34 milestone Mar 31, 2026
@nickvergessen
Copy link
Copy Markdown
Member Author

Tested locally with 1.2.1 and it shows the error:

[Tue Mar 31 13:39:32 2026] 127.0.0.1:60132 [500]: POST /ocs/v2.php/apps/notifications/api/v2/webpush?format=json
[Tue Mar 31 13:39:32 2026] 127.0.0.1:60132 Closing
    Given user "test1" registers for webpush with                  # FeatureContext::registerForWebPush()
      | endpoint    | https://push.example.com/test |
      | uaPublicKey | VALID_KEY                     |
      | auth        | VALID_AUTH                    |
      | appTypes    | all                           |
      Server error: `POST http://localhost:8080/ocs/v2.php/apps/notifications/api/v2/webpush?format=json` resulted in a `500 Internal Server Error` response:
      {"ocs":{"meta":{"status":"failure","statuscode":996,"message":"Internal Server Error\nClass \"OCA\\Notifications\\Vendor (truncated...)
       (GuzzleHttp\Exception\ServerException)

Signed-off-by: Joas Schilling <coding@schilljs.com>
@nextcloud nextcloud deleted a comment from github-actions Bot Mar 31, 2026
@nickvergessen nickvergessen enabled auto-merge March 31, 2026 12:00
@nickvergessen nickvergessen merged commit 0af1d72 into master Mar 31, 2026
47 checks passed
@nickvergessen nickvergessen deleted the tests/noid/integration-webpush branch March 31, 2026 12:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants