diff options
author | Matej Vasek <mvasek@redhat.com> | 2021-11-01 22:53:49 +0100 |
---|---|---|
committer | Matej Vasek <mvasek@redhat.com> | 2021-11-01 23:01:17 +0100 |
commit | d0dfc5e2234a3567addd8163a0bd6a3f03d740cc (patch) | |
tree | 86897cbce7f2199dd73f214d58ca9d8824194f14 /pkg | |
parent | 48e1cca9f8f6a9f50e317214bd87e5dfa57d9ab4 (diff) | |
download | podman-d0dfc5e2234a3567addd8163a0bd6a3f03d740cc.tar.gz podman-d0dfc5e2234a3567addd8163a0bd6a3f03d740cc.tar.bz2 podman-d0dfc5e2234a3567addd8163a0bd6a3f03d740cc.zip |
Add tests
Signed-off-by: Matej Vasek <mvasek@redhat.com>
Diffstat (limited to 'pkg')
-rw-r--r-- | pkg/api/handlers/utils/handler_test.go | 48 |
1 files changed, 48 insertions, 0 deletions
diff --git a/pkg/api/handlers/utils/handler_test.go b/pkg/api/handlers/utils/handler_test.go index 18a1d2678..5957e7d74 100644 --- a/pkg/api/handlers/utils/handler_test.go +++ b/pkg/api/handlers/utils/handler_test.go @@ -138,3 +138,51 @@ func TestEqualVersion(t *testing.T) { rr.Body.String(), expected) } } + +func TestErrorEncoderFuncOmit(t *testing.T) { + data, err := json.Marshal(struct { + Err error `json:"err,omitempty"` + Errs []error `json:"errs,omitempty"` + }{}) + if err != nil { + t.Fatal(err) + } + + dataAsMap := make(map[string]interface{}) + err = json.Unmarshal(data, &dataAsMap) + if err != nil { + t.Fatal(err) + } + + _, ok := dataAsMap["err"] + if ok { + t.Errorf("the `err` field should have been omitted") + } + _, ok = dataAsMap["errs"] + if ok { + t.Errorf("the `errs` field should have been omitted") + } + + dataAsMap = make(map[string]interface{}) + data, err = json.Marshal(struct { + Err error `json:"err"` + Errs []error `json:"errs"` + }{}) + if err != nil { + t.Fatal(err) + } + + err = json.Unmarshal(data, &dataAsMap) + if err != nil { + t.Fatal(err) + } + + _, ok = dataAsMap["err"] + if !ok { + t.Errorf("the `err` field shouldn't have been omitted") + } + _, ok = dataAsMap["errs"] + if !ok { + t.Errorf("the `errs` field shouldn't have been omitted") + } +} |