From 3d454d9dc8ba2409046c0938618a70864e6eb8ef Mon Sep 17 00:00:00 2001 From: Michael Muré Date: Mon, 25 Jul 2022 13:16:16 +0200 Subject: entity/dag: proper base operation for simplified implementation - reduce boilerplace necessary to implement an operation - consolidate what an operation is in the core, which in turn pave the way for a generic cache layer mechanism - avoid the previously complex unmarshalling process - support operation metadata from the core - simplified testing --- bug/op_set_title_test.go | 31 ++++--------------------------- 1 file changed, 4 insertions(+), 27 deletions(-) (limited to 'bug/op_set_title_test.go') diff --git a/bug/op_set_title_test.go b/bug/op_set_title_test.go index 2a227709..8b45c74e 100644 --- a/bug/op_set_title_test.go +++ b/bug/op_set_title_test.go @@ -1,37 +1,14 @@ package bug import ( - "encoding/json" "testing" - "time" - - "github.com/stretchr/testify/require" + "github.com/MichaelMure/git-bug/entity/dag" "github.com/MichaelMure/git-bug/identity" - "github.com/MichaelMure/git-bug/repository" ) func TestSetTitleSerialize(t *testing.T) { - repo := repository.NewMockRepo() - - rene, err := identity.NewIdentity(repo, "René Descartes", "rene@descartes.fr") - require.NoError(t, err) - - unix := time.Now().Unix() - before := NewSetTitleOp(rene, unix, "title", "was") - - data, err := json.Marshal(before) - require.NoError(t, err) - - var after SetTitleOperation - err = json.Unmarshal(data, &after) - require.NoError(t, err) - - // enforce creating the ID - before.Id() - - // Replace the identity as it's not serialized - after.Author_ = rene - - require.Equal(t, before, &after) + dag.SerializeRoundTripTest(t, func(author identity.Interface, unixTime int64) *SetTitleOperation { + return NewSetTitleOp(author, unixTime, "title", "was") + }) } -- cgit