weiguoqiang/cert-manager
1
0
Fork 0
Code Issues Pull Requests Actions 2 Packages Projects Releases Wiki Activity

Adds a unit test to ensure serializing preserves CR spec in round trip

Browse Source 
Signed-off-by: joshvanl <vleeuwenjoshua@gmail.com>
This commit is contained in:
joshvanl 2022-02-07 11:11:44 +00:00
parent da67eb2b65
commit 07c243df2d
2 changed files with 8 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
internal/controller/certificaterequests/apply.go
View File

@ -1,5 +1,5 @@
/*
Copyright 2020 The cert-manager Authors.
Copyright 2022 The cert-manager Authors.
Licensed under the Apache License, Version 2.0 (the "License");
you may not use this file except in compliance with the License.

8
internal/controller/certificaterequests/apply_test.go
View File

@ -1,5 +1,5 @@
/*
Copyright 2020 The cert-manager Authors.
Copyright 2022 The cert-manager Authors.
Licensed under the Apache License, Version 2.0 (the "License");
you may not use this file except in compliance with the License.
@ -17,6 +17,7 @@ limitations under the License.
package certificaterequests
import (
"encoding/json"
"strconv"
"sync"
"testing"
@ -56,6 +57,11 @@ func Test_serializeApply(t *testing.T) {
assert.NoError(t, err)
assert.Regexp(t, expReg, string(reqData))
// Test round trip preserves the spec and object meta.
var rtReq cmapi.CertificateRequest
assert.NoError(t, json.Unmarshal(reqData, &rtReq))
assert.Equal(t, req.Spec, rtReq.Spec)
// String match on empty spec.
req.Spec = cmapi.CertificateRequestSpec{}
reqData, err = serializeApply(&req)