Un-hardcode the root user from manage.bootstrap
Merge pull request #13461 from s0undt3ch/develop

Revert "Cache the salt client"
Linux ACL: default mask and default other

If an ACL has default values for `other` and `mask` these values will overwrite the non-default `other` and `mask` values.

E.g.:
```
# file: t
# owner: root
# group: root
user::rwx
user:www-data:rwx		#effective:---
group::r-x			#effective:---
mask::---
other::---
default:user::rwx
default:user:www-data:rwx
default:group::r-x
default:mask::rwx
default:other::---
```

Here the default `mask`/`other` values would overwrite the `mask`/`other` value.

This commit fixes that.
Remove references to pylint E8712

E8712 is not a thing, at least not according to pylint's docs:

http://docs.pylint.org/search.html?q=E8712

Also, a couple other fixes:

1. Double quotes to single quotes to follow style conventions
2. "is Bool", not "== Bool".
Merge pull request #13462 from d--j/patch-2

Linux ACL: default mask and default other
Revert "detect the right transport for the client in tests"

This reverts commit d33b01d6355b6e7c7339c07d7d8f36ffedabb018.
Lay down computed configuration and use that source for all of the tests
Don't remove the root_dir, the tests suite takes care of that
Call `verify_env` on `transplant_configs`
Remove stray print and commented out entries
Store evaluated configs as `cls` attributes
Remove unused imports
Use `SaltClientTestCaseMixIn` instead
Tempdir under `integration.SYS_TMP_DIR` and use `AdaptedConfigurationTestCaseMixIn`
Leave any cleanup for the end
Call the right parent class
Remove unused imports
Disable PEP8 E256
No need for late imports. Use `AdaptedConfigurationTestCaseMixIn`.
Copy the extension modules into the new master `root_dir`