Skip to content

Commit 2b8392c

Browse files
committed
chore: tag a smoke subset and run only those in test_docker
Mirrors the microsoft/playwright tests-docker.yml setup: the Docker workflow runs only the @smoke subset (Docker compatibility check), leaving the full suite for the regular CI matrix on native runners. Five representative test classes get a class-level @tag("smoke"), giving ~5 chromium launches when surefire is invoked with -Dgroups=smoke: - TestBrowser1 (7 tests) - TestBrowserContextBasic (22 tests) - TestPageBasic (33 tests) - TestLocatorClick (5 tests) - TestSelectorsCss (20 tests) create_project_and_run_tests.sh forwards extra args to mvn so callers can pass -Dgroups=smoke. test_docker.yml runs only the smoke subset.
1 parent 1d23791 commit 2b8392c

7 files changed

Lines changed: 13 additions & 3 deletions

File tree

.github/workflows/test_docker.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -64,9 +64,9 @@ jobs:
6464
chown -R pwuser /home/pwuser/.m2
6565
'
6666
67-
- name: Run test in container
67+
- name: Run smoke tests in container
6868
run: |
69-
docker exec "$CONTAINER_ID" /home/pwuser/playwright/tools/test-local-installation/create_project_and_run_tests.sh
69+
docker exec "$CONTAINER_ID" /home/pwuser/playwright/tools/test-local-installation/create_project_and_run_tests.sh -Dgroups=smoke
7070
7171
- name: Test ClassLoader
7272
run: |

playwright/src/test/java/com/microsoft/playwright/TestBrowser1.java

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -20,6 +20,7 @@
2020
import com.google.gson.JsonObject;
2121
import com.microsoft.playwright.junit.FixtureTest;
2222
import com.microsoft.playwright.junit.UsePlaywright;
23+
import org.junit.jupiter.api.Tag;
2324
import org.junit.jupiter.api.Test;
2425
import org.junit.jupiter.api.condition.EnabledIf;
2526

@@ -30,6 +31,7 @@
3031

3132
@FixtureTest
3233
@UsePlaywright(TestOptionsFactories.BasicOptionsFactory.class)
34+
@Tag("smoke")
3335
public class TestBrowser1 {
3436

3537
@Test

playwright/src/test/java/com/microsoft/playwright/TestBrowserContextBasic.java

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -18,6 +18,7 @@
1818

1919
import com.microsoft.playwright.junit.FixtureTest;
2020
import com.microsoft.playwright.junit.UsePlaywright;
21+
import org.junit.jupiter.api.Tag;
2122
import org.junit.jupiter.api.Test;
2223

2324
import java.io.OutputStreamWriter;
@@ -32,6 +33,7 @@
3233

3334
@FixtureTest
3435
@UsePlaywright(TestOptionsFactories.BasicOptionsFactory.class)
36+
@Tag("smoke")
3537
public class TestBrowserContextBasic {
3638
@Test
3739
void shouldCreateNewContext(Browser browser) {

playwright/src/test/java/com/microsoft/playwright/TestLocatorClick.java

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -17,12 +17,14 @@
1717
package com.microsoft.playwright;
1818

1919
import com.microsoft.playwright.options.KeyboardModifier;
20+
import org.junit.jupiter.api.Tag;
2021
import org.junit.jupiter.api.Test;
2122

2223
import static com.microsoft.playwright.assertions.PlaywrightAssertions.assertThat;
2324
import static java.util.Arrays.asList;
2425
import static org.junit.jupiter.api.Assertions.assertEquals;
2526

27+
@Tag("smoke")
2628
public class TestLocatorClick extends TestBase {
2729

2830
@Test

playwright/src/test/java/com/microsoft/playwright/TestPageBasic.java

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,7 @@
1616

1717
package com.microsoft.playwright;
1818

19+
import org.junit.jupiter.api.Tag;
1920
import org.junit.jupiter.api.Test;
2021
import org.junit.jupiter.api.condition.DisabledIf;
2122

@@ -30,6 +31,7 @@
3031
import static java.util.Arrays.asList;
3132
import static org.junit.jupiter.api.Assertions.*;
3233

34+
@Tag("smoke")
3335
public class TestPageBasic extends TestBase {
3436

3537
@Test

playwright/src/test/java/com/microsoft/playwright/TestSelectorsCss.java

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,7 @@
1616

1717
package com.microsoft.playwright;
1818

19+
import org.junit.jupiter.api.Tag;
1920
import org.junit.jupiter.api.Test;
2021

2122
import java.util.ArrayList;
@@ -24,6 +25,7 @@
2425

2526
import static org.junit.jupiter.api.Assertions.*;
2627

28+
@Tag("smoke")
2729
public class TestSelectorsCss extends TestBase {
2830

2931
@Test

tools/test-local-installation/create_project_and_run_tests.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,6 @@ cp -R ../../driver-bundle/src/test/ $PROJECT_DIR/src/
1414
cp -R ../../playwright/src/test/ $PROJECT_DIR/src/
1515
cd $PROJECT_DIR
1616

17-
mvn test --no-transfer-progress
17+
mvn test --no-transfer-progress "$@"
1818

1919
rm -rf $PROJECT_DIR

0 commit comments

Comments
 (0)