Fix for module.run show result as True when module returned False as result

[SndR85]

What does this PR do?

What issues does this PR fix or reference?

Fixes #65842

Previous Behavior

If module.run was running a module which returned False in it's result, the module.run state would return it as successful with it's result set to True.

New Behavior

With this change the state will have result set to False if the module contains a result where the value not is equal to True.

Merge requirements satisfied?

[NOTICE] Bug fixes or features added to Salt require tests.

• Docs
• Changelog - https://docs.saltproject.io/en/master/topics/development/changelog.html
• Tests written/updated

Commits signed with GPG?

Yes

[SndR85]

@twangboy I found another flaw in the current change I made. When the result is set to a non-boolean value it will be automatically set to False. I think I should check if result is a boolean.

[SndR85]

I have fixed it, I won't change anything for now unless a maintainer or CI does not agree with current change. Personally I think as a bonus the change is now more readable.

[hurzhurz]

I also already made a PR for the same issue a while ago which is waiting for merging. Maybe you could have a look at it: #67115
And I choose 3006.x instead of master as target branch, so it will be fixed in a 3006 release. From there it should find its way to master automatically.

[twangboy]

Just need to migrate that test to pytest on #67115

[SndR85]

Close as #67115 seems to be a better fix and code cleanup.