Fix for broken salt cmd return codes - issue #7013

- new test related states that can be used to provide success / failure on request.
- fixed the broken logic chain that handles return codes
both when the minion decides what retcode to propagate to the master
and when the master decides what return code to return to the invoked salt cmd
- added new log.debug and log.trace statements to assist with further work on ensuring return
codes are properly handled by all components of salt.
fixing syntax error.
another syntax fix
Fixing W0632(unbalanced-tuple-unpacking) pylint violation
Fixes

- Fixed bad logic in data handling block.
- Fixed and refactored the test state to be
 more complete and in line with coding standards.
Pylint fixes

- Added newlines to fix E8302(expected-2-blank-lines,-found-0)
- Disabled pylint checks for E8712(comparison-to-True-should-be-'if-cond-is-True:'-or-'if-cond:') and documented my reason
Fixing E8261(at-least-two-spaces-before-inline-comment)
Trying to fix integration test failing on a missing file so CI actually runs on my PR.
Fixes & Updates

- fixed pep8 issue
- added more logic to check_state_result logic to handle list results properly and return false.
- ordered the check_state_result unit tests more logically. All the data type tests are now before the content & logic handling checks
- discovered duplicate test in check_state_result unit tests and removed it, both tests were asserting using the same data {'host': []}
Merge pull request #11337 from techdragon/fix-hightstate-failure-retcode

Fix for broken salt cmd return codes - issue #7013