Skip to content

Add integration test#178

Draft
julien-nc wants to merge 14 commits intomainfrom
enh/noid/oauth-integration-test
Draft

Add integration test#178
julien-nc wants to merge 14 commits intomainfrom
enh/noid/oauth-integration-test

Conversation

@julien-nc
Copy link
Member

@julien-nc julien-nc commented Mar 14, 2026

  • use phpunit 11 to get the external dependency feature (used in integration tests)
  • bump min NC version to 33 and max to 34 because phpunit 11 only supports php >= 8.2 and NC 33 is the first to drop support for php 8.1.
  • update composer dependencies
  • add oauth integration test that performs a full oauth flow with credentials stored as repo secrets (with totp support as we can't perform device validation per email 😁)
  • add feature integration tests that depend on the oauth test so we know we have a token stored in the DB

Signed-off-by: Julien Veyssier <julien-nc@posteo.net>
Signed-off-by: Julien Veyssier <julien-nc@posteo.net>
Signed-off-by: Julien Veyssier <julien-nc@posteo.net>
@julien-nc julien-nc force-pushed the enh/noid/oauth-integration-test branch 2 times, most recently from d910f50 to e05221c Compare March 14, 2026 01:23
Signed-off-by: Julien Veyssier <julien-nc@posteo.net>
@julien-nc julien-nc force-pushed the enh/noid/oauth-integration-test branch from e05221c to 73a3375 Compare March 14, 2026 01:38
…nse of the POST /session request

Signed-off-by: Julien Veyssier <julien-nc@posteo.net>
Signed-off-by: Julien Veyssier <julien-nc@posteo.net>
Signed-off-by: Julien Veyssier <julien-nc@posteo.net>
Signed-off-by: Julien Veyssier <julien-nc@posteo.net>
…lhost/apps/int_github/...

Signed-off-by: Julien Veyssier <julien-nc@posteo.net>
Signed-off-by: Julien Veyssier <julien-nc@posteo.net>
Signed-off-by: Julien Veyssier <julien-nc@posteo.net>
@julien-nc julien-nc force-pushed the enh/noid/oauth-integration-test branch 5 times, most recently from f9ba1f9 to 9bf9aea Compare March 14, 2026 15:41
…he oauth one

Signed-off-by: Julien Veyssier <julien-nc@posteo.net>
@julien-nc julien-nc force-pushed the enh/noid/oauth-integration-test branch from 9bf9aea to 8682f3e Compare March 14, 2026 15:59
Signed-off-by: Julien Veyssier <julien-nc@posteo.net>
@julien-nc julien-nc force-pushed the enh/noid/oauth-integration-test branch 2 times, most recently from 2becb1f to b63d540 Compare March 14, 2026 16:13
…rom nextcloud/.github

Signed-off-by: Julien Veyssier <julien-nc@posteo.net>
@julien-nc julien-nc force-pushed the enh/noid/oauth-integration-test branch from b63d540 to a9c420a Compare March 14, 2026 16:19
@julien-nc julien-nc changed the title Add oauth integration test Add integration test Mar 14, 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