search

hyrise / Cockpit

A Cockpit for Self-Driving Databases.
15 stars 3 forks source link

Remove e2e from gitHub action #780

Closed Alexander-Dubrawski closed 3 years ago

Alexander-Dubrawski commented 3 years ago

Since the e2e test behaved really inconsistent in GitHub actions, this PR removes them from the GitHub action workflow. So the e2e tests are not run after a commit to the repository in GitHub actions. This doesn't mean however they shouldn't be used. It is still possible to run the e2e test on the local machine.

I created an Issue #781 , so we still try to automate the e2e tests in the future.

Bouncner commented 3 years ago

Can we rename these tests? e2e is a little bit too short ...

Bensk1 commented 3 years ago

How's the behavior of these tests on your local machine? Are they more consistent there?

Alexander-Dubrawski commented 3 years ago

How's the behavior of these tests on your local machine? Are they more consistent there?

So far on my local machine, they are consistent.