Skip to content

Fixed AppVeyor errors.#52

Merged
phated merged 1 commit intogulpjs:masterfrom
sttk:fix_appveyor_errors
Dec 29, 2015
Merged

Fixed AppVeyor errors.#52
phated merged 1 commit intogulpjs:masterfrom
sttk:fix_appveyor_errors

Conversation

@sttk
Copy link
Copy Markdown
Contributor

@sttk sttk commented Dec 29, 2015

Since some test codes caused errors on AppVeyor, I've fixed them.

Modification:

  • appveyor.yml
    Use for statement because the wildcard in npm test was not expanded. (But coverage is unavailable).
  • test/completion.js
    Change stdout handling to reading a redirected file because the argument for stdout was empty on node v0.10 for Windows.
  • test/flags-help.js
    Change stdout handling to reading a redirected file because the argument for stdout contained strange strings in the tail on node v0.10 for Windows.
  • test/flags-tasks-json.js
    Add escaping backslashes in outputString to eliminate JSON.parse errors on Windows, and remove replacing backslashes to slashes in stdout.

phated added a commit that referenced this pull request Dec 29, 2015
@phated phated merged commit 031b163 into gulpjs:master Dec 29, 2015
@phated
Copy link
Copy Markdown
Member

phated commented Dec 29, 2015

Thanks!

@sttk sttk deleted the fix_appveyor_errors branch January 30, 2016 11:26
phated added a commit that referenced this pull request Dec 21, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants