fix: fixing Nullify() DeployNumber statment to actually work (#351)

* fix: fixing Nullify() DeployNumber statment to actually work * fix: changing test to work with new Nullify() DeployNumber statement --------- Co-authored-by: Claire.Nicholas <Z00B3R3@w6nxmk4t6t.target.com>
go-vela · Feb 1, 2024 · 867f2d1 · 867f2d1
1 parent 86cbe53
commit 867f2d1
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/database/build.go b/database/build.go
@@ -173,7 +173,7 @@ func (b *Build) Nullify() *Build {
 
 	// check if the DeployNumber field should be false
 	if b.DeployNumber.Int64 == 0 {
-		b.Deploy.Valid = false
+		b.DeployNumber.Valid = false
 	}
 
 	// check if the Clone field should be false

diff --git a/database/build_test.go b/database/build_test.go
@@ -276,7 +276,7 @@ func testBuild() *Build {
 		Started:       sql.NullInt64{Int64: 1563474078, Valid: true},
 		Finished:      sql.NullInt64{Int64: 1563474079, Valid: true},
 		Deploy:        sql.NullString{String: "", Valid: false},
-		DeployNumber:  sql.NullInt64{Int64: 0, Valid: true},
+		DeployNumber:  sql.NullInt64{Int64: 0, Valid: false},
 		DeployPayload: raw.StringSliceMap{"foo": "test1", "bar": "test2"},
 		Clone:         sql.NullString{String: "https://github.com/github/octocat.git", Valid: true},
 		Source:        sql.NullString{String: "https://github.com/github/octocat/48afb5bdc41ad69bf22588491333f7cf71135163", Valid: true},