pr feedback comments

Author: aswanidutt

dependency/vault_common_test.go

@@ -24,8 +24,8 @@ func TestVaultRenewDuration(t *testing.T) {
 	if err != nil {
 		t.Fatal(err)
 	}
-	if renewableDur.Seconds() < 16 || renewableDur.Seconds() >= 34 {
-		t.Fatalf("renewable duration is not within 1/6 to 1/3 of lease duration: %f", renewableDur.Seconds())
+	if renewableDur < 16*time.Second || renewableDur >= 34*time.Second {
+		t.Fatalf("renewable duration is not within 1/6 to 1/3 of lease duration: %s", renewableDur)
 	}
 
 	nonRenewable := Secret{LeaseDuration: 100}