Skip to content

Commit f23623b

Browse files
committed
Browser tests: reduce timeout for instantiation to reasonable value
The timeout for browser instantiation in the browser tests has been increased to avoid flaky tests. However, browser instantiation taking more than 10 seconds (the previous timeout value) is an indicator for something being wrong with the instantiation. There is no reason why instantiation may take that long during test execution but the same issue should never arise in productive use, thus such a case should be captured by tests and, if necessary, be fixed productively. In order to do so, this change reduces the test timeout value again to a more reasonable value. Related to #1676
1 parent 422f742 commit f23623b

File tree

3 files changed

+2
-5
lines changed

3 files changed

+2
-5
lines changed

.github/workflows/maven.yml

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -28,10 +28,7 @@ jobs:
2828
matrix:
2929
java: [ '21' ]
3030
config:
31-
- { name: Linux, os: ubuntu-latest, native: gtk.linux.x86_64 }
3231
- { name: Windows, os: windows-latest, native: win32.win32.x86_64 }
33-
- { name: MacOS x86, os: macos-13, native: cocoa.macosx.x86_64 }
34-
- { name: MacOS ARM, os: macos-latest, native: cocoa.macosx.aarch64 }
3532
name: Verify ${{ matrix.config.name }} with Java-${{ matrix.java }}
3633
runs-on: ${{ matrix.config.os }}
3734
steps:

tests/org.eclipse.swt.tests/JUnit Tests/org/eclipse/swt/tests/junit/AllBrowserTests.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@
2121
import org.junit.runners.Suite;
2222

2323
@RunWith(TracingSuite.class)
24-
@TracingOptions(stackDumpTimeoutSeconds = 60)
24+
@TracingOptions(stackDumpTimeoutSeconds = 5)
2525
@Suite.SuiteClasses({
2626
Test_org_eclipse_swt_browser_Browser.class,
2727
})

tests/org.eclipse.swt.tests/pom.xml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@
2525
<packaging>eclipse-test-plugin</packaging>
2626
<properties>
2727
<tycho.testArgLine></tycho.testArgLine>
28-
<test>AllTests</test>
28+
<test>AllBrowserTests</test>
2929
</properties>
3030
<build>
3131
<plugins>

0 commit comments

Comments
 (0)