Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
The CLI unit tests were not running correctly for a while.
The problem was:
~/.apify
directory exists (meaing somebody is already logged in withapify login
)checkLatestVersion
method was called, which writes the latest version to~/.apify/state.json
~/.apify
existence to be skippedI fixed it so that:
checkLatestVersion
is skipped when theAPIFY_CLI_SKIP_UPDATE_CHECK
environment variable is setAPIFY_CLI_SKIP_UPDATE_CHECK=1
~/.apify
directory exists at their start (so that we notice these issues immediately next time)after()
hook is skipped if thebefore()
hook throws an error (this was causing somechdir
s to be called when they shouldn't be)Fixing the test execution revealed some errors in the
push
test, so I fixed those too - I mostly deleted some tests which didn't really make sense.I also fixed the tests to clean up the created actors on the platform.
Closes #369.