Set not-kept classes when promise should be repaired, but is warn-only

There are probably more cases where Log instead of cfPS is used and
that are not covered by this fix, but this covers the immedate
issues as reported and covered by the test case.

Test case originally submitted as https://github.com/cfengine/core/pull/1311,
but merging here with the already existing test case.

See Redmine #2359.
Agree, this is confusing. There is already the ``classes_generic`` body via your included ``default.cf.sub``, which defines exactly the same classes. Just use that and remove this body altogether.