Andreas/watchtower
1
0
Fork 0
mirror of https://github.com/containrrr/watchtower.git synced 2025-12-16 23:20:12 +01:00
Code Issues Projects Releases Packages Wiki Activity Actions

feature/112: add additional tests that verify include-stopped

Browse source
This commit is contained in:
Simon Aronsson 2019-04-28 15:40:07 +02:00
parent 46ffa16ee2
commit 3c2e184823
1 changed files with 23 additions and 10 deletions
Download patch file Download diff file Expand all files Collapse all files

33
container/container_test.go
View file

@ -17,15 +17,16 @@ func TestContainer(t *testing.T) {
var _ = Describe("the container", func() { var _ = Describe("the container", func() {
Describe("the client", func() { Describe("the client", func() {
var docker *cli.Client
var client Client var client Client
BeforeSuite(func() { BeforeSuite(func() {
server := mocks.NewMockAPIServer() server := mocks.NewMockAPIServer()
c, _ := cli.NewClientWithOpts( docker, _ = cli.NewClientWithOpts(
cli.WithHost(server.URL), cli.WithHost(server.URL),
cli.WithHTTPClient(server.Client(), cli.WithHTTPClient(server.Client(),
)) ))
client = dockerClient{ client = dockerClient{
api: c, api: docker,
pullImages: false, pullImages: false,
} }
}) })
@ -41,7 +42,7 @@ var _ = Describe("the container", func() {
}) })
When("listing containers with a filter matching nothing", func() { When("listing containers with a filter matching nothing", func() {
It("should return an empty array", func() { It("should return an empty array", func() {
filter := filterByNames([]string { "lollercoaster"}, noFilter) filter := filterByNames([]string{"lollercoaster"}, noFilter)
containers, err := client.ListContainers(filter) containers, err := client.ListContainers(filter)
Expect(err).NotTo(HaveOccurred()) Expect(err).NotTo(HaveOccurred())
Expect(len(containers) == 0).To(BeTrue()) Expect(len(containers) == 0).To(BeTrue())
@ -55,13 +56,25 @@ var _ = Describe("the container", func() {
Expect(containers[0].ImageName()).To(Equal("containrrr/watchtower:latest")) Expect(containers[0].ImageName()).To(Equal("containrrr/watchtower:latest"))
}) })
}) })
When(`listing containers with the "include stopped" option`, func() {
It("should return both stopped and running containers", func() {
client = dockerClient{
api: docker,
pullImages: false,
includeStopped: true,
}
containers, err := client.ListContainers(noFilter)
Expect(err).NotTo(HaveOccurred())
Expect(len(containers) > 0).To(BeTrue())
})
})
}) })
When("asked for metadata", func() { When("asked for metadata", func() {
var c *Container var c *Container
BeforeEach(func() { BeforeEach(func() {
c = mockContainerWithLabels(map[string]string { c = mockContainerWithLabels(map[string]string{
"com.centurylinklabs.watchtower.enable": "true", "com.centurylinklabs.watchtower.enable": "true",
"com.centurylinklabs.watchtower": "true", "com.centurylinklabs.watchtower": "true",
}) })
}) })
It("should return its name on calls to .Name()", func() { It("should return its name on calls to .Name()", func() {
@ -84,7 +97,7 @@ var _ = Describe("the container", func() {
Expect(exists).NotTo(BeFalse()) Expect(exists).NotTo(BeFalse())
}) })
It("should return false, true if present but not true on calls to .Enabled()", func() { It("should return false, true if present but not true on calls to .Enabled()", func() {
c = mockContainerWithLabels(map[string]string{ "com.centurylinklabs.watchtower.enable": "false" }) c = mockContainerWithLabels(map[string]string{"com.centurylinklabs.watchtower.enable": "false"})
enabled, exists := c.Enabled() enabled, exists := c.Enabled()
Expect(enabled).To(BeFalse()) Expect(enabled).To(BeFalse())
@ -93,7 +106,7 @@ var _ = Describe("the container", func() {
Expect(exists).NotTo(BeFalse()) Expect(exists).NotTo(BeFalse())
}) })
It("should return false, false if not present on calls to .Enabled()", func() { It("should return false, false if not present on calls to .Enabled()", func() {
c = mockContainerWithLabels(map[string]string{ "lol": "false" }) c = mockContainerWithLabels(map[string]string{"lol": "false"})
enabled, exists := c.Enabled() enabled, exists := c.Enabled()
Expect(enabled).To(BeFalse()) Expect(enabled).To(BeFalse())
@ -102,7 +115,7 @@ var _ = Describe("the container", func() {
Expect(exists).NotTo(BeTrue()) Expect(exists).NotTo(BeTrue())
}) })
It("should return false, false if present but not parsable .Enabled()", func() { It("should return false, false if present but not parsable .Enabled()", func() {
c = mockContainerWithLabels(map[string]string{ "com.centurylinklabs.watchtower.enable": "falsy" }) c = mockContainerWithLabels(map[string]string{"com.centurylinklabs.watchtower.enable": "falsy"})
enabled, exists := c.Enabled() enabled, exists := c.Enabled()
Expect(enabled).To(BeFalse()) Expect(enabled).To(BeFalse())
@ -116,12 +129,12 @@ var _ = Describe("the container", func() {
Expect(isWatchtower).To(BeTrue()) Expect(isWatchtower).To(BeTrue())
}) })
It("should return false if the label is present but set to false", func() { It("should return false if the label is present but set to false", func() {
c = mockContainerWithLabels(map[string]string{ "com.centurylinklabs.watchtower": "false" }) c = mockContainerWithLabels(map[string]string{"com.centurylinklabs.watchtower": "false"})
isWatchtower := c.IsWatchtower() isWatchtower := c.IsWatchtower()
Expect(isWatchtower).To(BeFalse()) Expect(isWatchtower).To(BeFalse())
}) })
It("should return false if the label is not present", func() { It("should return false if the label is not present", func() {
c = mockContainerWithLabels(map[string]string{ "funny.label": "false" }) c = mockContainerWithLabels(map[string]string{"funny.label": "false"})
isWatchtower := c.IsWatchtower() isWatchtower := c.IsWatchtower()
Expect(isWatchtower).To(BeFalse()) Expect(isWatchtower).To(BeFalse())
}) })