Skip to content
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

Fix coveralls #1541

Merged
merged 4 commits into from Apr 22, 2015
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
6 changes: 6 additions & 0 deletions tests/test-agentOptions.js
@@ -1,5 +1,11 @@
'use strict'

if (process.env.running_under_istanbul) {
// test-agent.js modifies the process state
// causing these tests to fail when running under single process via tape
return
}

var request = require('../index')
, http = require('http')
, server = require('./server')
Expand Down
3 changes: 3 additions & 0 deletions tests/test-har.js
@@ -1,5 +1,6 @@
'use strict'

var path = require('path')
var request = require('..')
var tape = require('tape')
var fixture = require('./fixtures/har.json')
Expand Down Expand Up @@ -107,6 +108,8 @@ tape('multipart-file', function (t) {
url: s.url,
har: fixture['multipart-file']
}
var absolutePath = path.resolve(__dirname, options.har.postData.params[0].fileName)
options.har.postData.params[0].fileName = absolutePath

request(options, function (err, res, body) {
var json = JSON.parse(body)
Expand Down
8 changes: 7 additions & 1 deletion tests/test-https.js
Expand Up @@ -117,4 +117,10 @@ function runAllTests(strict, s) {
}

runAllTests(false, s)
runAllTests(true, sStrict)

if (!process.env.running_under_istanbul) {
// somehow this test modifies the process state
// test coverage runs all tests in a single process via tape
// executing this test causes one of the tests in test-tunnel.js to throw
runAllTests(true, sStrict)
}