-
Notifications
You must be signed in to change notification settings - Fork 434
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
Correct EasyEngine update command - Behat tests #1144
Projects
Comments
Merged
update
behat tests
update
behat tests
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
The update behat test according to current implementation generates a phar and after update it replaces itself with the existing published phar. Hence it downgrades in reality instead of upgrading to the latest version which has not yet been released. Thus, the subsequent tests use the stable released phar which replaced the original latest changes phar. Giving no meaning to the tests performed then after in checking if they are in any way breaking update functionality or anything else.
The tests need to be updated to use the stable released phar in start and then update itself to the phar made of latest changes.
Steps to update the test will be updated here or in comments later with the inputs from @rahulsprajapati and @kirtangajjar. Any suggestions for the same are welcome.
The text was updated successfully, but these errors were encountered: