diff options
Diffstat (limited to 'filetree')
-rw-r--r-- | filetree/data.go | 20 | ||||
-rw-r--r-- | filetree/data_test.go | 16 | ||||
-rw-r--r-- | filetree/node.go | 10 | ||||
-rw-r--r-- | filetree/node_test.go | 14 | ||||
-rw-r--r-- | filetree/tree.go | 2 | ||||
-rw-r--r-- | filetree/tree_test.go | 14 |
6 files changed, 38 insertions, 38 deletions
diff --git a/filetree/data.go b/filetree/data.go index 1218964..743ecb0 100644 --- a/filetree/data.go +++ b/filetree/data.go @@ -10,8 +10,8 @@ import ( ) const ( - Unchanged DiffType = iota - Changed + Unmodified DiffType = iota + Modified Added Removed ) @@ -23,7 +23,7 @@ func NewNodeData() *NodeData { return &NodeData{ ViewInfo: *NewViewInfo(), FileInfo: FileInfo{}, - DiffType: Unchanged, + DiffType: Unmodified, } } @@ -129,19 +129,19 @@ func (data *FileInfo) Compare(other FileInfo) DiffType { data.Mode == other.Mode && data.Uid == other.Uid && data.Gid == other.Gid { - return Unchanged + return Unmodified } } - return Changed + return Modified } // String of a DiffType func (diff DiffType) String() string { switch diff { - case Unchanged: - return "Unchanged" - case Changed: - return "Changed" + case Unmodified: + return "Unmodified" + case Modified: + return "Modified" case Added: return "Added" case Removed: @@ -157,5 +157,5 @@ func (diff DiffType) merge(other DiffType) DiffType { if diff == other { return diff } - return Changed + return Modified } diff --git a/filetree/data_test.go b/filetree/data_test.go index 7c9fad7..351d6df 100644 --- a/filetree/data_test.go +++ b/filetree/data_test.go @@ -10,23 +10,23 @@ func TestAssignDiffType(t *testing.T) { if err != nil { t.Errorf("Expected no error from fetching path. got: %v", err) } - node.Data.DiffType = Changed - if tree.Root.Children["usr"].Data.DiffType != Changed { + node.Data.DiffType = Modified + if tree.Root.Children["usr"].Data.DiffType != Modified { t.Fail() } } func TestMergeDiffTypes(t *testing.T) { - a := Unchanged - b := Unchanged + a := Unmodified + b := Unmodified merged := a.merge(b) - if merged != Unchanged { + if merged != Unmodified { t.Errorf("Expected Unchaged (0) but got %v", merged) } - a = Changed - b = Unchanged + a = Modified + b = Unmodified merged = a.merge(b) - if merged != Changed { + if merged != Modified { t.Errorf("Expected Unchaged (0) but got %v", merged) } } diff --git a/filetree/node.go b/filetree/node.go index b58c986..f0c880e 100644 --- a/filetree/node.go +++ b/filetree/node.go @@ -17,10 +17,10 @@ const ( ) var diffTypeColor = map[DiffType]*color.Color{ - Added: color.New(color.FgGreen), - Removed: color.New(color.FgRed), - Changed: color.New(color.FgYellow), - Unchanged: color.New(color.Reset), + Added: color.New(color.FgGreen), + Removed: color.New(color.FgRed), + Modified: color.New(color.FgYellow), + Unmodified: color.New(color.Reset), } // NewNode creates a new FileNode relative to the given parent node with a payload. @@ -292,7 +292,7 @@ func (node *FileNode) AssignDiffType(diffType DiffType) error { // compare the current node against the given node, returning a definitive DiffType. func (node *FileNode) compare(other *FileNode) DiffType { if node == nil && other == nil { - return Unchanged + return Unmodified } if node == nil && other != nil { diff --git a/filetree/node_test.go b/filetree/node_test.go index 0470978..a344475 100644 --- a/filetree/node_test.go +++ b/filetree/node_test.go @@ -115,7 +115,7 @@ func TestIsWhiteout(t *testing.T) { func TestDiffTypeFromAddedChildren(t *testing.T) { tree := NewFileTree() node, _, _ := tree.AddPath("/usr", *BlankFileChangeInfo("/usr")) - node.Data.DiffType = Unchanged + node.Data.DiffType = Unmodified node, _, _ = tree.AddPath("/usr/bin", *BlankFileChangeInfo("/usr/bin")) node.Data.DiffType = Added @@ -123,11 +123,11 @@ func TestDiffTypeFromAddedChildren(t *testing.T) { node, _, _ = tree.AddPath("/usr/bin2", *BlankFileChangeInfo("/usr/bin2")) node.Data.DiffType = Removed - err := tree.Root.Children["usr"].deriveDiffType(Unchanged) + err := tree.Root.Children["usr"].deriveDiffType(Unmodified) checkError(t, err, "unable to setup test") - if tree.Root.Children["usr"].Data.DiffType != Changed { - t.Errorf("Expected Changed but got %v", tree.Root.Children["usr"].Data.DiffType) + if tree.Root.Children["usr"].Data.DiffType != Modified { + t.Errorf("Expected Modified but got %v", tree.Root.Children["usr"].Data.DiffType) } } func TestDiffTypeFromRemovedChildren(t *testing.T) { @@ -142,11 +142,11 @@ func TestDiffTypeFromRemovedChildren(t *testing.T) { node, _, _ = tree.AddPath("/usr/.wh.bin2", *info2) node.Data.DiffType = Removed - err := tree.Root.Children["usr"].deriveDiffType(Unchanged) + err := tree.Root.Children["usr"].deriveDiffType(Unmodified) checkError(t, err, "unable to setup test") - if tree.Root.Children["usr"].Data.DiffType != Changed { - t.Errorf("Expected Changed but got %v", tree.Root.Children["usr"].Data.DiffType) + if tree.Root.Children["usr"].Data.DiffType != Modified { + t.Errorf("Expected Modified but got %v", tree.Root.Children["usr"].Data.DiffType) } } diff --git a/filetree/tree.go b/filetree/tree.go index fc4cf7f..2036f91 100644 --- a/filetree/tree.go +++ b/filetree/tree.go @@ -334,7 +334,7 @@ func (tree *FileTree) CompareAndMark(upper *FileTree) error { if err != nil { return err } - } else if pair.lowerNode.Data.DiffType == Unchanged { + } else if pair.lowerNode.Data.DiffType == Unmodified { err = pair.lowerNode.deriveDiffType(pair.tentative) if err != nil { return err diff --git a/filetree/tree_test.go b/filetree/tree_test.go index 7be1bb2..fc57036 100644 --- a/filetree/tree_test.go +++ b/filetree/tree_test.go @@ -423,7 +423,7 @@ func TestCompareWithNoChanges(t *testing.T) { if n.Path() == "/" { return nil } - if (n.Data.DiffType) != Unchanged { + if (n.Data.DiffType) != Unmodified { t.Errorf("Expecting node at %s to have DiffType unchanged, but had %v", n.Path(), n.Data.DiffType) } return nil @@ -477,11 +477,11 @@ func TestCompareWithAdds(t *testing.T) { failedAssertions = append(failedAssertions, err) } } else if stringInSlice(p, []string{"/usr/bin", "/usr"}) { - if err := AssertDiffType(n, Changed); err != nil { + if err := AssertDiffType(n, Modified); err != nil { failedAssertions = append(failedAssertions, err) } } else { - if err := AssertDiffType(n, Unchanged); err != nil { + if err := AssertDiffType(n, Unmodified); err != nil { failedAssertions = append(failedAssertions, err) } } @@ -588,11 +588,11 @@ func TestCompareWithChanges(t *testing.T) { if p == "/" { return nil } else if stringInSlice(p, changedPaths) { - if err := AssertDiffType(n, Changed); err != nil { + if err := AssertDiffType(n, Modified); err != nil { failedAssertions = append(failedAssertions, err) } } else { - if err := AssertDiffType(n, Unchanged); err != nil { + if err := AssertDiffType(n, Unmodified); err != nil { failedAssertions = append(failedAssertions, err) } } @@ -656,11 +656,11 @@ func TestCompareWithRemoves(t *testing.T) { failedAssertions = append(failedAssertions, err) } } else if stringInSlice(p, []string{"/usr", "/root"}) { - if err := AssertDiffType(n, Changed); err != nil { + if err := AssertDiffType(n, Modified); err != nil { failedAssertions = append(failedAssertions, err) } } else { - if err := AssertDiffType(n, Unchanged); err != nil { + if err := AssertDiffType(n, Unmodified); err != nil { failedAssertions = append(failedAssertions, err) } } |