handle typed nil pointers in Encode method (apache/dubbo-go#2517)

decode_test.go

@@ -81,6 +81,10 @@ func getJavaReply(method, className string) []byte {
 	cmd := exec.Command("java", cmdArgs...)
 	out, err := cmd.Output()
 	if err != nil {
+		if exitErr, ok := err.(*exec.ExitError); ok {
+			log.Printf("Java command exited with status %d: %s", exitErr.ExitCode(), string(out))
+			return out
+		}
 		log.Fatal(cmd.Args, err)
 	}
 	return out

encode.go

@@ -99,6 +99,12 @@ func (e *Encoder) Encode(v interface{}) error {
 		return nil
 	}
 
+	vVal := reflect.ValueOf(v)
+	if vVal.Kind() == reflect.Ptr && vVal.IsNil() {
+		e.buffer = EncNull(e.buffer)
+		return nil
+	}
+
 	switch val := v.(type) {
 	case nil:
 		e.buffer = EncNull(e.buffer)

encode_test.go

@@ -92,6 +92,36 @@ func testSimpleEncode(t *testing.T, v interface{}) {
 	assert.Nil(t, err)
 }
 
+func TestEncodeTypedNilPointer(t *testing.T) {
+	encoder := NewEncoder()
+	var val *int32 = nil
+	err := encoder.Encode(val)
+	assert.Nil(t, err)
+
+	expected := []byte{'N'}
+	assertEqual(expected, encoder.Buffer(), t)
+}
+
+func TestEncodeUntypedNil(t *testing.T) {
+	encoder := NewEncoder()
+	err := encoder.Encode(nil)
+	assert.Nil(t, err)
+
+	expected := []byte{'N'}
+	assertEqual(expected, encoder.Buffer(), t)
+}
+
+func TestEncodeNonNilPointer(t *testing.T) {
+	encoder := NewEncoder()
+	val := int32(42)
+	ptr := &val
+	err := encoder.Encode(ptr)
+	assert.Nil(t, err)
+
+	expected := []byte{0xba}
+	assertEqual(expected, encoder.Buffer(), t)
+}
+
 type BenchData struct {
 	name string
 }

test_hessian/src/main/java/test/TestThrowable.java

@@ -410,7 +410,7 @@ public static Object throw_AnnotationTypeMismatchException() {
   }
 
   public static Object throw_UserDefinedException() {
-      return new UserDefindException("throw UserDefinedException");
+      return new UserDefinedException("throw UserDefinedException");
   }
 
 }

test_hessian/src/main/java/test/UserDefindException.java → test_hessian/src/main/java/test/UserDefinedException.java

@@ -17,8 +17,8 @@
 
 package test;
 
-public class UserDefindException extends RuntimeException{
-    public UserDefindException(String dd){
+public class UserDefinedException extends RuntimeException{
+    public UserDefinedException(String dd){
         super(dd);
     }
 }