Skip to content
This repository was archived by the owner on Mar 27, 2025. It is now read-only.

Conversation

@guerinoni
Copy link
Contributor

With this PR will cover all API about docker.

Fabio Falzoi and others added 20 commits August 6, 2020 11:17
Signed-off-by: Fabio Falzoi <fabio@develer.com>
Signed-off-by: Fabio Falzoi <fabio@develer.com>
Signed-off-by: Fabio Falzoi <fabio@develer.com>
Signed-off-by: Fabio Falzoi <fabio@develer.com>
Signed-off-by: Fabio Falzoi <fabio@develer.com>
Signed-off-by: Fabio Falzoi <fabio@develer.com>
I prefer to remove bool value and call error func directly inside
SendInfo, in this way we avoid to check return value of SendInfo.

Signed-off-by: Federico Guerinoni <fguerinoni@develer.com>
Signed-off-by: Federico Guerinoni <fguerinoni@develer.com>
This allow us to create a mock test of struct Token.

Signed-off-by: Federico Guerinoni <fguerinoni@develer.com>
With this test I mock the mqtt client and token in order to check
what application sends in case of an error when publish.

Signed-off-by: Federico Guerinoni <fguerinoni@develer.com>
Signed-off-by: Federico Guerinoni <fguerinoni@develer.com>
Signed-off-by: Federico Guerinoni <fguerinoni@develer.com>
Signed-off-by: Federico Guerinoni <fguerinoni@develer.com>
In this case I prefer this check because we can ignore if other
containers are running and is more appropriated then check len of list
of containers.

Signed-off-by: Federico Guerinoni <fguerinoni@develer.com>
Signed-off-by: Federico Guerinoni <fguerinoni@develer.com>
Signed-off-by: Federico Guerinoni <fguerinoni@develer.com>
Signed-off-by: Federico Guerinoni <fguerinoni@develer.com>
Signed-off-by: Federico Guerinoni <fguerinoni@develer.com>
Signed-off-by: Federico Guerinoni <fguerinoni@develer.com>
@guerinoni guerinoni requested review from pippolo84 and rsora August 6, 2020 20:18
@guerinoni guerinoni force-pushed the additional-docker-functional-tests branch from 822b128 to 1a19e64 Compare August 7, 2020 08:12
Signed-off-by: Federico Guerinoni <fguerinoni@develer.com>
@guerinoni guerinoni force-pushed the additional-docker-functional-tests branch from 1a19e64 to 3e1f293 Compare August 7, 2020 09:28
While executing tests in GitHub Actions environment, a lot of
containers are started, so the prune operation can take a very long
time.
To avoid failing the test due to the high number of containers to
prune, we use a very large timeout value.

Signed-off-by: Fabio Falzoi <fabio@develer.com>
@pippolo84 pippolo84 force-pushed the additional-docker-functional-tests branch from 647505d to a3fb080 Compare August 7, 2020 16:14
@guerinoni guerinoni merged commit 971c1a3 into master Aug 11, 2020
@guerinoni guerinoni deleted the additional-docker-functional-tests branch August 11, 2020 08:45
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant