unit tests on commands about cli HOT 10 CLOSED

We may need to move some of the fakeClient into cli/internal/test/ and export the struct and fields. There are some packages (stack, and service at least) that use parts of other clients.

from cli.

Some items on the list seem to be deprecated as they do not appear when setting the DOCKER_HIDE_LEGACY_COMMANDS variable:

• bundlefile
• inspect
• prune
• registry
• task

Should we remove them from the list ?

(cc @kelu27)

from cli.

@jeremcs I would say not (because some of them won't ever be removed).

from cli.

Starting working on service (#201)

from cli.

#dibs on task package

from cli.

#dibs on checkpoint package.

from cli.

hi @vdemeester ,
I was working on prune test in the last hackergarten but I saw the commit two days ago which modify the prune.go file and add a prune_test.go too.
I believe this part is ok or is there something else to do?

from cli.

@Zebrilee oh right yeah, I should mark it on the issue

from cli.

#dibs on network package.

from cli.

Closing this as all those packages have some test. We can enhance the coverage but I feel this issue can be considered as done.

from cli.