Skip to content
This repository was archived by the owner on Dec 15, 2022. It is now read-only.

Commit 754d8dc

Browse files
authored
Merge pull request #2573 from atom/deflake-test
Correct a flaky test
2 parents a5245a5 + 78f301f commit 754d8dc

File tree

1 file changed

+5
-4
lines changed

1 file changed

+5
-4
lines changed

test/containers/github-tab-container.test.js

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -195,10 +195,11 @@ describe('GitHubTabContainer', function() {
195195
branches = new BranchSet([otherBranch, mainBranch]);
196196
});
197197

198-
function installRemoteSet(remoteSet) {
199-
return Promise.all(
200-
Array.from(remoteSet, remote => loadedRepo.addRemote(remote.getName(), remote.getUrl())),
201-
);
198+
async function installRemoteSet(remoteSet) {
199+
for (const remote of remoteSet) {
200+
// In your face, no-await-in-loop rule
201+
await loadedRepo.addRemote(remote.getName(), remote.getUrl());
202+
}
202203
}
203204

204205
it('derives the subset of GitHub remotes', async function() {

0 commit comments

Comments
 (0)