Revert "Stop running the test while it's failing"

This reverts commit a084303c91fca4c4a14a31a30d8c15f001665611.
Merge branch 'develop' of github.com:saltstack/salt into hotfix/immutable-lists-sum
Merge branch 'develop' of github.com:saltstack/salt into hotfix/immutable-lists-sum
Test `ImmutableList` sum
Revert previous intrusive fixes
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)
state.saltmod: kwarg support
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': []}
fix saltenv for aptpkg.mod_repo from pkgrepo state

so that when key_url=salt://gpg.key, salt will get gpg.key from the correct
environment instead of the 'base' environment.
Adds a third boolean parameter, merge, to pillar.get() for situations where the returned data (dict) should be merged with the passed default via salt.utils.dictupdate.update(). A convenience param for creating default pillar data in situations where grains.filter_by() is unnecessary because no actual grains need be filtered.
Merge remote-tracking branch 'upstream/develop' into add-pillar-get-merge
Replace socket.get_fqdn with internal salt.utils.network.get_fqhostname
Update error message for `cmd.script` on cache fail to use saltenv