From 43da5e3a1baae2b2fa4f00e2218632e882654517 Mon Sep 17 00:00:00 2001 From: Dmitry Vyukov Date: Wed, 27 Jun 2018 13:07:03 +0200 Subject: pkg/vcs: move from pkg/git Rename pkg/git to pkg/vcs because we need to support not only git. --- pkg/git/git_test.go | 239 ---------------------------------------------------- 1 file changed, 239 deletions(-) delete mode 100644 pkg/git/git_test.go (limited to 'pkg/git/git_test.go') diff --git a/pkg/git/git_test.go b/pkg/git/git_test.go deleted file mode 100644 index 86f033acf..000000000 --- a/pkg/git/git_test.go +++ /dev/null @@ -1,239 +0,0 @@ -// 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 git - -import ( - "reflect" - "strings" - "testing" - "time" -) - -func TestParseCommit(t *testing.T) { - tests := map[string]*Commit{ - `2075b16e32c26e4031b9fd3cbe26c54676a8fcb5 -rbtree: include rcu.h -foobar@foobar.de -Fri May 11 16:02:14 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 -`: &Commit{ - Hash: "2075b16e32c26e4031b9fd3cbe26c54676a8fcb5", - Title: "rbtree: include rcu.h", - Author: "foobar@foobar.de", - CC: []string{ - "and@me.com", - "foobar@foobar.de", - "mustbe@correct.com", - "name@name.com", - "subsystem@reviewer.com", - "yetanother@email.org", - }, - Date: time.Date(2018, 5, 11, 16, 02, 14, 0, time.FixedZone("", -7*60*60)), - }, - } - for input, com := range tests { - res, err := parseCommit([]byte(input)) - 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 !reflect.DeepEqual(com.CC, res.CC) { - t.Fatalf("want CC %q, got %q", com.CC, res.CC) - } - if !com.Date.Equal(res.Date) { - t.Fatalf("want date %v, got %v", com.Date, res.Date) - } - } -} - -func TestCanonicalizeCommit(t *testing.T) { - tests := map[string]string{ - "foo bar": "foo bar", - " foo ": "foo", - "UPSTREAM: foo bar": "foo bar", - "BACKPORT: UPSTREAM: foo bar": "UPSTREAM: foo bar", - } - for in, want := range tests { - got := CanonicalizeCommit(in) - if got != want { - t.Errorf("input %q: got %q, want %q", in, got, want) - } - } -} - -func TestCheckRepoAddress(t *testing.T) { - testPredicate(t, CheckRepoAddress, map[string]bool{ - "git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git": true, - "https://github.com/torvalds/linux.git": true, - "git://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable.git": true, - "git://git.cmpxchg.org/linux-mmots.git": true, - "https://anonscm.debian.org/git/kernel/linux.git": true, - "git://kernel.ubuntu.com/ubuntu/ubuntu-zesty.git": true, - "http://host.xz:123/path/to/repo.git/": true, - "": false, - "foobar": false, - "linux-next": false, - "foo://kernel.ubuntu.com/ubuntu/ubuntu-zesty.git": false, - "git://kernel/ubuntu.git": false, - "git://kernel.com/ubuntu": false, - "gitgit://kernel.ubuntu.com/ubuntu/ubuntu-zesty.git": false, - }) -} - -func TestCheckBranch(t *testing.T) { - testPredicate(t, CheckBranch, map[string]bool{ - "master": true, - "core/core": true, - "irq-irqdomain-for-linus": true, - "timers/2038": true, - "ubuntu-zesty/v4.9.4": true, - "WIP.locking/atomics": true, - "linux-4.9.y": true, - "abi_spec": true, - "@": false, - "": false, - }) -} - -func TestCheckCommitHash(t *testing.T) { - testPredicate(t, CheckCommitHash, map[string]bool{ - "ff12bea91c22bba93d3ffc3034d813d686bc7eeb": true, // 40 - "eae05cb0aaeae05cb0aa": true, // 20 - "449dd6984d0eaabb": true, // 16 - "449dd6984d0e": true, // 12 - "eae05cb0aa": true, // 10 - "eae05cb0": true, // 8 - "": false, - "aa": false, - "eae05cb0aab": false, - "xxxxxxxx": false, - }) -} - -func testPredicate(t *testing.T, fn func(string) bool, tests map[string]bool) { - for input, want := range tests { - res := fn(input) - if res != want { - t.Errorf("%v: got %v, want %v", input, res, want) - } - } -} - -func TestParseReleaseTags(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 -v2.6.32 -v4.0 -voo -v1.foo -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, err := parseReleaseTags([]byte(input)) - if err != nil { - t.Fatal(err) - } - if !reflect.DeepEqual(got, want) { - t.Fatalf("got bad tags\ngot: %+v\nwant: %+v", got, want) - } -} - -func TestExtractFixTags(t *testing.T) { - commits, err := extractFixTags(strings.NewReader(extractFixTagsInput), extractFixTagsEmail) - if err != nil { - t.Fatal(err) - } - if !reflect.DeepEqual(commits, extractFixTagsOutput) { - t.Fatalf("got : %+v\twant: %+v", commits, extractFixTagsOutput) - } -} - -const extractFixTagsEmail = "\"syzbot\" " - -var extractFixTagsOutput = []FixCommit{ - {"8e4090902540da8c6e8f", "dashboard/app: bump max repros per bug to 10"}, - {"8e4090902540da8c6e8f", "executor: remove dead code"}, - {"a640a0fc325c29c3efcb", "executor: remove dead code"}, - {"8e4090902540da8c6e8fa640a0fc325c29c3efcb", "pkg/csource: fix string escaping bug"}, -} - -var extractFixTagsInput = ` -commit 73aba437a774237b1130837b856f3b40b3ec3bf0 (HEAD -> master, origin/master) -Author: me -Date: Fri Dec 22 19:59:56 2017 +0100 - - dashboard/app: bump max repros per bug to 10 - - Reported-by: syzbot+8e4090902540da8c6e8f@my.mail.com - -commit 26cd53f078db858a6ccca338e13e7f4d1d291c22 -Author: me -Date: Fri Dec 22 13:42:27 2017 +0100 - - executor: remove dead code - - Reported-by: syzbot+8e4090902540da8c6e8f@my.mail.com - Reported-by: syzbot - -commit 7b62abdb0abadbaf7b3f3a23ab4d78485fbf9059 -Author: Dmitry Vyukov -Date: Fri Dec 22 11:59:09 2017 +0100 - - pkg/csource: fix string escaping bug - - Reported-and-tested-by: syzbot+8e4090902540da8c6e8fa640a0fc325c29c3efcb@my.mail.com -` -- cgit mrf-deployment