Enhancement: - firewall_lib.py - check if service exists before running firewalld methods that would cause failure - fails if service does not exist and in diff mode, warns if in check mode and service does not exist - README.md - reflects changes and explains interaction with check mode - tests/tests_service.yml - add integration test case for adding non-existent services in check mode - tests/unit/test_firewall_lib.py - Mock necessary output from fw.config().getServiceNames() Reason: Better compliance with Ansible best practices for check mode (not failing in check mode, especially where they would not fail in diff mode) Reason for this particular solution - We cannot track changes from previous check modes without overhauling how check mode is handled throughout the entire system role. Result: Undefined services being enabled or disabled will not result in failure while in check mode, but a warning will be displayed intended to prompt the user to confirm that the service is defined in a previous play, since the same action could result in failure when run in diff mode. Issue Tracker Tickets (Jira or BZ if any): - Addresses GitHub Issue #146