Fix apache_module states to return None and changes dict in test mode

Fixes #16953.
Added changes dict to apache.configfile state in test mode
Cleanup the logic-- no reason to have so many negatives
Increase backoff each iteration by one up to syndic_wait as opposed to by one
Misc comment cleanup in tests
Spelling fixes for docs
Spelling
More spelling
More spelling
Fix for #16939

In the case that the master-master has no immediate downstream minions we were just incrementing the syndic_wait N times and then exiting, not pinging the lower level masters/minions. Now, I've consolidate the jinfo_timeout and timeout_at (since they were the same thing) and changed it to add syndic_wait to the timeout_at if you are a master-master, have more time to add, and would have timed out.
Cleanup logging output (to not be confusing)
Make sure `kwargs=None` is properly handled. Fixes #16945.
Change syndic_wait in get_iter_returns

This will ensure we only do the quick return if you are *not* a syndic, since when you are a syndic we have no idea what the target is-- so we have to wait. This means that syndic masters (the master of masters) will have at least syndic_wait seconds until the return comes back
Fix syntax error
Disable pillar/compound matching for peer interface, master-side
Disable compound matching for pillar.get, minion-side
Disable pillar/compound for mine.get, master-side
Disable compound/pillar targeting for peer, minion-side
Fix typo
Also remove the expr_form match for compound