Skip to content

Commit 3a98d04

Browse files
committed
Make test-packages actually run again meteor#6470
1 parent 730e713 commit 3a98d04

File tree

2 files changed

+4
-30
lines changed

2 files changed

+4
-30
lines changed

packages/meteor/test_environment.js

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -7,12 +7,14 @@ if (Meteor.isClient) {
77

88
var TEST_METADATA = JSON.parse(TEST_METADATA_STR || "{}");
99

10+
// Note that if we are in test-packages mode neither of these will be set,
11+
// but we will have a test driver package
1012
Meteor.isTest = !!TEST_METADATA.isTest;
1113
Meteor.isAppTest = !!TEST_METADATA.isAppTest;
1214

13-
if (Meteor.isTest || Meteor.isAppTest) {
15+
var testDriverPackageName = TEST_METADATA.driverPackage;
16+
if (testDriverPackageName) {
1417
Meteor.startup(function() {
15-
var testDriverPackageName = TEST_METADATA.driverPackage;
1618
if (typeof testDriverPackageName !== "string") {
1719
throw new Error("No --driver-package specified for `meteor test`");
1820
}

tools/isobuild/linker.js

Lines changed: 0 additions & 28 deletions
Original file line numberDiff line numberDiff line change
@@ -1048,31 +1048,3 @@ export var fullLink = Profile("linker.fullLink", function (inputFiles, {
10481048
}
10491049
});
10501050
});
1051-
1052-
function getTestPreamble() {
1053-
const testDriverPackageName = global.testCommandMetadata.driverPackage;
1054-
1055-
let setMeteorTest = "";
1056-
if (global.testCommandMetadata.isTest) {
1057-
setMeteorTest = "Package.meteor.Meteor.isTest = true;";
1058-
} else if (global.testCommandMetadata.isAppTest) {
1059-
setMeteorTest = "Package.meteor.Meteor.isAppTest = true;";
1060-
}
1061-
1062-
return `\
1063-
${setMeteorTest}
1064-
1065-
if (Package.meteor.Meteor.isClient) {
1066-
Package.meteor.Meteor.startup(function() {
1067-
var testDriverPackage = Package[\"${testDriverPackageName}\"];
1068-
if (!testDriverPackage) {
1069-
throw new Error(\"Can\'t find test driver package: ${testDriverPackageName}\");
1070-
}
1071-
1072-
if (!testDriverPackage.runTests) {
1073-
throw new Error("Test driver package ${testDriverPackageName} missing \`runTests\` export");
1074-
}
1075-
testDriverPackage.runTests();
1076-
});
1077-
}`;
1078-
}

0 commit comments

Comments
 (0)