Set scm.PR.Sha to scm.PR.Head.Sha when loading from disk.

This guarantee is made when retrieving a PR from a provider, but we do not persist that guarantee when roundtripping from disk.
tektoncd · Dec 6, 2019 · fad2bef · fad2bef
1 parent c5fdfb5
commit fad2bef
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 0 deletions.
diff --git a/cmd/pullrequest-init/disk.go b/cmd/pullrequest-init/disk.go
@@ -201,6 +201,8 @@ func FromDisk(path string) (*Resource, error) {
 		return nil, err
 	}
 
+	r.PR.Sha = r.PR.Head.Sha
+
 	return r, nil
 }
 

diff --git a/cmd/pullrequest-init/disk_test.go b/cmd/pullrequest-init/disk_test.go
@@ -310,6 +310,9 @@ func TestFromDisk(t *testing.T) {
 	if diff := cmp.Diff(rsrc.PR.Head, head); diff != "" {
 		t.Errorf("Get Head: -want +got: %s", diff)
 	}
+	if diff := cmp.Diff(rsrc.PR.Sha, head.Sha); diff != "" {
+		t.Errorf("Get Sha: -want +got: %s", diff)
+	}
 
 	// Check the comments
 	commentMap := map[int]scm.Comment{}