Skip to content

Commit 9095927

Browse files
Apply suggestions from code review
Co-authored-by: sungshik <[email protected]>
1 parent 69870e2 commit 9095927

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

src/test/java/engineering/swat/watch/TestHelper.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ public class TestHelper {
66

77
public static final Duration SHORT_WAIT;
88
public static final Duration NORMAL_WAIT;
9-
public final static Duration LONG_WAIT;
9+
public static final Duration LONG_WAIT;
1010

1111
static {
1212
var delayFactorConfig = System.getenv("DELAY_FACTOR");

src/test/java/engineering/swat/watch/TortureTests.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -76,7 +76,7 @@ private void startJob(final Path root, Random r, Executor exec) {
7676
break;
7777
}
7878
try {
79-
// burst a bunch of creates creates and then sleep a bit
79+
// burst a bunch of creates and then sleep a bit
8080
for (int i = 0; i< BURST_SIZE; i++) {
8181
var file = root.resolve("l1-" + r.nextInt(1000))
8282
.resolve("l2-" + r.nextInt(100))
@@ -209,7 +209,7 @@ void manyRegistrationsForSamePath() throws InterruptedException, IOException {
209209
try {
210210
startRegistering.release(TORTURE_REGISTRATION_THREADS);
211211
startDeregistring.release(TORTURE_REGISTRATION_THREADS - 1);
212-
startedWatching.acquire(TORTURE_REGISTRATION_THREADS); // make sure they area ll started
212+
startedWatching.acquire(TORTURE_REGISTRATION_THREADS); // make sure they are all started
213213
done.acquire(TORTURE_REGISTRATION_THREADS - 1);
214214
assertTrue(seen.isEmpty(), "No events should have been sent");
215215
var target = testDir.getTestDirectory().resolve("test124.txt");

0 commit comments

Comments
 (0)