// Copyright 2017 syzkaller project authors. All rights reserved. // Use of this source code is governed by Apache 2 LICENSE that can be found in the LICENSE file. package vcs import ( "os" "reflect" "sort" "testing" "time" "github.com/google/go-cmp/cmp" "github.com/stretchr/testify/assert" ) func TestGitParseCommit(t *testing.T) { tests := map[string]*Commit{ `2075b16e32c26e4031b9fd3cbe26c54676a8fcb5 rbtree: include rcu.h foobar@foobar.de Foo Bar Fri May 11 16:02:14 2018 -0700 78eb0c6356cda285c6ee6e29bea0c0188368103e Fri May 11 17:28:45 2018 -0700 Since commit c1adf20052d8 ("Introduce rb_replace_node_rcu()") rbtree_augmented.h uses RCU related data structures but does not include the header file. It works as long as it gets somehow included before that and fails otherwise. Link: http://lkml.kernel.org/r/20180504103159.19938-1-bigeasy@linutronix.de Signed-off-by: Foo Bad Baz Reviewed-by: Cc: Unrelated Guy Acked-by: Subsystem reviewer Reported-and-tested-by: and@me.com Reported-and-Tested-by: Name-name Tested-by: Must be correct Signed-off-by: Linux Master `: { Hash: "2075b16e32c26e4031b9fd3cbe26c54676a8fcb5", Title: "rbtree: include rcu.h", Author: "foobar@foobar.de", AuthorName: "Foo Bar", Recipients: NewRecipients([]string{ "and@me.com", "another@email.de", "foobar@foobar.de", "linux@linux-foundation.org", "mustbe@correct.com", "name@name.com", "subsystem@reviewer.com", "yetanother@email.org", }, To), Date: time.Date(2018, 5, 11, 16, 02, 14, 0, time.FixedZone("", -7*60*60)), CommitDate: time.Date(2018, 5, 11, 17, 28, 45, 0, time.FixedZone("", -7*60*60)), }, } for input, com := range tests { res, err := gitParseCommit([]byte(input), nil, nil, nil) if err != nil && com != nil { t.Fatalf("want %+v, got error: %v", com, err) } if err == nil && com == nil { t.Fatalf("want error, got commit %+v", res) } if com == nil { continue } if com.Hash != res.Hash { t.Fatalf("want hash %q, got %q", com.Hash, res.Hash) } if com.Title != res.Title { t.Fatalf("want title %q, got %q", com.Title, res.Title) } if com.Author != res.Author { t.Fatalf("want author %q, got %q", com.Author, res.Author) } if diff := cmp.Diff(com.Recipients, res.Recipients); diff != "" { t.Fatalf("bad CC: %v", diff) } if !com.Date.Equal(res.Date) { t.Fatalf("want date %v, got %v", com.Date, res.Date) } if !com.CommitDate.Equal(res.CommitDate) { t.Fatalf("want date %v, got %v", com.CommitDate, res.CommitDate) } } } func TestGitParseReleaseTags(t *testing.T) { input := ` v3.1 v2.6.12 v2.6.39 v3.0 v3.10 v2.6.13 v3.11 v3.19 v3.9 v3.2 v4.9-rc1 v4.9 v4.9-rc3 v4.9-rc2 v2.6.32 v4.0 vv4.1 v2.6-rc5 v4.1foo voo v1.foo v2.6-rc2 v10.2.foo v1.2. v1. ` want := []string{ "v4.9", "v4.0", "v3.19", "v3.11", "v3.10", "v3.9", "v3.2", "v3.1", "v3.0", "v2.6.39", "v2.6.32", "v2.6.13", "v2.6.12", } got := gitParseReleaseTags([]byte(input), false) if !reflect.DeepEqual(got, want) { t.Fatalf("got bad tags\ngot: %+v\nwant: %+v", got, want) } wantRC := []string{ "v4.9", "v4.9-rc3", "v4.9-rc2", "v4.9-rc1", "v4.0", "v3.19", "v3.11", "v3.10", "v3.9", "v3.2", "v3.1", "v3.0", "v2.6.39", "v2.6.32", "v2.6.13", "v2.6.12", "v2.6-rc5", "v2.6-rc2", } gotRC := gitParseReleaseTags([]byte(input), true) if !reflect.DeepEqual(gotRC, wantRC) { t.Fatalf("got bad tags\ngot: %+v\nwant: %+v", gotRC, wantRC) } } func TestGetCommitsByTitles(t *testing.T) { baseDir := t.TempDir() repo := MakeTestRepo(t, baseDir) validateSuccess := func(commit *Commit, results []*Commit, missing []string, err error) { if err != nil { t.Fatalf("expected success, got %v", err) } if len(missing) > 0 { t.Fatalf("expected 0 missing, got %v", missing) } if len(results) != 1 { t.Fatalf("expected 1 results, got %v", len(results)) } if results[0].Hash != commit.Hash { t.Fatalf("found unexpected commit %v", results[0].Hash) } } // Put three commits in branch-a, two with the title we search for. // We expect GetCommitsByTitles to only return the most recent match. repo.Git("checkout", "-b", "branch-a") repo.Git("commit", "--no-edit", "--allow-empty", "-m", "target") repo.Git("commit", "--no-edit", "--allow-empty", "-m", "abc") repo.Git("commit", "--no-edit", "--allow-empty", "-m", "target") commitA, _ := repo.repo.Commit(HEAD) results, missing, err := repo.repo.GetCommitsByTitles([]string{"target"}) validateSuccess(commitA, results, missing, err) // Put another commit with the title we search for in another branch. // We expect GetCommitsByTitles to only find commits in the current branch. repo.Git("checkout", "-b", "branch-b") repo.Git("commit", "--no-edit", "--allow-empty", "-m", "target") repo.Git("checkout", "branch-a") results, missing, err = repo.repo.GetCommitsByTitles([]string{"target"}) validateSuccess(commitA, results, missing, err) // We expect GetCommitsByTitles to only find commits in the current branch. repo.Git("checkout", "branch-b") results, missing, err = repo.repo.GetCommitsByTitles([]string{"xyz"}) if err != nil { t.Fatalf("expected success, got %v", err) } if len(results) > 0 { t.Fatalf("expected 0 results, got %v", len(results)) } if len(missing) != 1 { t.Fatalf("expected 1 missing, got %v", missing) } if missing[0] != "xyz" { t.Fatalf("found unexpected value in missing %v", missing[0]) } } func TestContains(t *testing.T) { baseDir := t.TempDir() repo := MakeTestRepo(t, baseDir) // We expect Contains to return true, if commit is in current checkout. repo.Git("checkout", "-b", "branch-a") repo.Git("commit", "--no-edit", "--allow-empty", "-m", "target") commitA, _ := repo.repo.Commit(HEAD) if contained, _ := repo.repo.Contains(commitA.Hash); !contained { t.Fatalf("contains claims commit that should be present is not") } if contained, _ := repo.repo.Contains("fake-hash"); contained { t.Fatalf("contains claims commit that is not present is present") } // Commits must only be searched for from the checkedout HEAD. repo.Git("checkout", "-b", "branch-b") repo.Git("commit", "--no-edit", "--allow-empty", "-m", "target") commitB, _ := repo.repo.Commit(HEAD) repo.Git("checkout", "branch-a") if contained, _ := repo.repo.Contains(commitB.Hash); contained { t.Fatalf("contains found commit that is not in current branch") } } func TestCommitHashes(t *testing.T) { baseDir := t.TempDir() repo := MakeTestRepo(t, baseDir) repo.Git("checkout", "-b", "branch-a") repo.Git("commit", "--no-edit", "--allow-empty", "-m", "target") repo.Git("checkout", "-b", "branch-b") repo.Git("commit", "--no-edit", "--allow-empty", "-m", "target") got, err := repo.repo.ListCommitHashes("HEAD") if err != nil { t.Fatal(err) } if len(got) != 2 { t.Fatalf("expected 2 commits") } for i, commit := range got { if contained, _ := repo.repo.Contains(commit); !contained { t.Fatalf("commit %d is not contained", i) } } // Now change HEAD. repo.Git("checkout", "branch-a") got, err = repo.repo.ListCommitHashes("HEAD") if err != nil { t.Fatal(err) } if len(got) != 1 { t.Fatalf("expected 1 commit, got %d", len(got)) } if contained, _ := repo.repo.Contains(got[0]); !contained { t.Fatalf("commit in branch-b is not contained") } } func TestObject(t *testing.T) { baseDir := t.TempDir() repo := MakeTestRepo(t, baseDir) firstRev := []byte("First revision") secondRev := []byte("Second revision") if err := os.WriteFile(baseDir+"/object.txt", firstRev, 0644); err != nil { t.Fatal(err) } repo.Git("add", "object.txt") repo.Git("commit", "--no-edit", "--allow-empty", "-m", "target") if err := os.WriteFile(baseDir+"/object.txt", secondRev, 0644); err != nil { t.Fatal(err) } repo.Git("add", "object.txt") repo.Git("commit", "--no-edit", "--allow-empty", "-m", "target") commits, err := repo.repo.ListCommitHashes("HEAD") if err != nil { t.Fatal(err) } if len(commits) != 2 { t.Fatalf("expected 2 commits, got %d", len(commits)) } // Verify file's contents at the first revision. data, err := repo.repo.Object("object.txt", commits[1]) if err != nil { t.Fatal(err) } if diff := cmp.Diff(data, firstRev); diff != "" { t.Fatal(diff) } // And at the second one. data, err = repo.repo.Object("object.txt", commits[0]) if err != nil { t.Fatal(err) } if diff := cmp.Diff(data, secondRev); diff != "" { t.Fatal(diff) } com, err := repo.repo.Commit(commits[0]) if err != nil { t.Fatal(err.Error()) } patch := []byte(`diff --git a/object.txt b/object.txt index 103167d..fbf7a68 100644 --- a/object.txt +++ b/object.txt @@ -1 +1 @@ -First revision \ No newline at end of file +Second revision \ No newline at end of file `) if diff := cmp.Diff(com.Patch, patch); diff != "" { t.Fatal(diff) } } func TestMergeBase(t *testing.T) { baseDir := t.TempDir() repo := MakeTestRepo(t, baseDir) // Create base branch. repo.Git("checkout", "-b", "base") repo.Git("commit", "--no-edit", "--allow-empty", "-m", "target") baseCommit, _ := repo.repo.Commit(HEAD) // Fork off another branch. repo.Git("checkout", "-b", "fork") repo.Git("commit", "--no-edit", "--allow-empty", "-m", "target") forkCommit, _ := repo.repo.Commit(HEAD) // Ensure that merge base points to the base commit. mergeCommits, err := repo.repo.MergeBases(baseCommit.Hash, forkCommit.Hash) if err != nil { t.Fatal(err) } else if len(mergeCommits) != 1 || mergeCommits[0].Hash != baseCommit.Hash { t.Fatalf("expected base commit, got %v", mergeCommits) } // Let branches diverge. repo.Git("checkout", "base") repo.Git("commit", "--no-edit", "--allow-empty", "-m", "newBase") newBaseCommit, _ := repo.repo.Commit(HEAD) repo.Git("checkout", "fork") repo.Git("commit", "--no-edit", "--allow-empty", "-m", "newFork") newForkCommit, _ := repo.repo.Commit(HEAD) // The merge base should remain the same. mergeCommits, err = repo.repo.MergeBases(newBaseCommit.Hash, newForkCommit.Hash) if err != nil { t.Fatal(err) } else if len(mergeCommits) != 1 || mergeCommits[0].Hash != baseCommit.Hash { t.Fatalf("expected base commit (%s), got %d other commits", baseCommit.Hash, len(mergeCommits)) } // Now do the merge. repo.Git("merge", "base") // And advance the fork branch. repo.Git("commit", "--no-edit", "--allow-empty", "-m", "target") newNewForkCommit, _ := repo.repo.Commit(HEAD) // The merge base should point to the last commit in `base`. mergeCommits, err = repo.repo.MergeBases(newBaseCommit.Hash, newNewForkCommit.Hash) if err != nil { t.Fatal(err) } else if len(mergeCommits) != 1 || mergeCommits[0].Hash != newBaseCommit.Hash { t.Fatalf("expected base commit, got %v", mergeCommits) } } func TestGitCustomRefs(t *testing.T) { remoteRepoDir := t.TempDir() remote := MakeTestRepo(t, remoteRepoDir) remote.Git("commit", "--no-edit", "--allow-empty", "-m", "base commit") remote.Git("checkout", "-b", "base_branch") remote.Git("tag", "base_tag") // Create a commit non reachable from any branch or tag. remote.Git("checkout", "base_branch") remote.Git("checkout", "-b", "temp_branch") remote.Git("commit", "--no-edit", "--allow-empty", "-m", "detached commit") // Add a ref to prevent the commit from getting garbage collected. remote.Git("update-ref", "refs/custom/test", "temp_branch") refCommit, _ := remote.repo.Commit(HEAD) // Remove the branch, let the commit stay only in refs. remote.Git("checkout", "base_branch") remote.Git("branch", "-D", "temp_branch") // Create a local repo. localRepoDir := t.TempDir() local := newGit(localRepoDir, nil, nil) // Fetch the commit from the custom ref. _, err := local.CheckoutCommit(remoteRepoDir, refCommit.Hash) assert.NoError(t, err) } func TestGitRemoteTags(t *testing.T) { remoteRepoDir := t.TempDir() remote := MakeTestRepo(t, remoteRepoDir) remote.Git("commit", "--no-edit", "--allow-empty", "-m", "base commit") remote.Git("checkout", "-b", "base_branch") remote.Git("tag", "v1.0") // Diverge sub_branch and add a tag. remote.Git("commit", "--no-edit", "--allow-empty", "-m", "sub-branch") remote.Git("checkout", "-b", "sub_branch") remote.Git("tag", "v2.0") // Create a local repo. localRepoDir := t.TempDir() local := newGit(localRepoDir, nil, nil) // Ensure all tags were fetched. commit, err := local.CheckoutCommit(remoteRepoDir, "sub_branch") assert.NoError(t, err) tags, err := local.previousReleaseTags(commit.Hash, true, false, false) assert.NoError(t, err) sort.Strings(tags) assert.Equal(t, []string{"v1.0", "v2.0"}, tags) } func TestGitFetchShortHash(t *testing.T) { remoteRepoDir := t.TempDir() remote := MakeTestRepo(t, remoteRepoDir) remote.Git("commit", "--no-edit", "--allow-empty", "-m", "base commit") remote.Git("checkout", "-b", "base_branch") remote.Git("tag", "base_tag") remote.Git("checkout", "-b", "temp_branch") remote.Git("commit", "--no-edit", "--allow-empty", "-m", "detached commit") refCommit, _ := remote.repo.Commit(HEAD) // Create a local repo. localRepoDir := t.TempDir() local := newGit(localRepoDir, nil, nil) // Fetch the commit from the custom ref. _, err := local.CheckoutCommit(remoteRepoDir, refCommit.Hash[:12]) assert.NoError(t, err) } func TestParseGitDiff(t *testing.T) { files := ParseGitDiff([]byte(`diff --git a/a.txt b/a.txt index 4c5fd91..8fe1e32 100644 --- a/a.txt +++ b/a.txt @@ -1 +1 @@ -First file +First file! diff --git a/b.txt b/b.txt new file mode 100644 index 0000000..f8a9677 --- /dev/null +++ b/b.txt @@ -0,0 +1 @@ +Second file. diff --git a/c/c.txt b/c/c.txt new file mode 100644 index 0000000..e69de29 `)) assert.ElementsMatch(t, files, []string{"a.txt", "b.txt", "c/c.txt"}) }