Fixes #36830 - transform_vars on preseed_path

Co-Authored-by: quartsize

Fixes #37287 - Iterate on hashes when both key and value are used

This replaces the `hash.keys.each { |k| hash[k] }` and `hash.keys.sort`
patterns with `hash.each { |k, v| }` where it makes sense. This avoids
doing hash lookups when you already know the value anyway.

Fix Style/CommentAnnotation cop

sort taxable_ids before comparing them

For some reason, taxable_ids and visible_dummies are sometimes sorted
differently, resulting in failed tests even tho the right taxonomies are
found, just in the wrong order.

Use assert_equal_arrays that sorts the arrays before comparing them.

Fixes #36891 - Provide a scope for email-notification-eligible users

Refs #36849 - Enable strict keyword argument matching

Fixes #36930 - Make invalid YAML test compatible with libyaml 0.2.x

The valid {a:test} is invalid with libyaml 0.1.x but is valid with
0.2.x. This changes the test to @@ which should still raise a
Psych::SyntaxError.

fixes #36160 - Redefine append domain names setting

This PR aims to unify the format of host names stored in the database and the way they are displayed.
With this change, the name of the host is always going to be stored with the domain name appended.
The setting formerly named `append_domain_name_for_hosts` is now renamed to `display_fqdn_for_hosts`...

Fixes #36796 - Make host_facts_updated event visible

Fixes #36786 - Permit Symbol for YAML.safe_load report parser