Test filebeat/metricbeat modules list

[andrewkroh]

Test that the filebeat modules list and metricbeat modules list commands are working.

Relates elastic/beats#7488 elastic/beats#8105

[andrewkroh]

This should fail for Metricbeat on Linux until elastic/beats#8105 is merged. Both the Filebeat and Metricbeat+Windows changes should be running successfully.

[tsg]

This would catch the bug we had, but I'm thinking to also do a modules enable command and check that modules list had enabled that module? This would cover a little bit more from the Kibana tutorials.

[ruflin]

LGTM.

@tsg agree we should add more and run enable/disable. Should we merge this PR and do it in a follow up?

[andrewkroh]

We have those tests in the product's system tests. Do you still want to add them here as well?

• enable - https://github.com/elastic/beats/blob/master/metricbeat/tests/system/test_cmd.py#L43
• disable - https://github.com/elastic/beats/blob/master/metricbeat/tests/system/test_cmd.py#L80

[tsg]

Between that and the package tests and the module list from here we're probably well covered, so maybe we don't need it. If it's very simple to add them, it might still be worth it to do it, but I'm good to do it in a future PR.

[andrewkroh]

I opened a separate issue for the enable/disable testing enhancement. See #88.

[ruflin]

@andrewkroh Want to merge it yourself in case @tsg gives his 👍 ? :-)