From ee10b47da79ac7ebb83387b0f3a85f43ef176da7 Mon Sep 17 00:00:00 2001 From: David Glasser Date: Thu, 7 Mar 2013 00:06:19 -0800 Subject: [PATCH] use multiple arguments instead of comma-separation for test-packages also, update its docs --- engine/meteor.js | 9 +++------ 1 file changed, 3 insertions(+), 6 deletions(-) diff --git a/engine/meteor.js b/engine/meteor.js index be7a785de7..679648a4ba 100644 --- a/engine/meteor.js +++ b/engine/meteor.js @@ -192,12 +192,9 @@ Fiber(function () { "meteor test-packages [--release=x.y.z] --package-dir= [options]\n" + " Runs unit tests on one package located on disk.\n" + "\n" + - "meteor test-packages [--release=x.y.z] [comma delimited package names] [options]\n" + + "meteor test-packages [--release=x.y.z] [options] [package...]\n" + " Runs unit tests for a set of packages.\n" + "\n" + - "You must either pass the --release option, or be in a git checkout of Meteor,\n" + - "in which cases local packages are loaded.\n" + - "\n" + "Point your browser to localhost:3000 to run tests and see results.\n"); var new_argv = opt.argv; @@ -213,8 +210,8 @@ Fiber(function () { // was important to strip trailing path.sep, otherwise we'd get a package named "foo/" var packageName = path.basename(packageDir); testPackages = [packages.loadFromDir(packageName, packageDir)]; - } else if (new_argv._[1]) { - testPackages = new_argv._[1].split(','); + } else if (!_.isEmpty(argv._)) { + testPackages = argv._; } else if (!files.usesWarehouse()) { testPackages = _.keys(packages.list()); } else {