Tests cleanup

[wszaranski]

Thanks for great project, here is a little cleanup - because less is more ✨ ✨ ✨

---

Change Run() to RunT(t) in tests

Goleak found unclosed goroutines in tests, for example
miniredis_test.go:TestDump never calls s.Close()

miniredis/miniredis_test.go

Lines 88 to 129 in 9ac631e

	func TestDump(t *testing.T) {
	s, err := Run()
	ok(t, err)
	s.Set("aap", "noot")
	s.Set("vuur", "mies")
	s.HSet("ahash", "aap", "noot")
	s.HSet("ahash", "vuur", "mies")
	if have, want := s.Dump(), `- aap
	"noot"
	- ahash
	aap: "noot"
	vuur: "mies"
	- vuur
	"mies"
	`; have != want {
	t.Errorf("have: %q, want: %q", have, want)
	}
	// Tricky whitespace
	s.Select(1)
	s.Set("whitespace", "foo\nbar\tbaz!")
	if have, want := s.Dump(), `- whitespace
	"foo\nbar\tbaz!"
	`; have != want {
	t.Errorf("have: %q, want: %q", have, want)
	}
	// Long key
	s.Select(2)
	s.Set("long", "This is a rather long key, with some fox jumping over a fence or something.")
	s.Set("countonme", "0123456789012345678901234567890123456789012345678901234567890123456789")
	s.HSet("hlong", "long", "This is another rather long key, with some fox jumping over a fence or something.")
	if have, want := s.Dump(), `- countonme
	"01234567890123456789012345678901234567890123456789012"...(70)
	- hlong
	long: "This is another rather long key, with some fox jumpin"...(81)
	- long
	"This is a rather long key, with some fox jumping over"...(75)
	`; have != want {
	t.Errorf("have: %q, want: %q", have, want)
	}
	}

I've changed Run() calls to RunT(t) calls to:

• ensure all goroutines all closed after test is finished
• we have shorter tests (no need to check for error and close server

There is no risk in using RunT even if test manually closes server
because calling Close multiple times is not an issue.

---

Move create of client in tests to helper function

Removed duplicated code in tests using helper function.

[alicebob]

:-)

good catch

[alicebob]

Great idea, thanks. t.Cleanup didn't exist back then.