Skip to content
Permalink
Browse files

42: Notifier tests can fail depending on execution order

Reviewed-by: ehelin
  • Loading branch information
Robin Westberg
Robin Westberg committed Jul 4, 2019
1 parent 965be9c commit e83e47337e32deb4fb239399ed5d3834a106019a
Showing with 6 additions and 3 deletions.
  1. +6 −3 bots/notify/src/test/java/org/openjdk/skara/bots/notify/UpdaterTests.java
@@ -149,7 +149,8 @@ void testMailingList(TestInfo testInfo) throws IOException {
var repoFolder = tempFolder.path().resolve("repo");
var localRepo = CheckableRepository.init(repoFolder, repo.getRepositoryType());
var masterHash = localRepo.resolve("master").orElseThrow();
localRepo.push(masterHash, repo.getUrl(), "master", true);
credentials.commitLock(localRepo);
localRepo.pushAll(repo.getUrl());

var tagStorage = createTagStorage(repo);
var branchStorage = createBranchStorage(repo);
@@ -185,7 +186,8 @@ void testMailingListMultiple(TestInfo testInfo) throws IOException {
var repoFolder = tempFolder.path().resolve("repo");
var localRepo = CheckableRepository.init(repoFolder, repo.getRepositoryType());
var masterHash = localRepo.resolve("master").orElseThrow();
localRepo.push(masterHash, repo.getUrl(), "master", true);
credentials.commitLock(localRepo);
localRepo.pushAll(repo.getUrl());

var tagStorage = createTagStorage(repo);
var branchStorage = createBranchStorage(repo);
@@ -225,7 +227,8 @@ void testMailingListSponsored(TestInfo testInfo) throws IOException {
var repoFolder = tempFolder.path().resolve("repo");
var localRepo = CheckableRepository.init(repoFolder, repo.getRepositoryType());
var masterHash = localRepo.resolve("master").orElseThrow();
localRepo.push(masterHash, repo.getUrl(), "master", true);
credentials.commitLock(localRepo);
localRepo.pushAll(repo.getUrl());

var tagStorage = createTagStorage(repo);
var branchStorage = createBranchStorage(repo);

0 comments on commit e83e473

Please sign in to comment.
You can’t perform that action at this time.