github/meteor
1
1
Fork 0
mirror of https://github.com/meteor/meteor.git synced 2026-05-02 03:01:46 -04:00
Code Issues Packages Projects Releases Wiki Activity

Fix direct dependency reporting.

Browse Source 
And test for it.
This commit is contained in:
Emily Stark
2014-06-24 19:45:25 -07:00
parent e1d4140f35
commit e4ecda9fe8
2 changed files with 9 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
tools/stats.js
View File

@@ -48,7 +48,7 @@ var packageList = function (_currentProjectForTest) {
return {
name: name,
version: version,
direct: _.contains(directDeps, name)
direct: _.has(directDeps, name)
};
}
);

8
tools/tests/report-stats.js
View File

@@ -73,6 +73,14 @@ selftest.define("report-stats", ["slow"], function () {
selftest.expectEqual(_.sortBy(usage.packages, "name"),
_.sortBy(stats.packageList(sandboxProject), "name"));
// Check that the direct dependency was recorded as such.
_.each(usage.packages, function (package) {
if (package.name === "local-package" &&
! package.direct) {
selftest.fail("local-package is not marked as a direct dependency");
}
});
// verify that the stats server recorded that with no userId
var appPackages = stats.getPackagesForAppIdInTest(sandboxProject);
selftest.expectEqual(appPackages.appId, identifier);