-
Notifications
You must be signed in to change notification settings - Fork 9
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.
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
Integration Tests CI #21
Labels
hygiene
Clean-up and maintenance work that needs to be implemented
Comments
norton120
added
bug
Something isn't working
hygiene
Clean-up and maintenance work that needs to be implemented
and removed
bug
Something isn't working
labels
Feb 26, 2020
ankur112358
added a commit
to ankur112358/snowshu
that referenced
this issue
Mar 8, 2021
ankur112358
added a commit
to ankur112358/snowshu
that referenced
this issue
Mar 8, 2021
ankur112358
added a commit
to ankur112358/snowshu
that referenced
this issue
Mar 8, 2021
ankur112358
added a commit
to ankur112358/snowshu
that referenced
this issue
Mar 8, 2021
ankur112358
added a commit
to ankur112358/snowshu
that referenced
this issue
Mar 8, 2021
ankur112358
added a commit
to ankur112358/snowshu
that referenced
this issue
Mar 8, 2021
ankur112358
added a commit
to ankur112358/snowshu
that referenced
this issue
Mar 18, 2021
The exec commands run properly. Linting also performed. Ref Health-Union#21
ankur112358
added a commit
to ankur112358/snowshu
that referenced
this issue
Mar 18, 2021
Renaming the file from `dockerfile` to `Dockerfile` fixes the file not found error in CI. Ref Health-Union#21
ankur112358
added a commit
to ankur112358/snowshu
that referenced
this issue
Mar 18, 2021
Renaming the file from `dockerfile` to `Dockerfile` fixes the file not found error in CI. Ref Health-Union#21
ankur112358
added a commit
to ankur112358/snowshu
that referenced
this issue
Mar 18, 2021
Remove unnecessary step of installation. Run linting before tests. Ref Health-Union#21
ankur112358
added a commit
to ankur112358/snowshu
that referenced
this issue
Mar 19, 2021
Credential file is required to run all the tests. This uses the credentials from the environment secret SNOWFLAKE_CREDENTIALS set up in the SNOWSHU_DEVELOPMENT environment. Ref Health-Union#21
ankur112358
added a commit
to ankur112358/snowshu
that referenced
this issue
Mar 20, 2021
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Why?
Docker - In - Docker for CI isn’t working out 😕
We need a strategy for integration tests that will likely require a bunch of pipelines & test modifications
Def of Done
tests that run and pass on a local machine (inside containers) also pass in CI.
[Issue created by ethanknox: 2020-01-17]
[Last updated on bitbucket: 2020-01-20]
[Comment created by ethanknox: 2020-01-17]
@{557058:14b4a374-d910-43c3-83d6-270f344452e0} we should probably include this in 0.0.1 Alpha right?
The text was updated successfully, but these errors were encountered: