Merge pull request go-quicktest#17 from lmb/add-helper-calls

Call TB.Helper from Assert and Check

quicktest.go

@@ -9,6 +9,7 @@ import (
 // Assert checks that the provided argument passes the given check and calls
 // tb.Fatal otherwise, including any Comment arguments in the failure.
 func Assert(t testing.TB, checker Checker, comments ...Comment) bool {
+	t.Helper()
 	return check(t, checkParams{
 		fail:     t.Fatal,
 		checker:  checker,
@@ -19,6 +20,7 @@ func Assert(t testing.TB, checker Checker, comments ...Comment) bool {
 // Check checks that the provided argument passes the given check and calls
 // tb.Error otherwise, including any Comment arguments in the failure.
 func Check(t testing.TB, checker Checker, comments ...Comment) bool {
+	t.Helper()
 	return check(t, checkParams{
 		fail:     t.Error,
 		checker:  checker,

quicktest_test.go

@@ -266,6 +266,20 @@ func TestCAssertCheck(t *testing.T) {
 	}
 }
 
+func TestHelperCalls(t *testing.T) {
+	tt := &testingT{}
+	qt.Assert(tt, qt.IsTrue(false))
+	if tt.helperCalls < 2 {
+		t.Error("Assert doesn't invoke TB.Helper()")
+	}
+
+	tt = &testingT{}
+	qt.Check(tt, qt.IsTrue(false))
+	if tt.helperCalls < 2 {
+		t.Error("Check doesn't invoke TB.Helper()")
+	}
+}
+
 func checkResult(t *testing.T, ok bool, got, want string) {
 	t.Helper()
 	if want != "" {