Further refining --only

- Set 'this.runOnly' as the flag on the test object.  Too confusing otherwise.
- Handle the environment variable properly in the test runner.
- Downgrade the stderr warning to a comment.

bin/run.js

@@ -768,6 +768,7 @@ function runTests (options) {
   // At this point, we know we need to use the tap root,
   // because there are 1 or more files to spawn.
   var tap = require('../lib/tap.js')
+  tap.runOnly = false
 
   // greps are passed to children, but not the runner itself
   tap.grep = []

lib/point.js

@@ -31,8 +31,6 @@ function tpMessage (message, extra) {
     message += ' # TODO'
     if (typeof extra.todo === 'string')
       message += ' ' + extra.todo
-  } else if (extra.only) {
-    message += ' # ONLY'
   } else if (extra.time)
     message += ' # time=' + extra.time + 'ms'
 

lib/tap.js

@@ -8,6 +8,7 @@ var yaml = require('js-yaml')
 util.inherits(TAP, Test)
 function TAP (options) {
   Test.call(this, options)
+  this.runOnly = process.env.TAP_ONLY === '1'
   this.start = Date.now()
 }
 

lib/test.js

@@ -122,6 +122,14 @@ Test.prototype.sub = function (Class, extra, caller) {
     }
   }
 
+  if (extra.only && !this.runOnly) {
+    this.comment('%j has `only` set but all tests run', extra.name)
+  }
+
+  if (this.runOnly && !extra.only) {
+    extra.skip = true
+  }
+
   if (extra.todo || extra.skip) {
     this.pass(extra.name, extra)
     return Promise.resolve(this)