maintner/gerrit_test.go - build - Git at Google

 // Copyright 2017 The Go Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style
 // license that can be found in the LICENSE file.

 package maintner

 import (
 	"testing"
 	"time"
 )

 var statusTests = []struct {
 	msg  string
 	want string
 }{
 	{`Create change

 Uploaded patch set 1.

 Patch-set: 1 (draft)
 Change-id: I38a08cacc17bcd9587475495111fe98f10d6875c
 Subject: test: test
 Branch: refs/heads/master
 Status: draft
 Topic:
 Commit: fee468c613a70d89f60fb5d683b0f796aabecaac`, "draft"},
 	{`Update patch set 1

 Change has been successfully cherry-picked as 117ac82c422a11e4dd5f4c14b50bafc1df840481 by Brad Fitzpatrick

 Patch-set: 1
 Status: merged
 Submission-id: 16401-1446004855021-a20b3823`, "merged"},
 	{`Create patch set 8

 Uploaded patch set 8: Patch Set 7 was rebased.

 Patch-set: 8
 Subject: devapp: initial support for App Engine Flex
 Commit: 17839a9f284b473986f235ad2757a2b445d05068
 Tag: autogenerated:gerrit:newPatchSet
 Groups: 17839a9f284b473986f235ad2757a2b445d05068`, ""},
 }

 func TestGetGerritStatus(t *testing.T) {
 	for _, tt := range statusTests {
 		gc := &GitCommit{Msg: tt.msg}
 		got := getGerritStatus(gc)
 		if got != tt.want {
 			t.Errorf("getGerritStatus msg:\n%s\ngot %s, want %s", tt.msg, got, tt.want)
 		}
 	}
 }

 var normalizeTests = []struct {
 	in  string
 	out string
 }{
 	{"foo", "foo"},
 	{"http://foo", "foo"},
 	{"upspin-review.googlesource.com", "upspin.googlesource.com"},
 	{"go-review.googlesource.com", "go.googlesource.com"},
 }

 func TestNormalizeServer(t *testing.T) {
 	for _, tt := range normalizeTests {
 		got := normalizeGerritServer(tt.in)
 		if got != tt.out {
 			t.Errorf("normalizeGerritServer(%q) = %q, want %q", tt.in, got, tt.out)
 		}
 	}
 }

 func TestGerritProject(t *testing.T) {
 	var c Corpus
 	c.EnableLeaderMode(new(dummyMutationLogger), "/fake/dir")
 	c.TrackGerrit("go.googlesource.com/build")
 	gp := c.Gerrit().Project("go-review.googlesource.com", "build")
 	if gp == nil {
 		t.Errorf("expected go-review.googlesource.com to return a project, got nil")
 	}
 	gp = c.Gerrit().Project("go-review.googlesource.com", "unknown")
 	if gp != nil {
 		t.Errorf("expected go-review.googlesource.com to return nil, got a project")
 	}
 }

 var messageTests = []struct {
 	in      string
 	want    string
 	wantNil bool
 }{
 	{
 		in: `Update patch set 1

 Patch Set 1: Code-Review+2

 Just to confirm, "go test" will consider an empty test file to be passing?

 Patch-set: 1
 Reviewer: Quentin Smith <13020@62eb7196-b449-3ce5-99f1-c037f21e1705>
 Label: Code-Review=+2
 `,
 		want: "Patch Set 1: Code-Review+2\n\nJust to confirm, \"go test\" will consider an empty test file to be passing?",
 	},
 	{
 		in: `Create change

 Uploaded patch set 1: Run-TryBot+1.

 Patch-set: 1
 Change-id: I1e0035ffba986c3551479d5742809e43da5e7c73
 Subject: runtime: fall back to small mmaps if we fail to grow reservation
 Branch: refs/heads/master
 Status: new
 Topic:
 Commit: 8776f8d725c001456037e8888a72885d46cd6744
 Tag: autogenerated:gerrit:newPatchSet
 Groups: 8776f8d725c001456037e8888a72885d46cd6744
 Reviewer: Keith Randall <5200@62eb7196-b449-3ce5-99f1-c037f21e1705>
 Reviewer: Rick Hudson <5186@62eb7196-b449-3ce5-99f1-c037f21e1705>
 Reviewer: Austin Clements <5167@62eb7196-b449-3ce5-99f1-c037f21e1705>
 Label: Run-TryBot=+1
 Private: false
 Work-in-progress: false
 `,
 		want: "Uploaded patch set 1: Run-TryBot+1.",
 	},
 	{
 		in: `Uploaded patch set 1.

 Patch-set: 1
 `,
 		wantNil: true,
 	},
 	{
 		in: `Create change

 Uploaded patch set 1.

 Patch-set: 1
 Change-id: I3799148a111f1ab6bfee24c9e03e6ebbf9e9595b
 Subject: net: make error messages consistent for invalid ports
 Branch: refs/heads/master
 Commit: 8a7de7048dc194d5e6f761add433b915beebb2e0
 Groups: 8a7de7048dc194d5e6f761add433b915beebb2e0
 `,
 		wantNil: true,
 	},
 	{
 		in: `Create patch set 7

 Uploaded patch set 7.: Commit message was updated

 Patch-set: 7
 Subject: cmd/vet: -lostcancel: check for discarded result of context.WithCancel
 Commit: 5487cc78ea332c7b49d43ef5955211387aca73bb
 Groups: 5487cc78ea332c7b49d43ef5955211387aca73bb
 `,
 		wantNil: true,
 	},
 }

 func TestGetGerritMessage(t *testing.T) {
 	var c Corpus
 	c.EnableLeaderMode(new(dummyMutationLogger), "/fake/dir")
 	c.TrackGerrit("go.googlesource.com/build")
 	gp := c.gerrit.projects["go.googlesource.com/build"]
 	for i, tt := range messageTests {
 		gc := &GitCommit{
 			Msg:        tt.in,
 			CommitTime: time.Now().UTC(),
 		}
 		msg := gp.getGerritMessage(gc)
 		if msg == nil != tt.wantNil {
 			if tt.wantNil {
 				t.Errorf("%d. getGerritMessage returned item; want nil", i)
 			} else {
 				t.Errorf("%d. getGerritMessage = nil; want a message", i)
 			}
 			continue
 		}
 		if msg == nil {
 			continue
 		}
 		// just checking these get copied through appropriately
 		if msg.Version != 1 {
 			t.Errorf("%d. getGerritMessage: want Version 1, got %d", i, msg.Version)
 		}
 		if msg.Date.IsZero() {
 			t.Errorf("%d. getGerritMessage: expected Date to be non-zero, got zero", i)
 		}
 		if msg.Message != tt.want {
 			t.Errorf("%d. getGerritMessage = %q; want %q", i, msg.Message, tt.want)
 		}
 	}
 }

 func TestOwnerNameAndID(t *testing.T) {
 	testCases := []struct {
 		cl        *GerritCL
 		OwnerID   int
 		OwnerName string
 	}{
 		{&GerritCL{}, -1, ""},
 		{&GerritCL{
 			Meta: &GitCommit{
 				Parents: []*GitCommit{
 					&GitCommit{
 						Author: &GitPerson{
 							Str: "Rick Sanchez <137@62eb7196-b449-3ce5-99f1-c037f21e1705>",
 						},
 					},
 				},
 			},
 		}, 137, "Rick Sanchez"},
 	}
 	for _, tc := range testCases {
 		if got := tc.cl.OwnerID(); got != tc.OwnerID {
 			t.Errorf("cl.OwnerID() = %d; want %d", got, tc.OwnerID)
 		}
 		if got := tc.cl.OwnerName(); got != tc.OwnerName {
 			t.Errorf("cl.OwnerName() = %q; want %q", got, tc.OwnerName)
 		}
 	}
 }

 func TestSubject(t *testing.T) {
 	cl := &GerritCL{}
 	if w, e := cl.Subject(), ""; w != e {
 		t.Errorf("cl.Subject() = %q; want %q", w, e)
 	}

 	testcases := []struct{ msg, subject string }{
 		{"maintner: slurp up all the things", "maintner: slurp up all the things"},
 		{"cmd/go: build stuff\n\nand do other stuff, too.", "cmd/go: build stuff"},
 	}
 	for _, tc := range testcases {
 		cl = &GerritCL{Commit: &GitCommit{Msg: tc.msg}}
 		if cl.Subject() != tc.subject {
 			t.Errorf("cl.Subject() = %q; want %q", cl.Subject(), tc.subject)
 		}
 	}
 }

 func TestLineValue(t *testing.T) {
 	tests := []struct {
 		all, prefix, want, wantRest string
 	}{
 		{
 			all:      "foo:  value ",
 			prefix:   "foo:",
 			want:     "value",
 			wantRest: "",
 		},
 		{
 			all:      "foo:  value\n",
 			prefix:   "foo:",
 			want:     "value",
 			wantRest: "",
 		},
 		{
 			all:      "bar: other\nfoo:  value\n",
 			prefix:   "foo:",
 			want:     "value",
 			wantRest: "",
 		},
 		{
 			all:      "notfoo: other\nfoo:  value\n",
 			prefix:   "foo:",
 			want:     "value",
 			wantRest: "",
 		},
 		{
 			all:      "Foo: bar\nLabel: Vote=+1\nLabel: Vote=+2\n",
 			prefix:   "Label: ",
 			want:     "Vote=+1",
 			wantRest: "Label: Vote=+2\n",
 		},
 		{
 			all:      "Label: Vote=+2\n",
 			prefix:   "Label: ",
 			want:     "Vote=+2",
 			wantRest: "",
 		},
 	}
 	for _, tt := range tests {
 		got, gotRest := lineValue(tt.all, tt.prefix)
 		if got != tt.want {
 			t.Errorf("lineValue(%q, %q) returned value %q; want %q", tt.all, tt.prefix, got, tt.want)
 		}
 		if gotRest != tt.wantRest {
 			t.Errorf("lineValue(%q, %q) returned rest %q; want %q", tt.all, tt.prefix, gotRest, tt.wantRest)
 		}
 	}
 }

 func TestParseGerritLabelValue(t *testing.T) {
 	tests := []struct {
 		in        string
 		wantLabel string
 		wantValue int8
 		wantWhose string
 	}{
 		{"Run-TryBot=+1", "Run-TryBot", 1, ""},
 		{"-Run-TryBot", "-Run-TryBot", 0, ""},
 		{"-TryBot-Result Gobot Gobot <5976@62eb7196-b449-3ce5-99f1-c037f21e1705>", "-TryBot-Result", 0, "5976@62eb7196-b449-3ce5-99f1-c037f21e1705"},
 		{"Run-TryBot=+1 Brad Fitzpatrick <5065@62eb7196-b449-3ce5-99f1-c037f21e1705>", "Run-TryBot", 1, "5065@62eb7196-b449-3ce5-99f1-c037f21e1705"},
 		{"TryBot-Result=-1 Gobot Gobot <5976@62eb7196-b449-3ce5-99f1-c037f21e1705>", "TryBot-Result", -1, "5976@62eb7196-b449-3ce5-99f1-c037f21e1705"},
 	}
 	for _, tt := range tests {
 		label, value, whose := parseGerritLabelValue(tt.in)
 		if label != tt.wantLabel || value != tt.wantValue || whose != tt.wantWhose {
 			t.Errorf("parseGerritLabelValue(%q) = %q, %v, %q; want %q, %v, %q",
 				tt.in,
 				label, value, whose,
 				tt.wantLabel, tt.wantValue, tt.wantWhose)
 		}
 	}
 }
	// Copyright 2017 The Go Authors. All rights reserved.
	// Use of this source code is governed by a BSD-style
	// license that can be found in the LICENSE file.

	package maintner

	import (
	"testing"
	"time"
	)

	var statusTests = []struct {
	msg string
	want string
	}{
	{`Create change

	Uploaded patch set 1.

	Patch-set: 1 (draft)
	Change-id: I38a08cacc17bcd9587475495111fe98f10d6875c
	Subject: test: test
	Branch: refs/heads/master
	Status: draft
	Topic:
	Commit: fee468c613a70d89f60fb5d683b0f796aabecaac`, "draft"},
	{`Update patch set 1

	Change has been successfully cherry-picked as 117ac82c422a11e4dd5f4c14b50bafc1df840481 by Brad Fitzpatrick

	Patch-set: 1
	Status: merged
	Submission-id: 16401-1446004855021-a20b3823`, "merged"},
	{`Create patch set 8

	Uploaded patch set 8: Patch Set 7 was rebased.

	Patch-set: 8
	Subject: devapp: initial support for App Engine Flex
	Commit: 17839a9f284b473986f235ad2757a2b445d05068
	Tag: autogenerated:gerrit:newPatchSet
	Groups: 17839a9f284b473986f235ad2757a2b445d05068`, ""},
	}

	func TestGetGerritStatus(t *testing.T) {
	for _, tt := range statusTests {
	gc := &GitCommit{Msg: tt.msg}
	got := getGerritStatus(gc)
	if got != tt.want {
	t.Errorf("getGerritStatus msg:\n%s\ngot %s, want %s", tt.msg, got, tt.want)
	}
	}
	}

	var normalizeTests = []struct {
	in string
	out string
	}{
	{"foo", "foo"},
	{"http://foo", "foo"},
	{"upspin-review.googlesource.com", "upspin.googlesource.com"},
	{"go-review.googlesource.com", "go.googlesource.com"},
	}

	func TestNormalizeServer(t *testing.T) {
	for _, tt := range normalizeTests {
	got := normalizeGerritServer(tt.in)
	if got != tt.out {
	t.Errorf("normalizeGerritServer(%q) = %q, want %q", tt.in, got, tt.out)
	}
	}
	}

	func TestGerritProject(t *testing.T) {
	var c Corpus
	c.EnableLeaderMode(new(dummyMutationLogger), "/fake/dir")
	c.TrackGerrit("go.googlesource.com/build")
	gp := c.Gerrit().Project("go-review.googlesource.com", "build")
	if gp == nil {
	t.Errorf("expected go-review.googlesource.com to return a project, got nil")
	}
	gp = c.Gerrit().Project("go-review.googlesource.com", "unknown")
	if gp != nil {
	t.Errorf("expected go-review.googlesource.com to return nil, got a project")
	}
	}

	var messageTests = []struct {
	in string
	want string
	wantNil bool
	}{
	{
	in: `Update patch set 1

	Patch Set 1: Code-Review+2

	Just to confirm, "go test" will consider an empty test file to be passing?

	Patch-set: 1
	Reviewer: Quentin Smith <13020@62eb7196-b449-3ce5-99f1-c037f21e1705>
	Label: Code-Review=+2
	`,
	want: "Patch Set 1: Code-Review+2\n\nJust to confirm, \"go test\" will consider an empty test file to be passing?",
	},
	{
	in: `Create change

	Uploaded patch set 1: Run-TryBot+1.

	Patch-set: 1
	Change-id: I1e0035ffba986c3551479d5742809e43da5e7c73
	Subject: runtime: fall back to small mmaps if we fail to grow reservation
	Branch: refs/heads/master
	Status: new
	Topic:
	Commit: 8776f8d725c001456037e8888a72885d46cd6744
	Tag: autogenerated:gerrit:newPatchSet
	Groups: 8776f8d725c001456037e8888a72885d46cd6744
	Reviewer: Keith Randall <5200@62eb7196-b449-3ce5-99f1-c037f21e1705>
	Reviewer: Rick Hudson <5186@62eb7196-b449-3ce5-99f1-c037f21e1705>
	Reviewer: Austin Clements <5167@62eb7196-b449-3ce5-99f1-c037f21e1705>
	Label: Run-TryBot=+1
	Private: false
	Work-in-progress: false
	`,
	want: "Uploaded patch set 1: Run-TryBot+1.",
	},
	{
	in: `Uploaded patch set 1.

	Patch-set: 1
	`,
	wantNil: true,
	},
	{
	in: `Create change

	Uploaded patch set 1.

	Patch-set: 1
	Change-id: I3799148a111f1ab6bfee24c9e03e6ebbf9e9595b
	Subject: net: make error messages consistent for invalid ports
	Branch: refs/heads/master
	Commit: 8a7de7048dc194d5e6f761add433b915beebb2e0
	Groups: 8a7de7048dc194d5e6f761add433b915beebb2e0
	`,
	wantNil: true,
	},
	{
	in: `Create patch set 7

	Uploaded patch set 7.: Commit message was updated

	Patch-set: 7
	Subject: cmd/vet: -lostcancel: check for discarded result of context.WithCancel
	Commit: 5487cc78ea332c7b49d43ef5955211387aca73bb
	Groups: 5487cc78ea332c7b49d43ef5955211387aca73bb
	`,
	wantNil: true,
	},
	}

	func TestGetGerritMessage(t *testing.T) {
	var c Corpus
	c.EnableLeaderMode(new(dummyMutationLogger), "/fake/dir")
	c.TrackGerrit("go.googlesource.com/build")
	gp := c.gerrit.projects["go.googlesource.com/build"]
	for i, tt := range messageTests {
	gc := &GitCommit{
	Msg: tt.in,
	CommitTime: time.Now().UTC(),
	}
	msg := gp.getGerritMessage(gc)
	if msg == nil != tt.wantNil {
	if tt.wantNil {
	t.Errorf("%d. getGerritMessage returned item; want nil", i)
	} else {
	t.Errorf("%d. getGerritMessage = nil; want a message", i)
	}
	continue
	}
	if msg == nil {
	continue
	}
	// just checking these get copied through appropriately
	if msg.Version != 1 {
	t.Errorf("%d. getGerritMessage: want Version 1, got %d", i, msg.Version)
	}
	if msg.Date.IsZero() {
	t.Errorf("%d. getGerritMessage: expected Date to be non-zero, got zero", i)
	}
	if msg.Message != tt.want {
	t.Errorf("%d. getGerritMessage = %q; want %q", i, msg.Message, tt.want)
	}
	}
	}

	func TestOwnerNameAndID(t *testing.T) {
	testCases := []struct {
	cl *GerritCL
	OwnerID int
	OwnerName string
	}{
	{&GerritCL{}, -1, ""},
	{&GerritCL{
	Meta: &GitCommit{
	Parents: []*GitCommit{
	&GitCommit{
	Author: &GitPerson{
	Str: "Rick Sanchez <137@62eb7196-b449-3ce5-99f1-c037f21e1705>",
	},
	},
	},
	},
	}, 137, "Rick Sanchez"},
	}
	for _, tc := range testCases {
	if got := tc.cl.OwnerID(); got != tc.OwnerID {
	t.Errorf("cl.OwnerID() = %d; want %d", got, tc.OwnerID)
	}
	if got := tc.cl.OwnerName(); got != tc.OwnerName {
	t.Errorf("cl.OwnerName() = %q; want %q", got, tc.OwnerName)
	}
	}
	}

	func TestSubject(t *testing.T) {
	cl := &GerritCL{}
	if w, e := cl.Subject(), ""; w != e {
	t.Errorf("cl.Subject() = %q; want %q", w, e)
	}

	testcases := []struct{ msg, subject string }{
	{"maintner: slurp up all the things", "maintner: slurp up all the things"},
	{"cmd/go: build stuff\n\nand do other stuff, too.", "cmd/go: build stuff"},
	}
	for _, tc := range testcases {
	cl = &GerritCL{Commit: &GitCommit{Msg: tc.msg}}
	if cl.Subject() != tc.subject {
	t.Errorf("cl.Subject() = %q; want %q", cl.Subject(), tc.subject)
	}
	}
	}

	func TestLineValue(t *testing.T) {
	tests := []struct {
	all, prefix, want, wantRest string
	}{
	{
	all: "foo: value ",
	prefix: "foo:",
	want: "value",
	wantRest: "",
	},
	{
	all: "foo: value\n",
	prefix: "foo:",
	want: "value",
	wantRest: "",
	},
	{
	all: "bar: other\nfoo: value\n",
	prefix: "foo:",
	want: "value",
	wantRest: "",
	},
	{
	all: "notfoo: other\nfoo: value\n",
	prefix: "foo:",
	want: "value",
	wantRest: "",
	},
	{
	all: "Foo: bar\nLabel: Vote=+1\nLabel: Vote=+2\n",
	prefix: "Label: ",
	want: "Vote=+1",
	wantRest: "Label: Vote=+2\n",
	},
	{
	all: "Label: Vote=+2\n",
	prefix: "Label: ",
	want: "Vote=+2",
	wantRest: "",
	},
	}
	for _, tt := range tests {
	got, gotRest := lineValue(tt.all, tt.prefix)
	if got != tt.want {
	t.Errorf("lineValue(%q, %q) returned value %q; want %q", tt.all, tt.prefix, got, tt.want)
	}
	if gotRest != tt.wantRest {
	t.Errorf("lineValue(%q, %q) returned rest %q; want %q", tt.all, tt.prefix, gotRest, tt.wantRest)
	}
	}
	}

	func TestParseGerritLabelValue(t *testing.T) {
	tests := []struct {
	in string
	wantLabel string
	wantValue int8
	wantWhose string
	}{
	{"Run-TryBot=+1", "Run-TryBot", 1, ""},
	{"-Run-TryBot", "-Run-TryBot", 0, ""},
	{"-TryBot-Result Gobot Gobot <5976@62eb7196-b449-3ce5-99f1-c037f21e1705>", "-TryBot-Result", 0, "5976@62eb7196-b449-3ce5-99f1-c037f21e1705"},
	{"Run-TryBot=+1 Brad Fitzpatrick <5065@62eb7196-b449-3ce5-99f1-c037f21e1705>", "Run-TryBot", 1, "5065@62eb7196-b449-3ce5-99f1-c037f21e1705"},
	{"TryBot-Result=-1 Gobot Gobot <5976@62eb7196-b449-3ce5-99f1-c037f21e1705>", "TryBot-Result", -1, "5976@62eb7196-b449-3ce5-99f1-c037f21e1705"},
	}
	for _, tt := range tests {
	label, value, whose := parseGerritLabelValue(tt.in)
	if label != tt.wantLabel \|\| value != tt.wantValue \|\| whose != tt.wantWhose {
	t.Errorf("parseGerritLabelValue(%q) = %q, %v, %q; want %q, %v, %q",
	tt.in,
	label, value, whose,
	tt.wantLabel, tt.wantValue, tt.wantWhose)
	}
	}
	}