-
Notifications
You must be signed in to change notification settings - Fork 18.8k
c8d/integration-cli: Skip part of TestListDanglingImagesWithDigests #46868
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
c8d/integration-cli: Skip part of TestListDanglingImagesWithDigests #46868
Conversation
| @@ -341,6 +341,13 @@ func (s *DockerRegistrySuite) TestListDanglingImagesWithDigests(c *testing.T) { | |||
| // make sure repo shown, tag=<none>, digest = $digest1 | |||
| re1 := regexp.MustCompile(`\s*` + repoName + `\s*<none>\s*` + digest1.String() + `\s`) | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this case already covered by TestListImagesWithDigests above?
If it is, it may be cleaner to t.Skip() the whole test
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe this one test is actually multiple sub-tests 🤔 we could make sub-tests and skip those, but I guess we'd have to (temporarily) disable the "integration-cli is deprecated" check.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, looks the same.
Let's just skip the whole test.
96797e1 to
308825c
Compare
| @@ -341,6 +344,7 @@ func (s *DockerRegistrySuite) TestListDanglingImagesWithDigests(c *testing.T) { | |||
| // make sure repo shown, tag=<none>, digest = $digest1 | |||
| re1 := regexp.MustCompile(`\s*` + repoName + `\s*<none>\s*` + digest1.String() + `\s`) | |||
| assert.Assert(c, re1.MatchString(out), "expected %q: %s", re1.String(), out) | |||
|
|
|||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
oh! unrelated change (if you still have the branch open locally)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed, sorry! 😄
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No worries! Thought you still had the branch at hand, so would be easy to fix (and wouldn't have been a massive disaster if it was left)
Skip TestListDanglingImagesWithDigests which tests graphdriver implementation specific behavior of `docker images --filter dangling=true`. Signed-off-by: Paweł Gronowski <pawel.gronowski@docker.com>
308825c to
fcb89da
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
danglingfilter behavior #46856- What I did
Skip part of TestListDanglingImagesWithDigests which tests graphdriver implementation specific behavior of
docker images --filter dangling=true.- How I did it
- How to verify it
Check TestListDanglingImagesWithDigests test result
- Description for the changelog
- A picture of a cute animal (not mandatory but encouraged)