fix(acl): correctly assert needed changes when recursive is true

[Silejonu]

SUMMARY

Right now, when setting recursive ACLs on a directory, all files in the directory are tested to check if a change is needed. If a single file has expected ACLs already set, then the test returns False and no changes are applied.

Fixes #592

ISSUE TYPE

• Bugfix Pull Request

COMPONENT NAME

acl

ADDITIONAL INFORMATION

I'm very much a beginner in Python, so any criticism is welcome.

[softwarefactory-project-zuul]

Build succeeded.
https://ansible.softwarefactory-project.io/zuul/buildset/794022f453a34f9f8eb3dd60cb8352f4

✔️ ansible-galaxy-importer SUCCESS in 3m 13s
✔️ build-ansible-collection SUCCESS in 5m 47s

[softwarefactory-project-zuul]

Build succeeded.
https://ansible.softwarefactory-project.io/zuul/buildset/d1bfc23b2975487ca8d2df34095833f6

✔️ ansible-galaxy-importer SUCCESS in 3m 23s
✔️ build-ansible-collection SUCCESS in 5m 23s

[softwarefactory-project-zuul]

Build succeeded.
https://ansible.softwarefactory-project.io/zuul/buildset/1b0e9ff354ad4403a22d93bc13ddc59e

✔️ ansible-galaxy-importer SUCCESS in 3m 16s
✔️ build-ansible-collection SUCCESS in 5m 46s

[dnorthup-ums]

@Silejonu Is there a chance you could add additional test cases to the relevant tests ( in tests/integration/targets/acl/tasks/acl.yml ) to help prevent accidental introduction of regression bugs in the future? I think that may make it more likely that the team will accept your pull request. (I'm not part of the team maintaining this, but that's what I've been told by related teams in the past.)