Add end to end tests to the "showCommandCmdF" mmctl command

[mgdelacroix]

Mattermost is increasing the end to end test coverage for the mmctl CLI tool. The goal is to create a suite of tests that check the main logic paths of each command and ensure that they work against a running server.

This Help Wanted issue covers the addition of end to end tests to the showCommandCmdF command, in the commands/command.go file. The expected way to add these tests is to create a test function in the commands/command_e2e_test.go file and add one s.Run block for each possible test case.

Thanks to the good amount of already existing unit tests that the commands have, it is not necessary to write an end to end test for every possible logic path of the command; the main focus should be in covering the happy path, the error path and the cases where the user doesn't have permissions to perform an action.

To help with the test process, there are some helpers that can be used. The s.SetupTestHelper().InitBasic() chain will create a set of default users, teams and channels to be used during the tests, s.RunForAllClients is useful when a test case should be valid for a sysadmin user, a local mode user and an unprivileged user, and s.RunForSystemAdminAndLocal will test a block for both a sysadmin user and a local mode user. Please take a look at the example pull request to see how all these work.

Example: mattermost/mmctl#202

For instructions on how to set up the end to end tests, take a look at the corresponding section of the mmctl README.

If you have questions about the ticket or you need any help, feel free to post your question in the community CLI channel or contact miguel.delacruz in https://community.mattermost.com/

---

If you're interested please comment here and come join our "Contributors" community channel on our daily build server, where you can discuss questions with community members and the Mattermost core team. For technical advice or questions, please join our "Developers" community channel.

New contributors please see our Developer's Guide.

JIRA: https://mattermost.atlassian.net/browse/MM-29565

[djanda97]

I would like to give this issue a shot!

[mgdelacroix]

Sure @djanda97, thanks for taking a look at the issue!!

[djanda97]

I am having some trouble building mmctl. When I cloned a fresh copy and ran make build inside the mmctl directory this is the output I receive in the terminal:
go mod vendor
go mod tidy
Running gofmt
Checking github.com/mattermost/mmctl
Checking github.com/mattermost/mmctl/client
Checking github.com/mattermost/mmctl/commands
Checking github.com/mattermost/mmctl/mocks
Checking github.com/mattermost/mmctl/printer
Gofmt success
Running golangci-lint
golangci-lint run ./...
WARN [runner/nolint] Found unknown linters in //nolint directives: ifelsechain
commands/group_test.go:629:3: commentFormatting: put a space between//and comment text (gocritic)
//grp1 := model.GroupWithSchemeAdmin{}
^
commands/init.go:113:22: G402: TLS MinVersion too low. (gosec)
TLSClientConfig: &tls.Config{
VerifyPeerCertificate: VerifyCertificates,
},
make: *** [Makefile:68: govet] Error 1

Do you have any ideas about what could be causing this error?

[born2ngopi]

@djanda97 try pulling the latest master

[djanda97]

I was successfully able to build mmctl and run the existing e2e tests, however I can't find the commands/command_e2e_test.go file in the mmctl repo. Is this a typo or should I create the file myself?

[wijayaerick]

@djanda97 You should create the file

[djanda97]

@djanda97 You should create the file

Great, thanks!

[mgdelacroix]

Hi @djanda97! Are you still planning to work on this issue? Is there anything we can help with?

[djanda97]

Hi @mgdelacroix! Yes, I still plan on working on this issue, I've just been busy with my new job.

[rdanielwetan]

I would like to work on this please.