Skip to content

Remove species test from root directory, and update the corresponding…#1180

Merged
juhanikat merged 3 commits into
mainfrom
merge-species-test-in-root-directory-to-backend-tests
Jun 29, 2026
Merged

Remove species test from root directory, and update the corresponding…#1180
juhanikat merged 3 commits into
mainfrom
merge-species-test-in-root-directory-to-backend-tests

Conversation

@juhanikat

Copy link
Copy Markdown
Collaborator

… test in api-tests

@juhanikat juhanikat force-pushed the merge-species-test-in-root-directory-to-backend-tests branch from 2401440 to 59d198d Compare June 9, 2026 10:35
@juhanikat juhanikat requested a review from karilint June 9, 2026 14:01
@karilint karilint requested a review from Copilot June 11, 2026 08:38

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR removes the duplicate species “create” API test from the root src/api-tests tree and updates the corresponding backend API test to be the single maintained version.

Changes:

  • Delete src/api-tests/species/create.test.ts (old/duplicate test location).
  • Refactor backend/src/api-tests/species/create.test.ts structure (flatten setup) and adjust permission expectations (EditRestricted behavior).
  • Minor wording update in an assertion comment.

Reviewed changes

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

File Description
src/api-tests/species/create.test.ts Removes the species create API test from the old root-level api-tests location.
backend/src/api-tests/species/create.test.ts Updates the backend species create API test logic/structure and permission expectations.

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

Comment thread backend/src/api-tests/species/create.test.ts
Comment thread backend/src/api-tests/species/create.test.ts Outdated
Comment thread backend/src/api-tests/species/create.test.ts Outdated
@juhanikat juhanikat force-pushed the merge-species-test-in-root-directory-to-backend-tests branch from f50df14 to e50a540 Compare June 16, 2026 09:24
@juhanikat juhanikat force-pushed the merge-species-test-in-root-directory-to-backend-tests branch from e50a540 to 2011317 Compare June 29, 2026 09:53
@juhanikat juhanikat merged commit 8f97de1 into main Jun 29, 2026
4 checks passed
@juhanikat juhanikat deleted the merge-species-test-in-root-directory-to-backend-tests branch June 29, 2026 10:56
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