From 00a7da3dd06be16f3cc8ebe79911642297bf853c Mon Sep 17 00:00:00 2001 From: David Glasser Date: Sat, 20 Dec 2014 20:38:25 -0800 Subject: [PATCH] Fix `test-packages *` if some packages lack tests Fixes #3334. This was an intentional but poorly thought through change in 1.0.2 (1a9536baa). --- tools/commands.js | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/tools/commands.js b/tools/commands.js index ac1fd638f5..2ad57b28a4 100644 --- a/tools/commands.js +++ b/tools/commands.js @@ -1470,10 +1470,11 @@ var getTestPackageNames = function (projectContext, packageNames) { } if (version.testName) { testPackages.push(version.testName); - } else { - // This case only happens when explicitly asked for. - buildmessage.error("Package has no tests"); } + // It is not an error to mention a package by directory that is a + // package but has no tests; this means you can run `meteor + // test-packages $APP/packages/*` without having to worry about the + // packages that don't have tests. } }); });