Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Update int tests setup with setup script #718

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

Update int tests setup with setup script #718

wants to merge 1 commit into from

Conversation

sfc-gh-turbaszek
Copy link
Contributor

Pre-review checklist

  • I've confirmed that instructions included in README.md are still correct after my changes in the codebase.
  • I've added or updated automated unit tests to verify correctness of my new code.
  • I've added or updated integration tests to verify correctness of my new code.
  • I've confirmed that my changes are working by executing CLI's commands manually.
  • I've confirmed that my changes are up-to-date with the target branch.
  • I've described my changes in the release notes.
  • I've described my changes in the section below.

Changes description

...

sfc-gh-astus
sfc-gh-astus reviewed Feb 1, 2024
View reviewed changes
CONTRIBUTING.md
Add the following connection to your `config.toml`
### Configuring integration tests connection
For running integration tests you will need to prepare your account. To do so you can execute the sql script
located in `test_integration/scripts/integration_account_setup.sql`. It will create required users, roles,
Copy link
Contributor

Choose a reason for hiding this comment

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

Should we add here exception with spcs service which requires docker image in repository?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sfc-gh-astus sfc-gh-astus sfc-gh-astus left review comments

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@sfc-gh-turbaszek @sfc-gh-astus