-
Notifications
You must be signed in to change notification settings - Fork 1
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
serve: Fix nc command, test output matching
On Linux, `nc` will exit with an error if the hostname argument is empty. Setting it to `localhost` by default fixes the problem. On Travis, it appears that file descriptors for standard output and error still look like terminal file descriptors to Bash, even after being redirected to a file (i.e. `[[ -t 0 && -t 1 ]]` is always true)`. This causes the `log` module from `go-script-bash` to always emit terminal-formatted log labels. This played havoc with the previous `assert_output_matches` assertions, which only checked for whitespace between log label text and the log message text. The fix was to update each assertion so that all log labels are immediately followed by `.* `.
- Loading branch information
Showing
2 changed files
with
14 additions
and
15 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters