Fix empty error, This closes #19 #23

SebastianCzoch · 2018-05-23T19:12:30Z

No description provided.

SebastianCzoch · 2018-05-23T19:15:07Z

mandrill_test.go

+
+	expect(t, len(responses), 0)
+
+	correctResponse := json.Unmarshal([]byte(` `), struct{}{})


json.SyntaxError has only Offset field exported, so this is the easiest way to test this case in current testing flow.

coveralls · 2018-05-23T19:16:05Z

Coverage increased (+0.02%) to 96.992% when pulling 9685096 on SebastianCzoch:master into 1775dd4 on keighl:master.

widmogrod · 2018-05-25T09:00:00Z

@keighl could you take a look at this PR, this improvement solves also my issues. It would be fantastic to have it merged.

kevinburke1 · 2021-05-27T00:28:56Z

FYI this fails for me when I run go vet ./... with this error:

./mandrill_test.go:93:35: call of Unmarshal passes non-pointer as second argument

Fix empty error

9685096

SebastianCzoch commented May 23, 2018

View reviewed changes

Provide feedback


		expect(t, len(responses), 0)

		correctResponse := json.Unmarshal([]byte(` `), struct{}{})