Skip to content

fix(test): adjust tests and add phpunit workflow - #275

Merged
kyteinsky merged 4 commits into
mainfrom
fix/test-runner-workflow
Sep 2, 2026
Merged

fix(test): adjust tests and add phpunit workflow#275
kyteinsky merged 4 commits into
mainfrom
fix/test-runner-workflow

Conversation

@kyteinsky

Copy link
Copy Markdown
Contributor

also add NC 36 support

🤖 AI (if applicable)

  • The content of this PR was partly or fully generated using AI

@kyteinsky
kyteinsky requested a review from marcelklehr August 28, 2026 01:02
@nextcloud-command nextcloud-command added the AI assisted This PR contains AI-assisted commits label Aug 28, 2026
@kyteinsky
kyteinsky requested a review from julien-nc as a code owner August 28, 2026 01:05
Signed-off-by: kyteinsky <kyteinsky@gmail.com>
Assisted-by: Github Copilot:claude-opus-5
Signed-off-by: kyteinsky <kyteinsky@gmail.com>
@kyteinsky
kyteinsky force-pushed the fix/test-runner-workflow branch from 05f166d to 08c5041 Compare September 1, 2026 03:43
The phpunit workflow failed on Nextcloud 36 with PHP >= 8.4:

  TypeError: OC\Files\Config\UserMountCache::__construct():
  Argument #1 ($connection) must be of type OCP\IDBConnection, null given

- Extend PHPUnit\Framework\TestCase instead of Test\TestCase.
- Add the DB group, since the test uses the real IJobList, and declare it
with the #[Group] attribute.

ref: nextcloud/server#52538

Signed-off-by: kyteinsky <kyteinsky@gmail.com>
Assisted-by: Github Copilot:claude-opus-5
@kyteinsky
kyteinsky force-pushed the fix/test-runner-workflow branch from d6f46d7 to 199f34c Compare September 1, 2026 04:47
Signed-off-by: kyteinsky <kyteinsky@gmail.com>
Assisted-by: Github Copilot:claude-opus-5
@kyteinsky
kyteinsky enabled auto-merge September 2, 2026 11:10
@kyteinsky
kyteinsky disabled auto-merge September 2, 2026 11:11
@kyteinsky
kyteinsky merged commit e850c4c into main Sep 2, 2026
51 of 59 checks passed
@kyteinsky
kyteinsky deleted the fix/test-runner-workflow branch September 2, 2026 11:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

AI assisted This PR contains AI-assisted commits

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants