-
Notifications
You must be signed in to change notification settings - Fork 113
Add e2e tests #12
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
Add e2e tests #12
Conversation
This requires the following repository secrets will be configured on GH: E2E_TEST_HOST E2E_TEST_HTTP_PATH E2E_TEST_TOKEN
@@ -1,7 +1,7 @@ | |||
name: Code Quality Checks | |||
on: [push] | |||
jobs: | |||
run-tests: | |||
run-unit-tests: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
what does unit test mean in this context?
Is this mock unit test without the need for databricks account?
or is it integration test e2e which require databricks account
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
unit
means no databricks account is required.
e2e
means databricks account is required.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm updating the CONTRIBUTING doc with this info.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks Jesse.
This reverts commit f7da497. Signed-off-by: Jesse Whitehouse <[email protected]>
Signed-off-by: Jesse Whitehouse <[email protected]>
Signed-off-by: Jesse Whitehouse <[email protected]>
Signed-off-by: Jesse Whitehouse <[email protected]>
Signed-off-by: Jesse Whitehouse <[email protected]> Signed-off-by: Moe Derakhshani <[email protected]>
Signed-off-by: Jesse Whitehouse <[email protected]>
Signed-off-by: Jesse Whitehouse <[email protected]> Signed-off-by: Sai Shree Pradhan <[email protected]>
Description