Add beforeRecordingSaveWaitTime

Author: AB-xdev

CHANGELOG.md

@@ -1,3 +1,9 @@
+# 1.2.4
+* New option ``beforeRecordingSaveWaitTime`` in ``BrowserWebDriverContainer``
+  * If not ``null``: Waits the amount of specified time before saving the recording
+  * This way no frames that may show the problem are accidentally lost
+  * Default value is set to ``70ms`` which is 1 full frame when recording at the default 15 FPS
+
 # 1.2.3
 * Make some constants externally accessible
 * Provide chromium image (Chrome doesn't work on ARM64)

testcontainers-selenium/src/main/java/software/xdev/testcontainers/selenium/containers/browser/BrowserWebDriverContainer.java

@@ -114,6 +114,8 @@ public class BrowserWebDriverContainer<SELF extends BrowserWebDriverContainer<SE
 	protected Path recordingDirectory;
 	protected TestRecordingFileNameFactory testRecordingFileNameFactory = new DefaultTestRecordingFileNameFactory();
 	protected Duration recordingSaveTimeout = Duration.ofMinutes(3);
+	// Ensure that the current frame will be fully recorded (default record FPS = 15 -> 67ms per Frame)
+	protected Duration beforeRecordingSaveWaitTime = Duration.ofMillis(70);
 
 	public BrowserWebDriverContainer(final String dockerImageName)
 	{
@@ -211,6 +213,13 @@ public SELF withRecordingSaveTimeout(final Duration recordingSaveTimeout)
 		this.recordingSaveTimeout = recordingSaveTimeout;
 		return this.self();
 	}
+	
+	public SELF withBeforeRecordingSaveWaitTime(final Duration beforeRecordingSaveWaitTime)
+	{
+		this.beforeRecordingSaveWaitTime = beforeRecordingSaveWaitTime;
+		return this.self();
+	}
+	
 	// endregion
 
 	// endregion
@@ -453,6 +462,18 @@ protected void retainRecordingIfNeeded(final String testName, final boolean succ
 			default -> false;
 		})
 		{
+			if(this.beforeRecordingSaveWaitTime != null)
+			{
+				try
+				{
+					Thread.sleep(this.beforeRecordingSaveWaitTime.toMillis());
+				}
+				catch(final InterruptedException e)
+				{
+					Thread.currentThread().interrupt();
+					throw new IllegalStateException("Got interrupted", e);
+				}
+			}
 			try
 			{
 				final Path recording = Timeouts.getWithTimeout(