Skip to content

add test suite#88

Open
orthagh wants to merge 3 commits intomasterfrom
feat/tests
Open

add test suite#88
orthagh wants to merge 3 commits intomasterfrom
feat/tests

Conversation

@orthagh
Copy link
Contributor

@orthagh orthagh commented Mar 24, 2026

Current test-project to evaluate claude capacity is to modernize this repository.
First steps:

  • add phpunit tests on api (with a specific case on api/src/core/PaginatedCollection.php that use sub-queries, and so a sqlite has been added instead mocking)
  • add integration tests with a real database
  • add e2e tests on the frontend

Copilot AI review requested due to automatic review settings March 24, 2026 08:57
@orthagh orthagh changed the title add api test units add test suite Mar 24, 2026

This comment was marked as resolved.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 15 out of 17 changed files in this pull request and generated 3 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@orthagh orthagh force-pushed the feat/tests branch 6 times, most recently from 0b6c244 to e428ca0 Compare March 25, 2026 09:00
@orthagh orthagh force-pushed the feat/tests branch 2 times, most recently from 25e7475 to 22c8173 Compare March 25, 2026 14:53
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.

2 participants