aboutsummaryrefslogtreecommitdiffstats
path: root/revlist
diff options
context:
space:
mode:
authorAlberto Cortés <alberto@sourced.tech>2015-11-16 03:20:01 +0100
committerAlberto Cortés <alberto@sourced.tech>2015-11-25 11:09:51 +0100
commitd643cea1e8a6d618b2eddfdbed086c7bdf208658 (patch)
treeb862c72ccb674910d24eac2ab424a7fb5ea3f0fb /revlist
parentcaab43e7f4ee10a15b2af826485b688473b34356 (diff)
downloadgo-git-d643cea1e8a6d618b2eddfdbed086c7bdf208658.tar.gz
Blame support for files
This also includes a diff package and revlist package (needed by blame) Some extra packfiles (<1MB) are also included, to be used as fixtures in the tests.
Diffstat (limited to 'revlist')
-rw-r--r--revlist/revlist.go250
-rwxr-xr-xrevlist/revlist2humantest.bash36
-rw-r--r--revlist/revlist_test.go398
3 files changed, 684 insertions, 0 deletions
diff --git a/revlist/revlist.go b/revlist/revlist.go
new file mode 100644
index 0000000..f34ddb5
--- /dev/null
+++ b/revlist/revlist.go
@@ -0,0 +1,250 @@
+// Package revlist allows to create the revision history of a file, this
+// is, the list of commits in the past that affect the file.
+//
+// The general idea is to traverse the git commit graph backward,
+// flattening the graph into a linear history, and skipping commits that
+// are irrelevant for the particular file.
+//
+// There is no single answer for this operation. The git command
+// "git-revlist" returns different histories depending on its arguments
+// and some internal heuristics.
+//
+// The current implementation tries to get something similar to what you
+// whould get using git-revlist. See the failing tests for some
+// insight about how the current implementation and git-revlist differs.
+package revlist
+
+import (
+ "bytes"
+ "errors"
+ "io"
+ "sort"
+
+ "github.com/sergi/go-diff/diffmatchpatch"
+
+ "gopkg.in/src-d/go-git.v2"
+ "gopkg.in/src-d/go-git.v2/core"
+ "gopkg.in/src-d/go-git.v2/diff"
+)
+
+// New errors defined by the package.
+var ErrFileNotFound = errors.New("file not found")
+
+// A Revs is a list of revisions for a file (basically a list of commits).
+// It implements sort.Interface.
+type Revs []*git.Commit
+
+func (l Revs) Len() int {
+ return len(l)
+}
+
+// sorts from older to newer commit.
+func (l Revs) Less(i, j int) bool {
+ return l[i].Committer.When.Before(l[j].Committer.When)
+}
+
+func (l Revs) Swap(i, j int) {
+ l[i], l[j] = l[j], l[i]
+}
+
+// for debugging
+func (l Revs) String() string {
+ var buf bytes.Buffer
+ for _, c := range l {
+ buf.WriteString(c.Hash.String()[:8])
+ buf.WriteString("\n")
+ }
+ return buf.String()
+}
+
+// New returns a Revs pointer for the
+// file at "path", from commit "commit" backwards in time.
+// The commits are stored in arbitrary order.
+// It stops searching a branch for a file upon reaching the commit
+// were it was created.
+// Moves and copies are not currently supported.
+// Cherry-picks are not detected and therefore are added to the list
+// (see git path-id for hints on how to fix this).
+// This function implements is equivalent to running go-rev-Revs.
+func New(repo *git.Repository, commit *git.Commit, path string) (Revs, error) {
+ result := make(Revs, 0)
+ seen := make(map[core.Hash]struct{}, 0)
+ err := walkGraph(&result, &seen, repo, commit, path)
+ if err != nil {
+ return nil, err
+ }
+ sort.Sort(result)
+ result = removeComp(path, result, equivalent) // for merges of identical cherry-picks
+ if err != nil {
+ return nil, err
+ }
+ return result, nil
+}
+
+// Recursive traversal of the commit graph, generating a linear history
+// of the path.
+func walkGraph(result *Revs, seen *map[core.Hash]struct{}, repo *git.Repository, current *git.Commit, path string) error {
+ // check and update seen
+ if _, ok := (*seen)[current.Hash]; ok {
+ return nil
+ }
+ (*seen)[current.Hash] = struct{}{}
+
+ // if the path is not in the current commit, stop searching.
+ if _, found := git.FindFile(path, current); !found {
+ return nil
+ }
+
+ parents := parentsContainingPath(path, current)
+
+ switch len(parents) {
+ // if the path is not found in any of its parents, the path was
+ // created by this commit; we must add it to the revisions list and
+ // stop searching. This includes the case when current is the
+ // initial commit.
+ case 0:
+ //fmt.Println(current.Hash.String(), ": case 0")
+ *result = append(*result, current)
+ return nil
+ case 1: // only one parent contains the path
+ // if the file contents has change, add the current commit
+ different, err := differentContents(path, current, parents)
+ if err != nil {
+ return err
+ }
+ if len(different) == 1 {
+ //fmt.Println(current.Hash.String(), ": case 1")
+ *result = append(*result, current)
+ }
+ // in any case, walk the parent
+ return walkGraph(result, seen, repo, parents[0], path)
+ default: // more than one parent contains the path
+ // TODO: detect merges that had a conflict, because they must be
+ // included in the result here.
+ for _, p := range parents {
+ err := walkGraph(result, seen, repo, p, path)
+ if err != nil {
+ return err
+ }
+ }
+ }
+ return nil
+}
+
+// TODO: benchmark this making git.Commit.parent public instead of using
+// an iterator
+func parentsContainingPath(path string, c *git.Commit) []*git.Commit {
+ var result []*git.Commit
+ iter := c.Parents()
+ for {
+ parent, err := iter.Next()
+ if err != nil {
+ if err == io.EOF {
+ return result
+ }
+ panic("unreachable")
+ }
+ if _, found := git.FindFile(path, parent); found {
+ result = append(result, parent)
+ }
+ }
+}
+
+// Returns an slice of the commits in "cs" that has the file "path", but with different
+// contents than what can be found in "c".
+func differentContents(path string, c *git.Commit, cs []*git.Commit) ([]*git.Commit, error) {
+ result := make([]*git.Commit, 0, len(cs))
+ h, found := blobHash(path, c)
+ if !found {
+ return nil, ErrFileNotFound
+ }
+ for _, cx := range cs {
+ if hx, found := blobHash(path, cx); found && h != hx {
+ result = append(result, cx)
+ }
+ }
+ return result, nil
+}
+
+// blobHash returns the hash of a path in a commit
+func blobHash(path string, commit *git.Commit) (hash core.Hash, found bool) {
+ file, found := git.FindFile(path, commit)
+ if !found {
+ var empty core.Hash
+ return empty, found
+ }
+ return file.Hash, true
+}
+
+// Returns a new slice of commits, with duplicates removed. Expects a
+// sorted commit list. Duplication is defined according to "comp". It
+// will always keep the first commit of a series of duplicated commits.
+func removeComp(path string, cs []*git.Commit, comp func(string, *git.Commit, *git.Commit) bool) []*git.Commit {
+ result := make([]*git.Commit, 0, len(cs))
+ if len(cs) == 0 {
+ return result
+ }
+ result = append(result, cs[0])
+ for i := 1; i < len(cs); i++ {
+ if !comp(path, cs[i], cs[i-1]) {
+ result = append(result, cs[i])
+ }
+ }
+ return result
+}
+
+// Equivalent commits are commits whos patch is the same.
+func equivalent(path string, a, b *git.Commit) bool {
+ numParentsA := a.NumParents()
+ numParentsB := b.NumParents()
+
+ // the first commit is not equivalent to anyone
+ // and "I think" merges can not be equivalent to anything
+ if numParentsA != 1 || numParentsB != 1 {
+ return false
+ }
+
+ iterA := a.Parents()
+ parentA, _ := iterA.Next()
+ iterB := b.Parents()
+ parentB, _ := iterB.Next()
+
+ dataA, _ := git.Data(path, a)
+ dataParentA, _ := git.Data(path, parentA)
+ dataB, _ := git.Data(path, b)
+ dataParentB, _ := git.Data(path, parentB)
+
+ diffsA := diff.Do(dataParentA, dataA)
+ diffsB := diff.Do(dataParentB, dataB)
+
+ if sameDiffs(diffsA, diffsB) {
+ return true
+ }
+ return false
+}
+
+func sameDiffs(a, b []diffmatchpatch.Diff) bool {
+ if len(a) != len(b) {
+ return false
+ }
+ for i := range a {
+ if !sameDiff(a[i], b[i]) {
+ return false
+ }
+ }
+ return true
+}
+
+func sameDiff(a, b diffmatchpatch.Diff) bool {
+ if a.Type != b.Type {
+ return false
+ }
+ switch a.Type {
+ case 0:
+ return git.CountLines(a.Text) == git.CountLines(b.Text)
+ case 1, -1:
+ return a.Text == b.Text
+ default:
+ panic("unreachable")
+ }
+}
diff --git a/revlist/revlist2humantest.bash b/revlist/revlist2humantest.bash
new file mode 100755
index 0000000..b7d2672
--- /dev/null
+++ b/revlist/revlist2humantest.bash
@@ -0,0 +1,36 @@
+#!/bin/bash
+
+# you can run this over a whole repo with:
+#
+# for file in `find . -type f | sed 's/^\.\///' | egrep -v '^\.git\/.*$'` ; do revlist2humantest.bash $file ; done > /tmp/output
+#
+# be careful with files with spaces, though
+
+set -e
+
+repo=`git remote show origin | grep Fetch | cut -d' ' -f5`
+branch=`git branch | egrep '^\* .*' | cut -d' ' -f2`
+if [ "$#" -eq 1 ] ; then
+ commit=`git log | head -1 | cut -d' ' -f2`
+ path=$1
+elif [ "$#" -eq 2 ] ; then
+ commit=$1
+ path=$2
+else
+ echo "bad number of parameters" > /dev/stderr
+ echo > /dev/stderr
+ echo " try with: [commit] path" > /dev/stderr
+ exit
+fi
+
+hashes=`git rev-list --remove-empty --reverse $commit -- $path`
+
+# some remotes have the .git, other don't,
+# repoDot makes sure all have
+repoDot="${repo%.git}.git"
+
+echo -e "\t&humanTest{\"${repoDot}\", \"${branch}\", \"${commit}\", \"${path}\", []string{"
+for i in $hashes ; do
+ echo -e "\t\t\"${i}\","
+done
+echo -e "\t}},"
diff --git a/revlist/revlist_test.go b/revlist/revlist_test.go
new file mode 100644
index 0000000..7aea54c
--- /dev/null
+++ b/revlist/revlist_test.go
@@ -0,0 +1,398 @@
+package revlist
+
+import (
+ "bytes"
+ "fmt"
+ "os"
+ "testing"
+
+ "gopkg.in/src-d/go-git.v2"
+ "gopkg.in/src-d/go-git.v2/core"
+ "gopkg.in/src-d/go-git.v2/formats/packfile"
+
+ . "gopkg.in/check.v1"
+)
+
+func Test(t *testing.T) { TestingT(t) }
+
+type SuiteCommon struct {
+ repos map[string]*git.Repository
+}
+
+var _ = Suite(&SuiteCommon{})
+
+var fixtureRepos = [...]struct {
+ url string
+ packfile string
+}{
+ {"https://github.com/tyba/git-fixture.git", "../formats/packfile/fixtures/git-fixture.ofs-delta"},
+ {"https://github.com/jamesob/desk.git", "../formats/packfile/fixtures/jamesob-desk.pack"},
+ {"https://github.com/spinnaker/spinnaker.git", "../formats/packfile/fixtures/spinnaker-spinnaker.pack"},
+}
+
+// create the repositories of the fixtures
+func (s *SuiteCommon) SetUpSuite(c *C) {
+ s.repos = make(map[string]*git.Repository, 0)
+ for _, fixRepo := range fixtureRepos {
+ s.repos[fixRepo.url] = git.NewPlainRepository()
+
+ d, err := os.Open(fixRepo.packfile)
+ defer d.Close()
+ c.Assert(err, IsNil)
+
+ r := packfile.NewReader(d)
+ r.Format = packfile.OFSDeltaFormat // TODO: how to know the format of a pack file ahead of time?
+
+ _, err = r.Read(s.repos[fixRepo.url].Storage)
+ c.Assert(err, IsNil)
+ }
+}
+
+var revListTests = [...]struct {
+ // input data to revlist
+ repo string
+ branch string // TODO: remove this, it is no longer needed for local packfiles
+ commit string
+ path string
+ // expected output data form the revlist
+ revs []string
+}{
+ // Tyba git-fixture
+ {"https://github.com/tyba/git-fixture.git", "master", "6ecf0ef2c2dffb796033e5a02219af86ec6584e5", "binary.jpg", []string{
+ "35e85108805c84807bc66a02d91535e1e24b38b9",
+ }},
+ {"https://github.com/tyba/git-fixture.git", "master", "6ecf0ef2c2dffb796033e5a02219af86ec6584e5", "CHANGELOG", []string{
+ "b8e471f58bcbca63b07bda20e428190409c2db47",
+ }},
+ {"https://github.com/tyba/git-fixture.git", "master", "6ecf0ef2c2dffb796033e5a02219af86ec6584e5", "go/example.go", []string{
+ "918c48b83bd081e863dbe1b80f8998f058cd8294",
+ }},
+ {"https://github.com/tyba/git-fixture.git", "master", "6ecf0ef2c2dffb796033e5a02219af86ec6584e5", "json/long.json", []string{
+ "af2d6a6954d532f8ffb47615169c8fdf9d383a1a",
+ }},
+ {"https://github.com/tyba/git-fixture.git", "master", "6ecf0ef2c2dffb796033e5a02219af86ec6584e5", "json/short.json", []string{
+ "af2d6a6954d532f8ffb47615169c8fdf9d383a1a",
+ }},
+ {"https://github.com/tyba/git-fixture.git", "master", "6ecf0ef2c2dffb796033e5a02219af86ec6584e5", "LICENSE", []string{
+ "b029517f6300c2da0f4b651b8642506cd6aaf45d",
+ }},
+ {"https://github.com/tyba/git-fixture.git", "master", "6ecf0ef2c2dffb796033e5a02219af86ec6584e5", "php/crappy.php", []string{
+ "918c48b83bd081e863dbe1b80f8998f058cd8294",
+ }},
+ {"https://github.com/tyba/git-fixture.git", "master", "6ecf0ef2c2dffb796033e5a02219af86ec6584e5", "vendor/foo.go", []string{
+ "6ecf0ef2c2dffb796033e5a02219af86ec6584e5",
+ }},
+ {"https://github.com/jamesob/desk.git", "master", "d4edaf0e8101fcea437ebd982d899fe2cc0f9f7b", "LICENSE", []string{
+ "ffcda27c2de6768ee83f3f4a027fa4ab57d50f09",
+ }},
+ {"https://github.com/jamesob/desk.git", "master", "d4edaf0e8101fcea437ebd982d899fe2cc0f9f7b", "README.md", []string{
+ "ffcda27c2de6768ee83f3f4a027fa4ab57d50f09",
+ "2e87a2dcc63a115f9a61bd969d1e85fb132a431b",
+ "215b0ac06225b0671bc3460d10da88c3406f796f",
+ "0260eb7a2623dd2309ab439f74e8681fccdc4285",
+ "d46b48933e94f30992486374fa9a6becfd28ea17",
+ "9cb4df2a88efee8836f9b8ad27ca2717f624164e",
+ "8c49acdec2ed441706d8799f8b17878aae4c1ffe",
+ "ebaca0c6f54c23193ee8175c3530e370cb2dabe3",
+ "77675f82039551a19de4fbccbe69366fe63680df",
+ "b9741594fb8ab7374f9be07d6a09a3bf96719816",
+ "04db6acd94de714ca48128c606b17ee1149a630e",
+ "ff737bd8a962a714a446d7592fae423a56e61e12",
+ "eadd03f7a1cc54810bd10eef6747ad9562ad246d",
+ "b5072ab5c1cf89191d71f1244eecc5d1f369ef7e",
+ "bfa6ebc9948f1939402b063c0a2a24bf2b1c1cc3",
+ "d9aef39828c670dfdb172502021a2ebcda8cf2fb",
+ "1a6b6e45c91e1831494eb139ee3f8e21649c7fb0",
+ "09fdbe4612066cf63ea46aee43c7cfaaff02ecfb",
+ "236f6526b1150cc1f1723566b4738f443fc70777",
+ "7862953f470b62397d22f6782a884f5bea6d760d",
+ "b0b0152d08c2333680266977a5bc9c4e50e1e968",
+ "13ce6c1c77c831f381974aa1c62008a414bd2b37",
+ "d3f3c8faca048d11709969fbfc0cdf2901b87578",
+ "8777dde1abe18c805d021366643218d3f3356dd9",
+ }},
+ {"https://github.com/spinnaker/spinnaker.git", "master", "b32b2aecae2cfca4840dd480f8082da206a538da", "pylib/spinnaker/reconfigure_spinnaker.py", []string{
+ "a24001f6938d425d0e7504bdf5d27fc866a85c3d",
+ }},
+ {"https://github.com/spinnaker/spinnaker.git", "master", "b32b2aecae2cfca4840dd480f8082da206a538da", "pylib/spinnaker/validate_configuration.py", []string{
+ "a24001f6938d425d0e7504bdf5d27fc866a85c3d",
+ "1e14f94bcf82694fdc7e2dcbbfdbbed58db0f4d9",
+ "1e3d328a2cabda5d0aaddc5dec65271343e0dc37",
+ "b5d999e2986e190d81767cd3cfeda0260f9f6fb8",
+ }},
+ {"https://github.com/spinnaker/spinnaker.git", "master", "b32b2aecae2cfca4840dd480f8082da206a538da", "pylib/spinnaker/fetch.py", []string{
+ "a24001f6938d425d0e7504bdf5d27fc866a85c3d",
+ }},
+ {"https://github.com/spinnaker/spinnaker.git", "master", "b32b2aecae2cfca4840dd480f8082da206a538da", "pylib/spinnaker/yaml_util.py", []string{
+ "a24001f6938d425d0e7504bdf5d27fc866a85c3d",
+ "1e14f94bcf82694fdc7e2dcbbfdbbed58db0f4d9",
+ "b5d999e2986e190d81767cd3cfeda0260f9f6fb8",
+ "023d4fb17b76e0fe0764971df8b8538b735a1d67",
+ }},
+ {"https://github.com/spinnaker/spinnaker.git", "master", "b32b2aecae2cfca4840dd480f8082da206a538da", "dev/build_release.py", []string{
+ "a24001f6938d425d0e7504bdf5d27fc866a85c3d",
+ "1e14f94bcf82694fdc7e2dcbbfdbbed58db0f4d9",
+ "f42771ba298b93a7c4f5b16c5b30ab96c15305a8",
+ "dd52703a50e71891f63fcf05df1f69836f4e7056",
+ "0d9c9cef53af38cefcb6801bb492aaed3f2c9a42",
+ "d375f1994ff4d0bdc32d614e698f1b50e1093f14",
+ "abad497f11a366548aa95303c8c2f165fe7ae918",
+ "6986d885626792dee4ef6b7474dfc9230c5bda54",
+ "5422a86a10a8c5a1ef6728f5fc8894d9a4c54cb9",
+ "09a4ea729b25714b6368959eea5113c99938f7b6",
+ }},
+ {"https://github.com/spinnaker/spinnaker.git", "master", "b32b2aecae2cfca4840dd480f8082da206a538da", "pkg_scripts/postUninstall.sh", []string{
+ "ce9f123d790717599aaeb76bc62510de437761be",
+ }},
+ {"https://github.com/spinnaker/spinnaker.git", "master", "b32b2aecae2cfca4840dd480f8082da206a538da", "install/first_google_boot.sh", []string{
+ "a24001f6938d425d0e7504bdf5d27fc866a85c3d",
+ "de25f576b888569192e6442b0202d30ca7b2d8ec",
+ "a596972a661d9a7deca8abd18b52ce1a39516e89",
+ "9467ec579708b3c71dd9e5b3906772841c144a30",
+ "c4a9091e4076cb740fa46e790dd5b658e19012ad",
+ "6eb5d9c5225224bfe59c401182a2939d6c27fc00",
+ "495c7118e7cf757aa04eab410b64bfb5b5149ad2",
+ "dd2d03c19658ff96d371aef00e75e2e54702da0e",
+ "2a3b1d3b134e937c7bafdab6cc2950e264bf5dee",
+ "a57b08a9072f6a865f760551be2a4944f72f804a",
+ "0777fadf4ca6f458d7071de414f9bd5417911037",
+ }},
+ {"https://github.com/spinnaker/spinnaker.git", "master", "b32b2aecae2cfca4840dd480f8082da206a538da", "install/install_spinnaker.sh", []string{
+ "0d9c9cef53af38cefcb6801bb492aaed3f2c9a42",
+ }},
+ {"https://github.com/spinnaker/spinnaker.git", "master", "b32b2aecae2cfca4840dd480f8082da206a538da", "install/install_fake_openjdk8.sh", []string{
+ "a24001f6938d425d0e7504bdf5d27fc866a85c3d",
+ }},
+ {"https://github.com/spinnaker/spinnaker.git", "master", "b32b2aecae2cfca4840dd480f8082da206a538da", "install/install_spinnaker.py", []string{
+ "a24001f6938d425d0e7504bdf5d27fc866a85c3d",
+ "37f94770d81232b1895fca447878f68d65aac652",
+ "46c9dcbb55ca3f4735e82ad006e8cae2fdd050d9",
+ "124a88cfda413cb7182ca9c739a284a9e50042a1",
+ "eb4faf67a8b775d7985d07a708e3ffeac4273580",
+ "0d9c9cef53af38cefcb6801bb492aaed3f2c9a42",
+ "01171a8a2e843bef3a574ba73b258ac29e5d5405",
+ "739d8c6fe16edcb6ef9185dc74197de561b84315",
+ "d33c2d1e350b03fb989eefc612e8c9d5fa7cadc2",
+ }},
+ {"https://github.com/spinnaker/spinnaker.git", "master", "b32b2aecae2cfca4840dd480f8082da206a538da", "install/__init__.py", []string{
+ "a24001f6938d425d0e7504bdf5d27fc866a85c3d",
+ }},
+ {"https://github.com/spinnaker/spinnaker.git", "master", "b32b2aecae2cfca4840dd480f8082da206a538da", "experimental/docker-compose/docker-compose.yml", []string{
+ "fda357835d889595dc39dfebc6181d863cce7d4f",
+ "57c59e7144354a76e1beba69ae2f85db6b1727af",
+ "7682dff881029c722d893a112a64fea6849a0428",
+ "66f1c938c380a4096674b27540086656076a597f",
+ "56dc238f6f397e93f1d1aad702976889c830e8bf",
+ "b95e442c064935709e789fa02126f17ddceef10b",
+ "f98965a8f42037bd038b86c3401da7e6dfbf4f2e",
+ "5344429749e8b68b168d2707b7903692436cc2ea",
+ "6a31f5d219766b0cec4ea4fbbbfe47bdcdb0ab8e",
+ "ddaae195b628150233b0a48f50a1674fd9d1a924",
+ "7119ad9cf7d4e4d8b059e5337374baae4adc7458",
+ }},
+ {"https://github.com/spinnaker/spinnaker.git", "master", "b32b2aecae2cfca4840dd480f8082da206a538da", "unittest/validate_configuration_test.py", []string{
+ "1e14f94bcf82694fdc7e2dcbbfdbbed58db0f4d9",
+ "1e3d328a2cabda5d0aaddc5dec65271343e0dc37",
+ }},
+
+ // FAILS
+ /*
+ // this contains an empty move
+ {"https://github.com/spinnaker/spinnaker.git", "master", "b32b2aecae2cfca4840dd480f8082da206a538da", "google/dev/build_google_tarball.py", []string{
+ "88e60ac93f832efc2616b3c165e99a8f2ffc3e0c",
+ "9e49443da49b8c862cc140b660744f84eebcfa51",
+ }},
+ */
+ /*
+ {"https://github.com/spinnaker/spinnaker.git", "master", "b32b2aecae2cfca4840dd480f8082da206a538da", "unittest/yaml_util_test.py", []string{
+ "edf909edb9319c5e615e4ce73da47bbdca388ebe",
+ "023d4fb17b76e0fe0764971df8b8538b735a1d67",
+ }},
+ */
+ /*
+ {"https://github.com/spinnaker/spinnaker.git", "master", "b32b2aecae2cfca4840dd480f8082da206a538da", "unittest/configurator_test.py", []string{
+ "1e14f94bcf82694fdc7e2dcbbfdbbed58db0f4d9",
+ "edf909edb9319c5e615e4ce73da47bbdca388ebe",
+ "d14f793a6cd7169ef708a4fc276ad876bd3edd4e",
+ "023d4fb17b76e0fe0764971df8b8538b735a1d67",
+ }},
+ */
+ /*
+ // this contains a cherry-pick at 094d0e7d5d691 (with 3f34438d)
+ {"https://github.com/jamesob/desk.git", "master", "d4edaf0e8101fcea437ebd982d899fe2cc0f9f7b", "desk", []string{
+ "ffcda27c2de6768ee83f3f4a027fa4ab57d50f09",
+ "a0c1e853158ccbaf95574220bbf3b54509034a9f",
+ "decfc524570c407d6bba0f217e534c8b47dbdbee",
+ "1413872d5b3af7cd674bbe0e1f23387cd5d940e6",
+ "40cd5a91d916e7b2f331e4e85fdc52636fd7cff7",
+ "8e07d73aa0e3780f8c7cf8ad1a6b263df26a0a52",
+ "19c56f95720ac3630efe9f29b1a252581d6cbc0c",
+ "9ea46ccc6d253cffb4b7b66e936987d87de136e4",
+ "094d0e7d5d69141c98a606910ba64786c5565da0",
+ "801e62706a9e4fef75fcaca9c78744de0bc36e6a",
+ "eddf335f31c73624ed3f40dc5fcad50136074b2b",
+ "c659093f06eb2bd68c6252caeab605e5cd8aa49e",
+ "d94b3fe8ce0e3a474874d742992d432cd040582f",
+ "93cddf036df2d8509f910063696acd556ca7600f",
+ "b3d4cb0c826b16b301f088581d681654d8de6c07",
+ "52d90f9b513dd3c5330663cba39396e6b8a3ba4e",
+ "15919e99ded03c6ceea9ff98558e77a322a4dadb",
+ "803bf37847633e2f685a46a27b11facf22efebec",
+ "c07ad524ee1e616c70bf2ea7a0ee4f4a01195d78",
+ "b91aff30f318fda461d009c308490613b394f3e2",
+ "67cec1e8a3f21c6eb11678e3f31ffd228b55b783",
+ "bbe404c78af7525fabc57b9e7aa7c100b0d39f7a",
+ "5dd078848786c2babc2511e9502fa98518cf3535",
+ "7970ae7cc165c5205945dfb704d67d53031f550a",
+ "33091ac904747747ff30f107d4d0f22fa872eccf",
+ "069f81cab12d185ba1b509be946c47897cd4fb1f",
+ "13ce6c1c77c831f381974aa1c62008a414bd2b37",
+ }},
+ */
+ /*
+ {"https://github.com/spinnaker/spinnaker.git", "master", "b32b2aecae2cfca4840dd480f8082da206a538da", "InstallSpinnaker.sh", []string{
+ "ce9f123d790717599aaeb76bc62510de437761be",
+ "23673af3ad70b50bba7fdafadc2323302f5ba520",
+ "b7015a5d36990d69a054482556127b9c7404a24a",
+ "582da9622e3a72a19cd261a017276d72b5b0051a",
+ "0c5bb1e4392e751f884f3c57de5d4aee72c40031",
+ "c9c2a0ec03968ab17e8b16fdec9661eb1dbea173",
+ "a3cdf880826b4d9af42b93f4a2df29a91ab31d35",
+ "18526c447f5174d33c96aac6d6433318b0e2021c",
+ "2a6288be1c8ea160c443ca3cd0fe826ff2387d37",
+ "9e74d009894d73dd07773ea6b3bdd8323db980f7",
+ "d2f6214b625db706384b378a29cc4c22237db97a",
+ "202a9c720b3ba8106e022a0ad027ebe279040c78",
+ "791bcd1592828d9d5d16e83f3a825fb08b0ba22d",
+ "01e65d67eed8afcb67a6bdf1c962541f62b299c9",
+ "6328ee836affafc1b52127147b5ca07300ac78e6",
+ "3de4f77c105f700f50d9549d32b9a05a01b46c4b",
+ "8980daf661408a3faa1f22c225702a5c1d11d5c9",
+ "8eb116de9128c314ac8a6f5310ca500b8c74f5db",
+ "88e841aad37b71b78a8fb88bc75fe69499d527c7",
+ "370d61cdbc1f3c90db6759f1599ccbabd40ad6c1",
+ "505577dc87d300cf562dc4702a05a5615d90d855",
+ "b5c6053a46993b20d1b91e7b7206bffa54669ad7",
+ "ba486de7c025457963701114c683dcd4708e1dee",
+ "b41d7c0e5b20bbe7c8eb6606731a3ff68f4e3941",
+ "a47d0aaeda421f06df248ad65bd58230766bf118",
+ "495c7118e7cf757aa04eab410b64bfb5b5149ad2",
+ "46670eb6477c353d837dbaba3cf36c5f8b86f037",
+ "dd2d03c19658ff96d371aef00e75e2e54702da0e",
+ "4bbcad219ec55a465fb48ce236cb10ca52d43b1f",
+ "50d0556563599366f29cb286525780004fa5a317",
+ "9a06d3f20eabb254d0a1e2ff7735ef007ccd595e",
+ "d4b48a39aba7d3bd3e8abef2274a95b112d1ae73",
+ }},
+ */
+ /*
+ {"https://github.com/spinnaker/spinnaker.git", "master", "b32b2aecae2cfca4840dd480f8082da206a538da", "config/default-spinnaker-local.yml", []string{
+ "ae904e8d60228c21c47368f6a10f1cc9ca3aeebf",
+ "99534ecc895fe17a1d562bb3049d4168a04d0865",
+ "caf6d62e8285d4681514dd8027356fb019bc97ff",
+ "eaf7614cad81e8ab5c813dd4821129d0c04ea449",
+ "5a2a845bc08974a36d599a4a4b7e25be833823b0",
+ "41e96c54a478e5d09dd07ed7feb2d8d08d8c7e3c",
+ "974b775a8978b120ff710cac93a21c7387b914c9",
+ "87e459a9a044b3109dfeb943cc82c627b61d84a6",
+ "5e09821cbd7d710405b61cab0a795c2982a71b9c",
+ "8cc2d4bdb0a15aafc7fe02cdcb03ab90c974cafa",
+ "3ce7b902a51bac2f10994f7d1f251b616c975e54",
+ "a596972a661d9a7deca8abd18b52ce1a39516e89",
+ "8980daf661408a3faa1f22c225702a5c1d11d5c9",
+ }},
+ */
+ /*
+ {"https://github.com/spinnaker/spinnaker.git", "master", "b32b2aecae2cfca4840dd480f8082da206a538da", "config/spinnaker.yml", []string{
+ "ae904e8d60228c21c47368f6a10f1cc9ca3aeebf",
+ "caf6d62e8285d4681514dd8027356fb019bc97ff",
+ "eaf7614cad81e8ab5c813dd4821129d0c04ea449",
+ "5a2a845bc08974a36d599a4a4b7e25be833823b0",
+ "41e96c54a478e5d09dd07ed7feb2d8d08d8c7e3c",
+ "974b775a8978b120ff710cac93a21c7387b914c9",
+ "ed887f6547d7cd2b2d741184a06f97a0a704152b",
+ "d4553dac205023fa77652308af1a2d1cf52138fb",
+ "a596972a661d9a7deca8abd18b52ce1a39516e89",
+ "66ac94f0b4442707fb6f695fbed91d62b3bd9d4a",
+ "079e42e7c979541b6fab7343838f7b9fd4a360cd",
+ }},
+ */
+}
+
+func (s *SuiteCommon) TestRevList(c *C) {
+ for _, t := range revListTests {
+ repo, ok := s.repos[t.repo]
+ c.Assert(ok, Equals, true)
+
+ commit, err := repo.Commit(core.NewHash(t.commit))
+ c.Assert(err, IsNil)
+
+ revs, err := New(repo, commit, t.path)
+ c.Assert(err, IsNil)
+
+ c.Assert(len(revs), Equals, len(t.revs), Commentf("\nrepo=%s, branch=%s, commit=%s, path=%s\n EXPECTED (len %d)\n%s\n OBTAINED (len %d)\n%s\n",
+ t.repo, t.branch, t.commit, t.path, len(t.revs), t.revs, len(revs), revs.String()))
+ for i := range revs {
+ if revs[i].Hash.String() != t.revs[i] {
+ commit, err := repo.Commit(core.NewHash(t.revs[i]))
+ c.Assert(err, IsNil)
+ if equivalent(t.path, revs[i], commit) {
+ fmt.Printf("cherry-pick detected: %s %s\n", revs[i].Hash.String(), t.revs[i])
+ } else {
+ c.Fatalf("\nrepo=%s, branch=%s, commit=%s, path=%s, \n%s",
+ t.repo, t.branch, t.commit, t.path, compareSideBySide(t.revs, revs))
+ }
+ }
+ }
+ fmt.Printf("OK repo=%s, branch=%s, commit=%s, path=%s\n",
+ t.repo, t.branch, t.commit, t.path)
+ }
+}
+
+// same length is assumed
+func compareSideBySide(a []string, b []*git.Commit) string {
+ var buf bytes.Buffer
+ buf.WriteString("\t EXPECTED OBTAINED ")
+ var sep string
+ var obtained string
+ for i := range a {
+ obtained = b[i].Hash.String()
+ if a[i] != obtained {
+ sep = "------"
+ } else {
+ sep = " "
+ }
+ buf.WriteString(fmt.Sprintf("\n%d", i+1))
+ buf.WriteString(sep)
+ buf.WriteString(a[i])
+ buf.WriteString(sep)
+ buf.WriteString(obtained)
+ }
+ return buf.String()
+}
+
+var cherryPicks = [...][]string{
+ // repo, path, commit a, commit b
+ []string{"https://github.com/jamesob/desk.git", "desk", "094d0e7d5d69141c98a606910ba64786c5565da0", "3f34438d54f4a1ca86db8c0f03ed8eb38f20e22c"},
+}
+
+// should detect cherry picks
+func (s *SuiteCommon) TestEquivalent(c *C) {
+ for _, t := range cherryPicks {
+ cs := s.commits(c, t[0], t[2], t[3])
+ c.Assert(equivalent(t[1], cs[0], cs[1]), Equals, true, Commentf("repo=%s, file=%s, a=%s b=%s", t[0], t[1], t[2], t[3]))
+ }
+}
+
+// returns the commits from a slice of hashes
+func (s *SuiteCommon) commits(cc *C, repo string, hs ...string) []*git.Commit {
+ r, ok := s.repos[repo]
+ cc.Assert(ok, Equals, true)
+ result := make([]*git.Commit, 0, len(hs))
+ for _, h := range hs {
+ c, err := r.Commit(core.NewHash(h))
+ cc.Assert(err, IsNil)
+ result = append(result, c)
+ }
+ return result
+}