diff options
author | Michael Muré <batolettre@gmail.com> | 2020-06-23 18:02:54 +0200 |
---|---|---|
committer | Michael Muré <batolettre@gmail.com> | 2020-06-26 19:14:22 +0200 |
commit | 88ad7e606f1cbf9e47b968a208e3510f7f9a81c5 (patch) | |
tree | 00e847500b34c6f9a721c71474993d1c08ae8fb1 /repository/tree_entry_test.go | |
parent | 2dd0dbb1344ae9293aae05346f977b5d5907934b (diff) | |
download | git-bug-88ad7e606f1cbf9e47b968a208e3510f7f9a81c5.tar.gz |
repository: remove tie to Bug, improved and reusable testing
- allow the creation of arbitrary Lamport clocks, freeing the way to new entities and removing Bug specific (upper layer) code.
- generalize the memory-only and persisted Lamport clocks behind a common interface
- rework the tests to provide reusable testing code for a Repo, a Clock, a Config, opening a path to add a new Repo implementation more easily
- test previously untested components with those new tests
Note: one problem found during this endeavor is that `identity.Version` also need to store one time + Lamport time for each other Entity (Bug, config, PR ...). This could possibly done without breaking change but it would be much easier to wait for https://github.com/MichaelMure/git-bug-migration to happen.
Diffstat (limited to 'repository/tree_entry_test.go')
-rw-r--r-- | repository/tree_entry_test.go | 8 |
1 files changed, 3 insertions, 5 deletions
diff --git a/repository/tree_entry_test.go b/repository/tree_entry_test.go index 58da075c..78f58ff6 100644 --- a/repository/tree_entry_test.go +++ b/repository/tree_entry_test.go @@ -3,11 +3,12 @@ package repository import ( "testing" + "github.com/stretchr/testify/assert" + "github.com/MichaelMure/git-bug/util/git" ) func TestTreeEntryFormat(t *testing.T) { - entries := []TreeEntry{ {Blob, git.Hash("a85730cf5287d40a1e32d3a671ba2296c73387cb"), "name"}, {Tree, git.Hash("a85730cf5287d40a1e32d3a671ba2296c73387cb"), "name"}, @@ -26,10 +27,7 @@ func TestTreeEntryParse(t *testing.T) { for _, line := range lines { _, err := ParseTreeEntry(line) - - if err != nil { - t.Fatal(err) - } + assert.NoError(t, err) } } |