[QUESTION] Increase test coverage for handler type about go-smtp-mock

[benjamin-rood]

@bestwebua Can you assign this to me? I will have time on this weekend to do it.

[bestwebua]

Good catch, @benjamin-rood 👍 Seems like good point to add one more file, handler_test.go with one test for handler#clearError() method:

go-smtp-mock/handler.go

Lines 13 to 15 in 34c709d

	func (handler *handler) clearError() {
	handler.session.clearError()
	}

Just one straightforward test with sessionMock about checking of clearError() calling. Also we have indirect calls of this one therefore we have 100% coverage for this case.

[bestwebua]

Seems we have test for this case:

go-smtp-mock/handler_test.go

Lines 10 to 18 in 1b2ab0e

	func TestHandlerClearError(t *testing.T) {
	t.Run("erases session error", func(t *testing.T) {
	session := &session{err: errors.New("some error")}
	handler := &handler{configuration: createConfiguration(), session: session}
	handler.clearError()
	assert.Nil(t, session.err)
	})
	}

@benjamin-rood We don't need tests for this method, it's my bad, sorry. But if you want, you can try to fix this issue: #135