Now a check_file_meta deletes temporary files when test=True

[sakateka]

What does this PR do?

fix check_file_meta that it deletes temporary files when test = True

Tests written?

• Yes
• No

[cachedout]

HI @sakateka

You mention in your description that this should only delete temporary files when test=True but I don't see this wrapped in a conditional for testing mode at all. Could you please clarify? Perhaps I'm just missing something obvious. :]

[sakateka]

To be precise, this function will always leave temporary files, but I saw a problem only when launched highstate test=True.
state example

/tmp/test.file:
  file.managed:
    - contents: |
        test contents

state file.managed call file.check_managed_changes and it call check_file_meta

[cachedout]

Gotcha. Thanks for the clarification.