ambiser
/
openvidu
mirror of https://github.com/OpenVidu/openvidu.git
8
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

openvidu-test-e2e: protect dispose method @AfterEach from possible NullPointer

pull/664/head
pabloFuente 2021-11-04 14:10:32 +01:00
parent 3baa72396f
commit 47792608ce
1 changed files with 16 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

28
openvidu-test-e2e/src/test/java/io/openvidu/test/e2e/AbstractOpenViduTestAppE2eTest.java
View File

@ -392,37 +392,41 @@ public class AbstractOpenViduTestAppE2eTest {
@AfterEach
protected void dispose() {
// Close all remaining OpenVidu sessions
this.closeAllSessions(OV);
// Remove all recordings
if (isRecordingTest) {
deleteAllRecordings(OV);
isRecordingTest = false;
}
// Reset Media Server
if (isKurentoRestartTest) {
this.stopMediaServer(false);
this.startMediaServer(true);
isKurentoRestartTest = false;
}
// Dispose all browsers
Iterator<MyUser> it = users.iterator();
while (it.hasNext()) {
MyUser u = it.next();
u.dispose();
it.remove();
}
if (chrome.isRunning()) {
chrome.stop();
}
if (firefox.isRunning()) {
firefox.stop();
}
if (opera.isRunning()) {
opera.stop();
}
if (edge.isRunning()) {
edge.stop();
}
// Stop and remove all browser containers if necessary
stopContainerIfPossible(chrome);
stopContainerIfPossible(firefox);
stopContainerIfPossible(opera);
stopContainerIfPossible(edge);
// Reset REST client
OV = new OpenVidu(OPENVIDU_URL, OPENVIDU_SECRET);
}
private void stopContainerIfPossible(GenericContainer<?> container) {
if (container != null && container.isRunning()) {
container.stop();
}
}
protected void closeAllSessions(OpenVidu client) {
try {
client.fetch();