ci: Add gitea testcontainer for environment testing #21008
Conversation
Allows us to override the configs of containers per test
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
| const DEFAULT_PASSWORD = 'giteapassword'; | ||
| const DEFAULT_REPO = 'n8n-test-repo'; | ||
| const DEFAULT_EMAIL = 'admin@example.com'; | ||
| const DEFAULT_BRANCHES = ['development', 'staging', 'production']; |
There was a problem hiding this comment.
Should we only have a main branch to align with the default branch name in the environments.ee feature?
See packages/cli/src/environments.ee/source-control/constants.ts
| await expect(n8n.page.getByRole('option', { name: 'development' })).toBeVisible(); | ||
| await expect(n8n.page.getByRole('option', { name: 'staging' })).toBeVisible(); | ||
| await expect(n8n.page.getByRole('option', { name: 'production' })).toBeVisible(); |
There was a problem hiding this comment.
should this be part of a specific test for the multi-branch setup. Because by default we start with a single branch setup
|
E2E Tests: n8n tests failed after 7m 5.9s Run Details
Groups
This message was posted automatically by
currents.dev | Integration Settings
|
![cubic-dev-ai[bot]](https://avatars.githubusercontent.com/in/1082092?s=60&v=4){kind=small}
shortstacked
changed the title
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
shortstacked
force-pushed
the
cat-1507-testcontainers-environments
branch
from
52d0284 to
0c4cf6a
Compare
3 participants
Summary
Adds Gitea as a test container to facilitate source control integration testing. This includes setting up a Gitea instance with a default admin user and repository, enabling connection from n8n via SSH.
Related Linear tickets, Github issues, and Community forum posts
Related to CAT-1507
Review / Merge checklist
release/backport(if the PR is an urgent fix that needs to be backported)