Single vs double spaced output about cupaloy

[cobbinma]

cupaloy.New(cupaloy.UseStringerMethods(false)).SnapshotT(t, hours)

[bradleyjkemp]

Ah awesome there already is an option for disabling String() methods!

[bradleyjkemp]

Hey @mattharrigan do you have an example of a test where you're seeing this inconsistency?
By double spaces are you meaning between the snapshots of variables or actually within the snapshot of a single variable?

Sorry for not replying sooner, the GitHub notification for this is showing as 6 hours ago 🤔

[cobbinma]

Hey @bradleyjkemp, I'm getting the same issue. The snapshot output for all tests seem to change from single / double spaces at random.

    --- FAIL: Test_Repository/get_opening_hours (0.00s)
        client_test.go:223: snapshot not equal:
            --- Previous
            +++ Current
            @@ -1,2 +1,2 @@
            -(*api.GetOpeningHoursSpecificationResponse)(specification:{dayOfWeek:2  opens:"11:00"  closes:"22:00"  validFrom:"3000-11-01T00:00:00Z"  validThrough:"3000-12-01T00:00:00Z"})
            +(*api.GetOpeningHoursSpecificationResponse)(specification:{dayOfWeek:2 opens:"11:00" closes:"22:00" validFrom:"3000-11-01T00:00:00Z" validThrough:"3000-12-01T00:00:00Z"})

{
	name: "get opening hours",
	test: func(t *testing.T) {
		ctx := context.Background()
		hours, err := repository.GetOpeningHoursSpecification(ctx, &api.GetOpeningHoursSpecificationRequest{
			VenueId: UUID,
			Date:    "3000-11-11T00:00:00Z",
		})
		require.NoError(t, err)

		cupaloy.SnapshotT(t, hours)
	},
},

[bradleyjkemp]

Ah are you using protobuf by any chance?

The default protobuf String() method intentionally introduces randomness: golang/protobuf#1269

Cupaloy shouldn't really be using the String() method by default but it'd be a breaking change to disable the behaviour. Cupaloy should probably have an option to disable this though.
An alternative is to marshal your protobuf message yourself before passing it to Cupaloy

[cobbinma]

thanks @bradleyjkemp 👍 , didn't know about that protobuf behaviour