Skip to content

Commit

Permalink
improve some tests
Browse files Browse the repository at this point in the history
  • Loading branch information
glasser committed Dec 1, 2014
1 parent e956a3b commit dc97864
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 8 deletions.
1 change: 1 addition & 0 deletions tools/tests/create.js
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,7 @@ selftest.define("create", function () {

// Can we create an app? Yes!
var run = s.run("create", "foobar");
run.waitSecs(15);
run.match("foobar: created");
run.match("To run your new app");
run.expectExit(0);
Expand Down
16 changes: 8 additions & 8 deletions tools/tests/publish.js
Original file line number Diff line number Diff line change
Expand Up @@ -93,7 +93,7 @@ selftest.define("publish-and-search",
run = s.run("publish", "--create");
run.waitSecs(30);
run.expectExit(0);
run.match("Done");
run.match("Published");
});

run = s.run("show", newPackageName);
Expand Down Expand Up @@ -123,7 +123,7 @@ selftest.define("publish-one-arch",
run = s.run("publish", "--create");
run.waitSecs(15);
run.expectExit(0);
run.match("Done");
run.match("Published");
run.forbidAll("WARNING");

packageName = utils.randomToken();
Expand Down Expand Up @@ -160,7 +160,7 @@ selftest.define("list-with-a-new-version",
run = s.run("publish", "--create");
run.waitSecs(15);
run.expectExit(0);
run.match("Done");
run.match("Published");
});

// Create an app. Add the package to it. Check that list shows the package and
Expand All @@ -186,7 +186,7 @@ selftest.define("list-with-a-new-version",
run = s.run("publish");
run.waitSecs(15);
run.expectExit(0);
run.match("Done");
run.match("Published");
});

// cd into the app and run list again. We should get some sort of message.
Expand Down Expand Up @@ -239,7 +239,7 @@ selftest.define("list-with-a-new-version",
run = s.run("publish");
run.waitSecs(15);
run.expectExit(0);
run.match("Done");
run.match("Published");
});

s.cd('mapp', function () {
Expand Down Expand Up @@ -421,15 +421,15 @@ selftest.define("package-depends-on-either-version",
s.cd(fullPackageNameDep, function() {
run = s.run("publish", "--create");
run.waitSecs(20);
run.match("Done");
run.match("Published");
});

// Then, we publish fullPackageNameDep at 2.0.
s.cd(fullPackageNameDep, function() {
s.cp("package3.js", "package.js");
run = s.run("publish");
run.waitSecs(20);
run.match("Done");
run.match("Published");
});

// Then, we make another one that depends on either version and publish.
Expand All @@ -445,7 +445,7 @@ selftest.define("package-depends-on-either-version",
s.write("package.js", packOpen);
run = s.run("publish", "--create");
run.waitSecs(20);
run.match("Done");
run.match("Published");
});

// Now we add them to an app.
Expand Down

0 comments on commit dc97864

Please sign in to comment.