Merge pull request #2025 from web3j/testWrappersFix

Fix for test wrappers generation

Author: gtebrean

CHANGELOG.md

@@ -7,7 +7,7 @@ See [Conventional Commits](https://conventionalcommits.org) for commit guideline
 
 ### Bug Fixes
 
-* 
+* Fix for test wrappers generation [#2025](https://github.com/web3j/web3j/pull/2025)
 
 ### Features
 

codegen/src/main/java/org/web3j/codegen/SolidityFunctionWrapper.java

@@ -672,7 +672,7 @@ private Set<String> getDuplicateFunctionNames(List<AbiDefinition> functionDefini
     private static MethodSpec buildGetDeploymentBinaryMethod() {
         MethodSpec.Builder toReturn =
                 MethodSpec.methodBuilder("getDeploymentBinary")
-                        .addModifiers(Modifier.PUBLIC, Modifier.STATIC)
+                        .addModifiers(Modifier.PRIVATE, Modifier.STATIC)
                         .returns(ClassName.get(String.class));
 
         CodeBlock codeBlock =

codegen/src/main/java/org/web3j/codegen/unit/gen/MethodFilter.java

@@ -13,6 +13,7 @@
 package org.web3j.codegen.unit.gen;
 
 import java.lang.reflect.Method;
+import java.lang.reflect.Modifier;
 import java.util.ArrayList;
 import java.util.Arrays;
 import java.util.HashMap;
@@ -61,9 +62,11 @@ public static List<MethodSpec> generateMethodSpecsForEachTest(Class theContract)
                 .forEach(
                         method -> {
                             String uniqueName = getUniqueName(method, methodNameCountMap);
-                            listOfMethodSpecs.add(
-                                    new MethodParser(method, theContract, uniqueName)
-                                            .getMethodSpec());
+                            if (!Modifier.isPrivate(method.getModifiers())) {
+                                listOfMethodSpecs.add(
+                                        new MethodParser(method, theContract, uniqueName)
+                                                .getMethodSpec());
+                            }
                         });
 
         return listOfMethodSpecs;
@@ -76,8 +79,11 @@ public static List<FunSpec> generateFunctionSpecsForEachTest(Class theContract)
                 .forEach(
                         method -> {
                             String uniqueName = getUniqueName(method, functionNameCountMap);
-                            listOfFunSpecs.add(
-                                    new FunParser(method, theContract, uniqueName).getFunSpec());
+                            if (!Modifier.isPrivate(method.getModifiers())) {
+                                listOfFunSpecs.add(
+                                        new FunParser(method, theContract, uniqueName)
+                                                .getFunSpec());
+                            }
                         });
 
         return listOfFunSpecs;

codegen/src/test/java/org/web3j/codegen/unit/gen/java/MethodParserTest.java

@@ -73,4 +73,22 @@ public void testGeneratedDuplicateGreetingMethods() {
                 greetMethodSpecs.stream().anyMatch(methodSpec -> methodSpec.name.equals("greet1")));
         assertEquals(2, greetMethodSpecs.size());
     }
+
+    @Test
+    public void testGetDeploymentBinaryMethodNotGenerated() {
+        List<MethodSpec> allMethodSpecs =
+                MethodFilter.generateMethodSpecsForEachTest(greeterContractClass);
+
+        // Filter all MethodSpecs for those related to "getDeploymentBinary" method
+        List<MethodSpec> getDeploymentBinaryMethodSpecs =
+                allMethodSpecs.stream()
+                        .filter(methodSpec -> methodSpec.name.startsWith("getDeploymentBinary"))
+                        .collect(Collectors.toList());
+
+        // Ensure no MethodSpecs were generated for getDeploymentBinary method
+        assertEquals(
+                0,
+                getDeploymentBinaryMethodSpecs.size(),
+                "MethodSpec list should not contain getDeploymentBinary method");
+    }
 }

codegen/src/test/java/org/web3j/codegen/unit/gen/kotlin/FunParserTest.java

@@ -78,4 +78,22 @@ public void testGeneratedDuplicateGreetingMethods() {
                         .anyMatch(methodSpec -> methodSpec.getName().equals("greet1")));
         assertEquals(2, greetFunSpecs.size());
     }
+
+    @Test
+    public void testGetDeploymentBinaryMethodNotGenerated() {
+        List<FunSpec> allMethodSpecs =
+                MethodFilter.generateFunctionSpecsForEachTest(greeterContractClass);
+
+        // Filter all FunSpecs for those related to "getDeploymentBinary" method
+        List<FunSpec> getDeploymentBinaryFunSpecs =
+                allMethodSpecs.stream()
+                        .filter(funSpec -> funSpec.getName().startsWith("getDeploymentBinary"))
+                        .collect(Collectors.toList());
+
+        // Ensure no MethodSpecs were generated for getDeploymentBinary method
+        assertEquals(
+                0,
+                getDeploymentBinaryFunSpecs.size(),
+                "MethodSpec list should not contain getDeploymentBinary function");
+    }
 }

codegen/src/test/resources/solidity/metacoin/build/java/MetaCoin.java

@@ -166,7 +166,7 @@ public static void linkLibraries(List<Contract.LinkReference> references) {
         librariesLinkedBinary = linkBinaryWithReferences(BINARY, references);
     }
 
-    public static String getDeploymentBinary() {
+    private static String getDeploymentBinary() {
         if (librariesLinkedBinary != null) {
             return librariesLinkedBinary;
         } else {