http-client-java, update test to fill exact param (microsoft#4770)

for Azure/cadl-ranch#755, as some tests validate the exact value of some parameter/property. for the 2 file with only format change, I guess the reason is Alan enabled spotless on emitter Java code (so `mvn package` would automatically format the code)
swatkatz · Nov 5, 2024 · 9a5bedd · 9a5bedd
1 parent e3694dd
commit 9a5bedd
Show file tree

Hide file tree

Showing 4 changed files with 19 additions and 19 deletions.
diff --git a/...nerator-test/src/test/java/com/_specs_/azure/clientgenerator/core/access/AccessTests.java b/...nerator-test/src/test/java/com/_specs_/azure/clientgenerator/core/access/AccessTests.java
@@ -20,18 +20,18 @@ public class AccessTests {
 
     @Test
     public void test() {
-        publicClient.publicDecoratorInPublic("");
-        publicClient.noDecoratorInPublic("");
+        publicClient.publicDecoratorInPublic("sample");
+        publicClient.noDecoratorInPublic("sample");
 
-        internalClient.internalDecoratorInInternal("");
-        internalClient.noDecoratorInInternal("");
-        internalClient.publicDecoratorInInternal("");
+        internalClient.internalDecoratorInInternal("sample");
+        internalClient.noDecoratorInInternal("sample");
+        internalClient.publicDecoratorInInternal("sample");
 
-        sharedClient.publicMethod("");
-        sharedClient.internalWithResponse("", null).getValue().toObject(SharedModel.class);
+        sharedClient.publicMethod("sample");
+        sharedClient.internalWithResponse("sample", null).getValue().toObject(SharedModel.class);
 
-        relativeClient.operation("");
+        relativeClient.operation("Madge");
         AbstractModel abstractModel = relativeClient.discriminator("real");
-        Assertions.assertTrue(abstractModel instanceof RealModel);
+        Assertions.assertInstanceOf(RealModel.class, abstractModel);
     }
 }
diff --git a/...rator/http-client-generator-test/src/test/java/com/authentication/apikey/ApiKeyTests.java b/...rator/http-client-generator-test/src/test/java/com/authentication/apikey/ApiKeyTests.java
@@ -25,7 +25,7 @@ public void testValid() {
     public void testInvalid() {
         ApiKeyClient client = new ApiKeyClientBuilder()
             // AzureKeyCredentialPolicy from core requires HTTPS
-            .addPolicy(new AzureKeyCredentialPolicy("x-ms-api-key", new AzureKeyCredential("valid-key")))
+            .addPolicy(new AzureKeyCredentialPolicy("x-ms-api-key", new AzureKeyCredential("invalid-key")))
             .buildClient();
 
         // assert HttpResponseException

diff --git a/...azure/resourcemanager/models/commontypes/managedidentity/ManagedIdentityManagerTests.java b/...azure/resourcemanager/models/commontypes/managedidentity/ManagedIdentityManagerTests.java
@@ -45,8 +45,9 @@ public void testManagedIdentityManager() {
         Map<String, UserAssignedIdentity> userAssignedIdentityMap = new HashMap<>();
         userAssignedIdentityMap.put(USER_ASSIGNED_IDENTITIES_KEY, new UserAssignedIdentity());
         resource.update()
-            .withIdentity(new ManagedServiceIdentity().withType(ManagedServiceIdentityType.SYSTEM_ASSIGNED_USER_ASSIGNED)
-                .withUserAssignedIdentities(userAssignedIdentityMap))
+            .withIdentity(
+                new ManagedServiceIdentity().withType(ManagedServiceIdentityType.SYSTEM_ASSIGNED_USER_ASSIGNED)
+                    .withUserAssignedIdentities(userAssignedIdentityMap))
             .apply();
         Assertions.assertEquals(ManagedServiceIdentityType.SYSTEM_ASSIGNED_USER_ASSIGNED, resource.identity().type());
         Assertions.assertNotNull(resource.identity().principalId());

diff --git a/...client-java/generator/http-client-generator-test/src/test/java/com/routes/RouteTests.java b/...client-java/generator/http-client-generator-test/src/test/java/com/routes/RouteTests.java
@@ -3,11 +3,10 @@
 
 package com.routes;
 
-import java.util.List;
-import java.util.Set;
-
 import com.azure.core.http.policy.HttpLogDetailLevel;
 import com.azure.core.http.policy.HttpLogOptions;
+import java.util.List;
+import java.util.Set;
 import org.junit.jupiter.api.Test;
 
 public class RouteTests {
@@ -79,10 +78,10 @@ public void testQueryExpansionExplode() {
 
     @Test
     public void buildQueryParametersQueryContinuationExplode() {
-        var client = new RoutesClientBuilder().httpLogOptions(new HttpLogOptions()
-                .setLogLevel(HttpLogDetailLevel.BODY_AND_HEADERS)
-                .setAllowedQueryParamNames(Set.of("fixed", "param"))
-        ).buildQueryParametersQueryContinuationExplodeClient();
+        var client = new RoutesClientBuilder()
+            .httpLogOptions(new HttpLogOptions().setLogLevel(HttpLogDetailLevel.BODY_AND_HEADERS)
+                .setAllowedQueryParamNames(Set.of("fixed", "param")))
+            .buildQueryParametersQueryContinuationExplodeClient();
 
         client.primitive("a");