address review comments

Author: czeslavo

assert/assertions.go

@@ -1875,18 +1875,15 @@ func (c *CollectT) FailNow() {
 }
 
 // Reset clears the collected errors.
+// Deprecated: Not used anymore.
 func (c *CollectT) Reset() {
-	c.errors = nil
+	panic("Reset() is deprecated")
 }
 
 // Copy copies the collected errors to the supplied t.
+// Deprecated: Not used anymore.
 func (c *CollectT) Copy(t TestingT) {
-	if tt, ok := t.(tHelper); ok {
-		tt.Helper()
-	}
-	for _, err := range c.errors {
-		t.Errorf("%v", err)
-	}
+	panic("Copy() is deprecated")
 }
 
 // EventuallyWithT asserts that given condition will be met in waitFor time,

assert/assertions_test.go

@@ -2787,15 +2787,14 @@ func TestEventuallyWithTTrue(t *testing.T) {
 }
 
 func TestEventuallyWithT_ConcurrencySafe(t *testing.T) {
-	mockT := new(CollectT)
+	mockT := new(testing.T)
 
 	condition := func(collect *CollectT) {
 		True(collect, false)
 	}
 
 	// To trigger race conditions, we run EventuallyWithT with a nanosecond tick.
 	False(t, EventuallyWithT(mockT, condition, 100*time.Millisecond, time.Nanosecond))
-	Len(t, mockT.errors, 2)
 }
 
 func TestNeverFalse(t *testing.T) {