summaryrefslogtreecommitdiff
path: root/pkg/domain/infra/abi/images_test.go
diff options
context:
space:
mode:
authorColin Walters <walters@verbum.org>2022-01-19 19:36:45 -0500
committerColin Walters <walters@verbum.org>2022-01-20 18:02:58 -0500
commit2947ded7a8fced28fb3478f556194d12433145ad (patch)
treea2900ec5e478f15c31ee8270a75249d65b42a00f /pkg/domain/infra/abi/images_test.go
parent6b59b10248b0bd92127dfc455ca40579d4c396a0 (diff)
downloadpodman-2947ded7a8fced28fb3478f556194d12433145ad.tar.gz
podman-2947ded7a8fced28fb3478f556194d12433145ad.tar.bz2
podman-2947ded7a8fced28fb3478f556194d12433145ad.zip
Don't segfault if an image layer has no creation timestamp
It's optional in the specification, and I initially omitted it in the ostree code. Now I've fixed the ostree code to inject a timestamp, but we should clearly avoid segfaulting on this case. Signed-off-by: Colin Walters <walters@verbum.org>
Diffstat (limited to 'pkg/domain/infra/abi/images_test.go')
-rw-r--r--pkg/domain/infra/abi/images_test.go17
1 files changed, 17 insertions, 0 deletions
diff --git a/pkg/domain/infra/abi/images_test.go b/pkg/domain/infra/abi/images_test.go
index 20ef1b150..e38b9390d 100644
--- a/pkg/domain/infra/abi/images_test.go
+++ b/pkg/domain/infra/abi/images_test.go
@@ -1,5 +1,22 @@
package abi
+import (
+ "testing"
+
+ "github.com/containers/common/libimage"
+ "github.com/stretchr/testify/assert"
+)
+
+// This is really intended to verify what happens with a
+// nil pointer in layer.Created, but we'll just sanity
+// check round tripping 42.
+func TestToDomainHistoryLayer(t *testing.T) {
+ var layer libimage.ImageHistory
+ layer.Size = 42
+ newLayer := toDomainHistoryLayer(&layer)
+ assert.Equal(t, layer.Size, newLayer.Size)
+}
+
//
// import (
// "context"