Initialize/setup mock via a constructor param about mockery HOT 3 CLOSED

Why not just have fooExp be fooExp: func(t *testing.T) *mocksFoo that returns the instantiated mock with expectations already added?

foo := tt.fooExp(t)

from mockery.

It's a pretty good solution that I did not think of (thanks for the suggestion, I'll probably use that if this doesn't get added!). I like that it keeps the mock initialization and setup together.

The only disadvantage I see is that it's a bit more verbose (2 more lines per expect function so if the table has >10 tests it can pile up) but I can live with that:

fooExp: func(m *mocks.Foo) {
    m.EXPECT().DoSomething().Return(nil)
}

// vs

fooExp: func(t *testing.T) *mocks.Foo {
    m := mocks.NewFoo(t)
    m.EXPECT().DoSomething().Return(nil)
    return m
}

from mockery.

That's great. FWIW add a nil-check in the t.Run around tt.fooExp(foo) is not a bad option in my opinion. This would probably be my preferred solution. But yeah, if there is an easy way around your problem, I prefer not to add to the mock objects.

from mockery.