test: add some tests for MustSet function

creasty · Aug 14, 2020 · ea4c44f · ea4c44f
1 parent c2889d7
commit ea4c44f
Showing 1 changed file with 42 additions and 14 deletions.
diff --git a/defaults_test.go b/defaults_test.go
@@ -143,21 +143,49 @@ type Emmbeded struct {
 }
 
 func TestMustSet(t *testing.T) {
-	sample := &Sample{
-		NonInitialString:            "string",
-		NonInitialSlice:             []int{1, 2, 3},
-		NonInitialStruct:            Struct{Foo: 123},
-		NonInitialStructPtr:         &Struct{Foo: 123},
-		DeepSliceOfStructsWithNoTag: [][][]Struct{{{{Foo: 123}}}},
-	}
 
-	MustSet(sample)
-	go func() {
-		if err := recover(); err != nil {
-			t.Fatalf("it should not panic error: %v", err)
-		}
-	}()
-	t.Log("it works.")
+	t.Run("right way", func(t *testing.T) {
+		defer func() {
+			if err := recover(); err != nil {
+				t.Fatalf("it should not panic error: %v", err)
+			}
+		}()
+		sample := &Sample{
+			NonInitialString:            "string",
+			NonInitialSlice:             []int{1, 2, 3},
+			NonInitialStruct:            Struct{Foo: 123},
+			NonInitialStructPtr:         &Struct{Foo: 123},
+			DeepSliceOfStructsWithNoTag: [][][]Struct{{{{Foo: 123}}}},
+		}
+		MustSet(sample)
+	})
+
+	t.Run("not struct", func(t *testing.T) {
+		defer func() {
+			if err := recover(); err != nil {
+				t.Logf("panic error: %v", err)
+			}
+		}()
+		var a int
+		MustSet(&a)
+	})
+
+	t.Run("not pointer", func(t *testing.T) {
+		defer func() {
+			if err := recover(); err != nil {
+				t.Logf("panic error: %v", err)
+			}
+		}()
+		sample := Sample{
+			NonInitialString:            "string",
+			NonInitialSlice:             []int{1, 2, 3},
+			NonInitialStruct:            Struct{Foo: 123},
+			NonInitialStructPtr:         &Struct{Foo: 123},
+			DeepSliceOfStructsWithNoTag: [][][]Struct{{{{Foo: 123}}}},
+		}
+		MustSet(sample)
+	})
+
 }
 
 func TestInit(t *testing.T) {