Added missing ci tests

PR-URL: #15
Credit: @ruyadorno
Close: #15
Reviewed-by: @isaacs

Author: ruyadorno

test/config.js

@@ -0,0 +1,22 @@
+'use strict'
+
+const config = require('../config.js')
+const npmlog = require('npmlog')
+const { test } = require('tap')
+
+npmlog.level = process.env.LOGLEVEL || 'silent'
+
+test('isFromCI config option', t => {
+  const CI = process.env.CI
+  process.env.CI = false
+  t.teardown(t => {
+    process.env.CI = CI
+  })
+  const OPTS = config({
+    log: npmlog,
+    timeout: 0,
+    registry: 'https://mock.reg/'
+  })
+  t.notOk(OPTS.isFromCI, 'should be false if not on a CI env')
+  t.end()
+})

test/index.js

@@ -440,9 +440,23 @@ test('log warning header info', t => {
     .then(res => t.equal(res.status, 200, 'got successful response'))
 })
 
+test('npm-in-ci header with forced CI=false', t => {
+  const CI = process.env.CI
+  process.env.CI = false
+  t.teardown(t => {
+    process.env.CI = CI
+  })
+  tnock(t, OPTS.registry)
+    .get('/hello')
+    .reply(200, { hello: 'world' })
+  return fetch('/hello', OPTS)
+    .then(res => {
+      t.equal(res.status, 200, 'got successful response')
+    })
+})
+
 // TODO
 // * npm-session
-// * npm-in-ci
 // * npm-scope
 // * referer (opts.refer)
 // * user-agent