fix(secrets): actually set updated_at and updated_by fields (#578)

* bug fix for secret metadata updated by field

* make sure updated at gets same treatment

secret/native/update.go

@@ -62,5 +62,11 @@ func (c *client) Update(sType, org, name string, s *library.Secret) error {
 		sec.SetAllowCommand(s.GetAllowCommand())
 	}
 
+	// update updated_at if set
+	sec.SetUpdatedAt(s.GetUpdatedAt())
+
+	// update updated_by if set
+	sec.SetUpdatedBy(s.GetUpdatedBy())
+
 	return c.Database.UpdateSecret(sec)
 }

secret/native/update_test.go

@@ -15,6 +15,22 @@ import (
 
 func TestNative_Update(t *testing.T) {
 	// setup types
+	original := new(library.Secret)
+	original.SetID(1)
+	original.SetOrg("foo")
+	original.SetRepo("bar")
+	original.SetTeam("")
+	original.SetName("baz")
+	original.SetValue("secretValue")
+	original.SetType("repo")
+	original.SetImages([]string{"foo", "baz"})
+	original.SetEvents([]string{"foob", "bar"})
+	original.SetAllowCommand(true)
+	original.SetCreatedAt(1)
+	original.SetCreatedBy("user")
+	original.SetUpdatedAt(1)
+	original.SetUpdatedBy("user")
+
 	want := new(library.Secret)
 	want.SetID(1)
 	want.SetOrg("foo")
@@ -40,7 +56,7 @@ func TestNative_Update(t *testing.T) {
 		_sql.Close()
 	}()
 
-	_ = db.CreateSecret(want)
+	_ = db.CreateSecret(original)
 
 	// run test
 	s, err := New(