use custom test assert helper

hoanhan101 · Oct 13, 2019 · 63f5f4a · 63f5f4a
1 parent f155f88
commit 63f5f4a
Show file tree

Hide file tree

Showing 23 changed files with 54 additions and 95 deletions.
diff --git a/interviewcake/2nd_largest_item_bst_test.go b/interviewcake/2nd_largest_item_bst_test.go
@@ -33,8 +33,9 @@
 package interviewcake
 
 import (
-	"reflect"
 	"testing"
+
+	"github.com/hoanhan101/algo/common"
 )
 
 func Test2ndLargestItem(t *testing.T) {
@@ -88,9 +89,7 @@ func Test2ndLargestItem(t *testing.T) {
 
 	for _, tt := range tests {
 		result := findSecondLargest(tt.in)
-		if !reflect.DeepEqual(result, tt.expected) {
-			t.Errorf("should be %v instead %v", tt.expected, result)
-		}
+		common.Equal(t, tt.expected, result)
 	}
 }
 

diff --git a/interviewcake/apple_stocks_test.go b/interviewcake/apple_stocks_test.go
@@ -16,7 +16,6 @@
 package interviewcake
 
 import (
-	"reflect"
 	"testing"
 
 	"github.com/hoanhan101/algo/common"
@@ -37,9 +36,7 @@ func TestGetMaxProfit(t *testing.T) {
 
 	for _, tt := range tests {
 		result := getMaxProfit(tt.in)
-		if !reflect.DeepEqual(result, tt.expected) {
-			t.Errorf("should be %v instead %v", tt.expected, result)
-		}
+		common.Equal(t, tt.expected, result)
 	}
 }
 

diff --git a/interviewcake/balanced_binary_tree_test.go b/interviewcake/balanced_binary_tree_test.go
@@ -29,7 +29,6 @@
 package interviewcake
 
 import (
-	"reflect"
 	"testing"
 
 	"github.com/hoanhan101/algo/common"
@@ -83,9 +82,7 @@ func TestIsSuperBalanced(t *testing.T) {
 
 	for _, tt := range tests {
 		result := isSuperBalanced(tt.in)
-		if !reflect.DeepEqual(result, tt.expected) {
-			t.Errorf("should be %v instead %v", tt.expected, result)
-		}
+		common.Equal(t, tt.expected, result)
 	}
 }
 

diff --git a/interviewcake/binary_search_tree_test.go b/interviewcake/binary_search_tree_test.go
@@ -29,8 +29,9 @@ package interviewcake
 
 import (
 	"math"
-	"reflect"
 	"testing"
+
+	"github.com/hoanhan101/algo/common"
 )
 
 func TestIsBinarySearchTree(t *testing.T) {
@@ -78,9 +79,7 @@ func TestIsBinarySearchTree(t *testing.T) {
 
 	for _, tt := range tests {
 		result := isBinarySearchTree(tt.in)
-		if !reflect.DeepEqual(result, tt.expected) {
-			t.Errorf("should be %v instead %v", tt.expected, result)
-		}
+		common.Equal(t, tt.expected, result)
 	}
 }
 

diff --git a/interviewcake/bracket_validator_test.go b/interviewcake/bracket_validator_test.go
@@ -39,7 +39,8 @@ func TestValidateBracket(t *testing.T) {
 	}
 
 	for _, tt := range tests {
-		common.Equal(t, tt.expected, validateBracket(tt.sentence))
+		result := validateBracket(tt.sentence)
+		common.Equal(t, tt.expected, result)
 	}
 }
 

diff --git a/interviewcake/fibonacci_number_test.go b/interviewcake/fibonacci_number_test.go
@@ -16,8 +16,9 @@
 package interviewcake
 
 import (
-	"reflect"
 	"testing"
+
+	"github.com/hoanhan101/algo/common"
 )
 
 func TestFib(t *testing.T) {
@@ -36,9 +37,7 @@ func TestFib(t *testing.T) {
 
 	for _, tt := range tests {
 		result := fib(tt.in)
-		if !reflect.DeepEqual(result, tt.expected) {
-			t.Errorf("should be %v instead %v", tt.expected, result)
-		}
+		common.Equal(t, tt.expected, result)
 	}
 }
 

diff --git a/interviewcake/find_rotation_point_test.go b/interviewcake/find_rotation_point_test.go
@@ -31,8 +31,9 @@
 package interviewcake
 
 import (
-	"reflect"
 	"testing"
+
+	"github.com/hoanhan101/algo/common"
 )
 
 func TestFindRotationPoint(t *testing.T) {
@@ -103,9 +104,7 @@ func TestFindRotationPoint(t *testing.T) {
 
 	for _, tt := range tests {
 		result := findRotationPoint(tt.in)
-		if !reflect.DeepEqual(result, tt.expected) {
-			t.Errorf("should be %v instead %v", tt.expected, result)
-		}
+		common.Equal(t, tt.expected, result)
 	}
 }
 

diff --git a/interviewcake/graph_coloring_test.go b/interviewcake/graph_coloring_test.go
@@ -21,7 +21,6 @@
 package interviewcake
 
 import (
-	"reflect"
 	"testing"
 
 	"github.com/hoanhan101/algo/common"
@@ -132,9 +131,7 @@ func TestColorGraph(t *testing.T) {
 			result = append(result, g.color)
 		}
 
-		if !reflect.DeepEqual(result, tt.expected) {
-			t.Errorf("should be %v instead %v", tt.expected, result)
-		}
+		common.Equal(t, tt.expected, result)
 	}
 }
 

diff --git a/interviewcake/highest_product_of_three_test.go b/interviewcake/highest_product_of_three_test.go
@@ -20,7 +20,6 @@
 package interviewcake
 
 import (
-	"reflect"
 	"testing"
 
 	"github.com/hoanhan101/algo/common"
@@ -38,9 +37,7 @@ func TestHighestProductOfThree(t *testing.T) {
 
 	for _, tt := range tests {
 		result := highestProductOfThree(tt.in)
-		if !reflect.DeepEqual(result, tt.expected) {
-			t.Errorf("should be %v instead %v", tt.expected, result)
-		}
+		common.Equal(t, tt.expected, result)
 	}
 }
 

diff --git a/interviewcake/inflight_test.go b/interviewcake/inflight_test.go
@@ -20,8 +20,9 @@
 package interviewcake
 
 import (
-	"reflect"
 	"testing"
+
+	"github.com/hoanhan101/algo/common"
 )
 
 func TestFillFlight(t *testing.T) {
@@ -41,9 +42,7 @@ func TestFillFlight(t *testing.T) {
 
 	for _, tt := range tests {
 		result := fillFlight(tt.in1, tt.in2)
-		if !reflect.DeepEqual(result, tt.expected) {
-			t.Errorf("should be %v instead %v", tt.expected, result)
-		}
+		common.Equal(t, tt.expected, result)
 	}
 }
 

diff --git a/interviewcake/inplace_shuffle_test.go b/interviewcake/inplace_shuffle_test.go
@@ -15,7 +15,6 @@
 package interviewcake
 
 import (
-	"reflect"
 	"testing"
 
 	"github.com/hoanhan101/algo/common"
@@ -33,9 +32,7 @@ func TestInplaceShuffle(t *testing.T) {
 
 	for _, tt := range tests {
 		result := inplaceShuffle(tt.in)
-		if !reflect.DeepEqual(result, tt.expected) {
-			t.Errorf("should be %v instead %v", tt.expected, result)
-		}
+		common.Equal(t, tt.expected, result)
 	}
 }
 

diff --git a/interviewcake/making_change_test.go b/interviewcake/making_change_test.go
@@ -25,8 +25,9 @@
 package interviewcake
 
 import (
-	"reflect"
 	"testing"
+
+	"github.com/hoanhan101/algo/common"
 )
 
 func TestMakeChange(t *testing.T) {
@@ -82,14 +83,8 @@ func TestMakeChange(t *testing.T) {
 
 	for _, tt := range tests {
 		r1, r2 := makeChange(tt.in1, tt.in2)
-
-		if !reflect.DeepEqual(r1, tt.expected1) {
-			t.Errorf("should be %v instead %v", tt.expected1, r1)
-		}
-
-		if !reflect.DeepEqual(r2, tt.expected2) {
-			t.Errorf("should be %v instead %v", tt.expected2, r2)
-		}
+		common.Equal(t, tt.expected1, r1)
+		common.Equal(t, tt.expected2, r2)
 	}
 }
 

diff --git a/interviewcake/merge_meetings_test.go b/interviewcake/merge_meetings_test.go
@@ -21,7 +21,6 @@
 package interviewcake
 
 import (
-	"reflect"
 	"sort"
 	"testing"
 
@@ -44,9 +43,7 @@ func TestMergeMeetings(t *testing.T) {
 
 	for _, tt := range tests {
 		result := mergeMeetings(tt.in)
-		if !reflect.DeepEqual(result, tt.expected) {
-			t.Errorf("should be %v instead %v", tt.expected, result)
-		}
+		common.Equal(t, tt.expected, result)
 	}
 }
 

diff --git a/interviewcake/merge_sorted_arrays_test.go b/interviewcake/merge_sorted_arrays_test.go
@@ -15,8 +15,9 @@
 package interviewcake
 
 import (
-	"reflect"
 	"testing"
+
+	"github.com/hoanhan101/algo/common"
 )
 
 func TestMergeSortedArray(t *testing.T) {
@@ -49,9 +50,7 @@ func TestMergeSortedArray(t *testing.T) {
 
 	for _, tt := range tests {
 		result := mergeSortedArray(tt.in1, tt.in2)
-		if !reflect.DeepEqual(result, tt.expected) {
-			t.Errorf("should be %v instead %v", tt.expected, result)
-		}
+		common.Equal(t, tt.expected, result)
 	}
 }
 

diff --git a/interviewcake/permutation_palindrome_test.go b/interviewcake/permutation_palindrome_test.go
@@ -23,8 +23,9 @@
 package interviewcake
 
 import (
-	"reflect"
 	"testing"
+
+	"github.com/hoanhan101/algo/common"
 )
 
 func TestHasPalindromePermutation(t *testing.T) {
@@ -44,9 +45,7 @@ func TestHasPalindromePermutation(t *testing.T) {
 
 	for _, tt := range tests {
 		result := hasPalindromePermutation(tt.in)
-		if !reflect.DeepEqual(result, tt.expected) {
-			t.Errorf("should be %v instead %v", tt.expected, result)
-		}
+		common.Equal(t, tt.expected, result)
 	}
 }
 

diff --git a/interviewcake/product_of_others_test.go b/interviewcake/product_of_others_test.go
@@ -18,8 +18,9 @@
 package interviewcake
 
 import (
-	"reflect"
 	"testing"
+
+	"github.com/hoanhan101/algo/common"
 )
 
 func TestGetProductOfOthers(t *testing.T) {
@@ -35,9 +36,7 @@ func TestGetProductOfOthers(t *testing.T) {
 
 	for _, tt := range tests {
 		result := getProductOfOthers(tt.in)
-		if !reflect.DeepEqual(result, tt.expected) {
-			t.Errorf("should be %v instead %v", tt.expected, result)
-		}
+		common.Equal(t, tt.expected, result)
 	}
 }
 

diff --git a/interviewcake/recursive_string_permutation_test.go b/interviewcake/recursive_string_permutation_test.go
@@ -17,9 +17,10 @@
 package interviewcake
 
 import (
-	"reflect"
 	"strings"
 	"testing"
+
+	"github.com/hoanhan101/algo/common"
 )
 
 func TestPermuteString(t *testing.T) {
@@ -35,9 +36,7 @@ func TestPermuteString(t *testing.T) {
 
 	for _, tt := range tests {
 		result := permuteString(tt.in)
-		if !reflect.DeepEqual(result, tt.expected) {
-			t.Errorf("should be %v instead %v", tt.expected, result)
-		}
+		common.Equal(t, tt.expected, result)
 	}
 }
 

diff --git a/interviewcake/reverse_string_test.go b/interviewcake/reverse_string_test.go
@@ -15,8 +15,9 @@
 package interviewcake
 
 import (
-	"reflect"
 	"testing"
+
+	"github.com/hoanhan101/algo/common"
 )
 
 func TestReverseString(t *testing.T) {
@@ -33,9 +34,7 @@ func TestReverseString(t *testing.T) {
 
 	for _, tt := range tests {
 		result := reverseString(tt.in)
-		if !reflect.DeepEqual(result, tt.expected) {
-			t.Errorf("should be %v instead %v", tt.expected, result)
-		}
+		common.Equal(t, tt.expected, result)
 	}
 }
 

diff --git a/interviewcake/reverse_word_test.go b/interviewcake/reverse_word_test.go
@@ -17,8 +17,9 @@
 package interviewcake
 
 import (
-	"reflect"
 	"testing"
+
+	"github.com/hoanhan101/algo/common"
 )
 
 func TestReverseWord(t *testing.T) {
@@ -34,9 +35,7 @@ func TestReverseWord(t *testing.T) {
 
 	for _, tt := range tests {
 		result := reverseWord(tt.in)
-		if !reflect.DeepEqual(result, tt.expected) {
-			t.Errorf("should be %v instead %v", tt.expected, result)
-		}
+		common.Equal(t, tt.expected, result)
 	}
 }