skip the test: testOnNext_writeValidation

risingwavelabs · May 14, 2024 · 5047397 · 5047397
1 parent a555559
commit 5047397
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 4 deletions.
diff --git a/.github/workflows/connector-node-integration.yml b/.github/workflows/connector-node-integration.yml
@@ -2,11 +2,11 @@ name: Connector Node Integration Tests
 
 on:
   push:
-    branches: [main]
+    branches: [ main ]
   pull_request:
-    branches: [main]
+    branches: [ main ]
   merge_group:
-    types: [checks_requested]
+    types: [ checks_requested ]
 
 jobs:
   build:
@@ -42,4 +42,8 @@ jobs:
           echo "--- build connector node"
           cd ${RISINGWAVE_ROOT}/java
           # run unit test
-          mvn --batch-mode --update-snapshots clean package -Dno-build-rust
+          # WARN: `testOnNext_writeValidation` is skipped because it relies on Rust code to decode message, 
+          #       while we don't build Rust code (`-Dno-build-rust`) here to save time
+          mvn --batch-mode --update-snapshots clean package -Dno-build-rust \
+            '-Dtest=!com.risingwave.connector.sink.SinkStreamObserverTest#testOnNext_writeValidation' \
+            -Dsurefire.failIfNoSpecifiedTests=false
diff --git a/...ve-connector-test/src/test/java/com/risingwave/connector/sink/SinkStreamObserverTest.java b/...ve-connector-test/src/test/java/com/risingwave/connector/sink/SinkStreamObserverTest.java
@@ -154,6 +154,8 @@ public void testOnNext_startEpochValidation() {
         sinkWriterStreamObserver.onNext(firstSync);
     }
 
+    // WARN! This test is skipped in CI pipeline see
+    // `.github/workflows/connector-node-integration.yml`
     @Test
     public void testOnNext_writeValidation() {
         SinkWriterStreamObserver sinkWriterStreamObserver;