Don't return False when target exists (file.rename)

[MTecknology]

What does this PR do?

This PR applies the change recommended by @DickerDackel in #49748.

Previous Behavior

When using file.rename and then creating a new file in it's place, subsequent executions will fail due to a file existing in it's place.

New Behavior

This patch removes the False return when a destination file exists if -force:False.

Tests written?

Yes

Commits signed with GPG?

Yes

[rallytime]

@MTecknology Thanks for submitting this! There's a test failing here: https://jenkinsci.saltstack.com/job/pr-kitchen-ubuntu1604-py2/job/PR-49843/1/

Can you take a look?

[terminalmage]

This looks good and the failing test just needs to be updated to reflect the new return value.

[damon-atkins]

Please doco the change in behavior in doc/topics/releases/neon.rst

On a side note, not an issue with this PR. However file.copy() has been change to file.copy_() in develop branch? Update found

__func_alias__ = {
    'copy_': 'copy',
}

No Problem.

[terminalmage]

You can ignore that, @MTecknology, there is no need to document a bugfix in the release notes.

[terminalmage]

We run a changelog script that adds this sort of stuff for us.

[terminalmage]

Ahh, after reading the original issue it's clear that this was not 100% bugfix, so I'll get the change documented.

[MTecknology]

@terminalmage Thanks!