We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
Good news, the tests pass.
Bad news, if some of the tests skip, then the test fails, e.g:
diff --git a/t/rose-app-run/28-git.t b/t/rose-app-run/28-git.t index 1b4eaaa90..92a411611 100644 --- a/t/rose-app-run/28-git.t +++ b/t/rose-app-run/28-git.t @@ -72,12 +72,12 @@ remote_locations=("$HOSTNAME:$TEST_DIR/hellorepo/" "http://localhost:$GIT_WS_POR for i in 0 1 2; do remote_mode="${remote_test_modes[$i]}" remote="${remote_locations[$i]}" - if [[ "$remote_mode" == "ssh" ]] && ! ssh -n -q -oBatchMode=yes $HOSTNAME true 1>'/dev/null' 2>/dev/null; then + if true; then skip 14 "cannot ssh to localhost $HOSTNAME" echo "Skip $remote" >/dev/tty continue fi - if [[ "$remote_mode" == "http" ]] && ! curl --head --silent --fail $remote >/dev/null 2>&1; then + if true; then skip 14 "failed to launch http on localhost" echo "Skip $remote" >/dev/tty continue
:(
[FAIL] ../config: rose-app.conf not found. Dubious, test returned 1 (wstat 256, 0x100) Failed 15/57 subtests (less 42 skipped subtests: 0 okay) Test Summary Report ------------------- t/rose-app-run/28-git.t (Wstat: 256 Tests: 42 Failed: 0) Non-zero exit status: 1 Parse errors: Bad plan. You planned 57 tests but ran 42.
Not a release blocker. Suggest bumping this to future work.
Originally posted by @oliver-sanders in #2754 (comment)
The text was updated successfully, but these errors were encountered:
Fixed on PR branch
Sorry, something went wrong.
No branches or pull requests
Good news, the tests pass.
Bad news, if some of the tests skip, then the test fails, e.g:
:(
Not a release blocker. Suggest bumping this to future work.
Originally posted by @oliver-sanders in #2754 (comment)
The text was updated successfully, but these errors were encountered: