OpenHAB
/
openhab-core
mirror of https://github.com/openhab/openhab-core.git
1
0
Fork 0
Code Issues Projects Releases Wiki Activity

Improve ScriptFileWatcherTest (#2857) 

Most of the tests add a timeout of 1s to account for small delays in async processing in the AbstractWatchService. This adds some more timeouts for tests which were missing those.

Signed-off-by: Jan N. Klug <github@klug.nrw>
pull/2859/head
J-N-K 2022-03-19 21:08:00 +01:00 committed by GitHub
parent 3d17049d52
commit 79e070c7ac
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
bundles/org.openhab.core.automation.module.script.rulesupport/src/test/java/org/openhab/core/automation/module/script/rulesupport/loader/ScriptFileWatcherTest.java
View File

@ -107,7 +107,7 @@ class ScriptFileWatcherTest {
scriptFileWatcher.processWatchEvent(null, ENTRY_CREATE, p);
verify(scriptEngineManager).createScriptEngine("js", p.toFile().toURI().toString());
verify(scriptEngineManager, timeout(1000)).createScriptEngine("js", p.toFile().toURI().toString());
}
@Test
@ -236,7 +236,7 @@ class ScriptFileWatcherTest {
scheduledTask.getValue().run();
// verify script has now been processed
verify(scriptEngineManager, times(1)).createScriptEngine("js", p.toFile().toURI().toString());
verify(scriptEngineManager, timeout(1000).times(1)).createScriptEngine("js", p.toFile().toURI().toString());
}
@Test
@ -281,7 +281,7 @@ class ScriptFileWatcherTest {
scriptFileWatcher.onDependencyChange(p.toFile().toURI().toString());
verify(scriptEngineManager, times(2)).createScriptEngine("js", p.toFile().toURI().toString());
verify(scriptEngineManager, timeout(1000).times(2)).createScriptEngine("js", p.toFile().toURI().toString());
}
@Test
@ -344,6 +344,6 @@ class ScriptFileWatcherTest {
scriptFileWatcher.processWatchEvent(null, ENTRY_MODIFY, p);
verify(scriptEngineManager).removeEngine(p.toFile().toURI().toString());
verify(scriptEngineManager, times(2)).createScriptEngine("js", p.toFile().toURI().toString());
verify(scriptEngineManager, timeout(1000).times(2)).createScriptEngine("js", p.toFile().toURI().toString());
}
}