Ensure we use the same GRPC version everywhere (#1137)

* Ensure we use the same GRPC version everywhere Signed-off-by: Artur Ciocanu <[email protected]> * Fix actors tests assert Signed-off-by: Artur Ciocanu <[email protected]> * Revert Dapr exception asserts Signed-off-by: Artur Ciocanu <[email protected]> * Increase sleep to allow Spring Context to bootstrap Signed-off-by: Artur Ciocanu <[email protected]> * Revert sleep value Signed-off-by: Artur Ciocanu <[email protected]> * Increase the sleep for messaging test Signed-off-by: Artur Ciocanu <[email protected]> * Move sleep before each, to ensure Spring context starts Signed-off-by: Artur Ciocanu <[email protected]> * Add more delays to ensure Spring Controller gets the messages Signed-off-by: Artur Ciocanu <[email protected]> --------- Signed-off-by: Artur Ciocanu <[email protected]> Co-authored-by: Artur Ciocanu <[email protected]>
dapr · Sep 26, 2024 · cedaebc · cedaebc
1 parent 436b7df
commit cedaebc
Show file tree

Hide file tree

Showing 6 changed files with 11 additions and 9 deletions.
diff --git a/pom.xml b/pom.xml
@@ -104,13 +104,13 @@
       <dependency>
         <groupId>io.grpc</groupId>
         <artifactId>grpc-testing</artifactId>
-        <version>1.56.1</version>
+        <version>${grpc.version}</version>
         <scope>test</scope>
       </dependency>
       <dependency>
         <groupId>io.grpc</groupId>
         <artifactId>grpc-inprocess</artifactId>
-        <version>1.59.0</version>
+        <version>${grpc.version}</version>
         <scope>test</scope>
       </dependency>
     </dependencies>

diff --git a/sdk-actors/pom.xml b/sdk-actors/pom.xml
@@ -18,7 +18,6 @@
 
   <properties>
     <maven.deploy.skip>false</maven.deploy.skip>
-    <grpc.version>1.59.0</grpc.version>
   </properties>
 
   <dependencies>

diff --git a/sdk-actors/src/test/java/io/dapr/actors/client/DaprGrpcClientTest.java b/sdk-actors/src/test/java/io/dapr/actors/client/DaprGrpcClientTest.java
@@ -129,7 +129,7 @@ public void invokeException() {
         ExecutionException.class,
         "UNKNOWN",
         "UNKNOWN: ",
-        () -> result.block());
+        result::block);
   }
 
   @Test

diff --git a/sdk-actors/src/test/java/io/dapr/actors/runtime/DaprGrpcClientTest.java b/sdk-actors/src/test/java/io/dapr/actors/runtime/DaprGrpcClientTest.java
@@ -96,7 +96,7 @@ public void getActorStateException() {
         assertThrowsDaprException(
                 ExecutionException.class,
                 "UNKNOWN",
-                "UNKNOWN: ",
+                "UNKNOWN: Application error processing RPC",
                 result::block);
     }
 
@@ -112,7 +112,7 @@ public void saveActorStateTransactionallyException() {
         assertThrowsDaprException(
                 ExecutionException.class,
                 "UNKNOWN",
-                "UNKNOWN: ",
+                "UNKNOWN: Application error processing RPC",
                 result::block);
     }
     @Test

diff --git a/sdk-tests/src/test/java/io/dapr/it/spring/messaging/DaprSpringMessagingIT.java b/sdk-tests/src/test/java/io/dapr/it/spring/messaging/DaprSpringMessagingIT.java
@@ -76,10 +76,15 @@ public class DaprSpringMessagingIT {
   private TestRestController testRestController;
 
   @BeforeAll
-  public static void setup(){
+  public static void beforeAll(){
     org.testcontainers.Testcontainers.exposeHostPorts(8080);
   }
 
+  @BeforeEach
+  public void beforeEach() throws InterruptedException {
+    Thread.sleep(1000);
+  }
+
   @Test
   public void testDaprMessagingTemplate() throws InterruptedException {
     for (int i = 0; i < 10; i++) {
@@ -97,5 +102,4 @@ public void testDaprMessagingTemplate() throws InterruptedException {
 
     assertThat(events.size()).isEqualTo(10);
   }
-
 }
diff --git a/sdk/pom.xml b/sdk/pom.xml
@@ -18,7 +18,6 @@
 
   <properties>
     <maven.deploy.skip>false</maven.deploy.skip>
-    <grpc.version>1.59.0</grpc.version>
     <argLine>
       --add-opens java.base/java.util=ALL-UNNAMED
     </argLine>