Detection of __salt__ in loader

This resolves #10275

It appears that there was an issue where in, __salt__ defined outside
of a function in a custom grain would be overwritten by the loader
when the grain was loaded.

The old logic overwrote __salt__ if it was not present in the pack,
regardless of whether or not it was an attribute of the module being
evaluated. I don't believe this is what was intended but there's
certainly a case I could be missing.

Should probably let tests run here before merging.
Fix loader condition
Fix broken tests
This time I banged my head on the table extra-hard.
Problem was actually with the ext grains. Doh!
Merge pull request #13261 from cachedout/10275

Detection of __salt__ in loader
Merge pull request #13269 from s0undt3ch/hotfix/lint

Lint fixes + no hardcoded paths
Clarify yaml indentation issues, and add another example
Merge pull request #13271 from basepi/yamlindentation

Clarify yaml indentation issues, and add another example
Stop complaining about PEP8 E265