diff --git a/components/net/sf/briar/plugins/file/UnixRemovableDriveMonitor.java b/components/net/sf/briar/plugins/file/UnixRemovableDriveMonitor.java index 0a7d251d449b35b91c1689ae8a2e2df1dc6543bc..a70f338e7f5dfa426856e6d0bf4c107fecdc966b 100644 --- a/components/net/sf/briar/plugins/file/UnixRemovableDriveMonitor.java +++ b/components/net/sf/briar/plugins/file/UnixRemovableDriveMonitor.java @@ -28,7 +28,7 @@ JNotifyListener { } synchronized(this) { assert !started; - assert callback == null; + assert this.callback == null; started = true; this.callback = callback; this.watches.addAll(watches); diff --git a/test/net/sf/briar/plugins/file/PollingRemovableDriveMonitorTest.java b/test/net/sf/briar/plugins/file/PollingRemovableDriveMonitorTest.java index d8932519495b39557abc80a217326caba0848cab..f3659ea3ad7ba3236a90d312154ea64df1477ec4 100644 --- a/test/net/sf/briar/plugins/file/PollingRemovableDriveMonitorTest.java +++ b/test/net/sf/briar/plugins/file/PollingRemovableDriveMonitorTest.java @@ -76,7 +76,11 @@ public class PollingRemovableDriveMonitorTest extends BriarTestCase { // Create the monitor, start it, and give it some time to run final RemovableDriveMonitor monitor = new PollingRemovableDriveMonitor( Executors.newCachedThreadPool(), finder, 10); - monitor.start(null); + monitor.start(new Callback() { + public void driveInserted(File root) { + fail(); + } + }); Thread.sleep(50); // The monitor should rethrow the exception when it stops try { diff --git a/test/net/sf/briar/plugins/file/UnixRemovableDriveMonitorTest.java b/test/net/sf/briar/plugins/file/UnixRemovableDriveMonitorTest.java index fa1fb4e5eab598ffcd51f7d632cfb3b9f33f1e9f..d03570621f40de413c2e6dd40b86cbc3e13e5ad8 100644 --- a/test/net/sf/briar/plugins/file/UnixRemovableDriveMonitorTest.java +++ b/test/net/sf/briar/plugins/file/UnixRemovableDriveMonitorTest.java @@ -26,19 +26,23 @@ public class UnixRemovableDriveMonitorTest extends BriarTestCase { @Test public void testNonexistentDir() throws Exception { - if(!OsUtils.isLinux() || OsUtils.isMacLeopardOrNewer()) { + if(!(OsUtils.isLinux() || OsUtils.isMacLeopardOrNewer())) { System.err.println("Warning: Skipping test"); return; } File doesNotExist = new File(testDir, "doesNotExist"); RemovableDriveMonitor monitor = createMonitor(doesNotExist); - monitor.start(null); + monitor.start(new Callback() { + public void driveInserted(File root) { + fail(); + } + }); monitor.stop(); } @Test public void testOneCallbackPerFile() throws Exception { - if(!OsUtils.isLinux() || OsUtils.isMacLeopardOrNewer()) { + if(!(OsUtils.isLinux() || OsUtils.isMacLeopardOrNewer())) { System.err.println("Warning: Skipping test"); return; }