ansible-meraki icon indicating copy to clipboard operation
ansible-meraki copied to clipboard

Warnings in the Ansible tests

Open y0rune opened this issue 2 years ago • 2 comments

Hello! I tested the repo using the command:

ansible-lint --nocolor  >> to-be-improved.txt
cat to-be-improved.txt | grep -ioE 'tests\/integration\/.*:' | awk -F: '{print $1}' | sort -u

The output showed the Finished with 1808 failure(s), 681 warning(s) on 157 files. I would like to check it and changed it to resolve all of them.

List where I would like to change the warnings:

  • [x] tests/integration/targets/meraki_action_batch/tasks/main.yml - #368
  • [x] tests/integration/targets/meraki_admin/tasks/main.yml - #370
  • [x] tests/integration/targets/meraki_alert/tasks/main.yml - #371
  • [x] tests/integration/targets/meraki_config_template/tasks/main.yml - #372
  • [x] tests/integration/targets/meraki_device/tasks/main.yml - #373
  • [x] tests/integration/targets/meraki_firewalled_services/tasks/tests.yml - #374
  • [x] tests/integration/targets/meraki_management_interface/tasks/main.yml - #375
  • [x] tests/integration/targets/meraki_mr_l3_firewall/tasks/main.yml - #377
  • [x] tests/integration/targets/meraki_mr_radio/tasks/main.yml - #378
  • [x] tests/integration/targets/meraki_mr_rf_profile/tasks/main.yml - #383
  • [x] tests/integration/targets/meraki_mr_settings/tasks/main.yml - #391
  • [x] tests/integration/targets/meraki_ms_access_list/tasks/tests.yml - #397
  • [x] tests/integration/targets/meraki_ms_stack/tasks/main.yml - #402
  • [x] tests/integration/targets/meraki_ms_storm_control/tasks/main.yml - #404
  • [x] tests/integration/targets/meraki_ssid/tasks/main.yml - #396
  • [x] tests/integration/targets/meraki_ms_ospf/tasks/main.yml - #401
  • [x] tests/integration/targets/meraki_mx_intrusion_prevention/tasks/tests.yml - #409
  • [x] tests/integration/targets/meraki_ms_switchport/tasks/main.yml - #405
  • [x] tests/integration/targets/meraki_mx_content_filtering/tasks/main.yml - #406, #407, #408
  • [ ] tests/integration/targets/meraki_ms_l3_interface/tasks/main.yml - #398
  • [x] tests/integration/targets/meraki_ms_link_aggregation/meraki_network/tasks/main.yml - #400
  • [x] tests/integration/targets/meraki_ms_link_aggregation/tasks/main.yml - #400
  • [ ] tests/integration/targets/meraki_ms_stack_l3_interface/tasks/main.yml - #403
  • [x] tests/integration/targets/meraki_mx_l2_interface/tasks/main.yml - #432
  • [x] tests/integration/targets/meraki_mx_l3_firewall/tasks/main.yml - #447
  • [ ] tests/integration/targets/meraki_mx_l7_firewall/tasks/main.yml
  • [ ] tests/integration/targets/meraki_mx_l7_firewall/tasks/tests.yml
  • [ ] tests/integration/targets/meraki_mx_malware/tasks/main.yml
  • [ ] tests/integration/targets/meraki_mx_nat/tasks/tests.yml
  • [ ] tests/integration/targets/meraki_mx_network_vlan_settings/tasks/main.yml
  • [x] tests/integration/targets/meraki_mx_site_to_site_firewall/tasks/main.yml - #454
  • [ ] tests/integration/targets/meraki_mx_site_to_site_vpn/tasks/main.yml
  • [ ] tests/integration/targets/meraki_mx_static_route/meraki_static_route/tasks/main.yml
  • [ ] tests/integration/targets/meraki_mx_third_party_vpn_peers/tasks/main.yml
  • [ ] tests/integration/targets/meraki_mx_uplink_bandwidth/tasks/main.yml
  • [ ] tests/integration/targets/meraki_mx_vlan/tasks/main.yml
  • [ ] tests/integration/targets/meraki_network/tasks/main.yml
  • [x] tests/integration/targets/meraki_organization/tasks/main.yml - #458
  • [x] tests/integration/targets/meraki_organization/tasks/tests.yml - #458
  • [x] tests/integration/targets/meraki_snmp/tasks/main.yml - #457
  • [x] tests/integration/targets/meraki_syslog/tasks/main.yml - #455
  • [x] tests/integration/targets/meraki_webhook/tasks/tests.yml - #456

Can you assign me to that task @kbreit?

Thank you! Marcin

to-be-improved.txt

y0rune avatar Sep 10 '22 16:09 y0rune

You should be assigned to this issue.

kbreit avatar Sep 10 '22 16:09 kbreit

@y0rune I'd like to integrate the checks you're doing for these into CI. Would you send me the command you're running and I can add it? Thank you.

kbreit avatar Jan 12 '23 13:01 kbreit