Clean up ready for new code
A roadmap on what we're going to attempt here.
Proof of concept version
Abstract as much as possible in to a class that doesn't rely on querying the database, to make testing less painful.
Fixes for late night coding bugs
Sort keyword args so that the tests are saner.
Test cases
Support for coercing dicts to lists.

Specify fields to be hit with dict.values()
Fixes for config, pep8 and ext_pillar
Proper docs
We cannot use the requires_salt_modules decorator because we do not inherit from an integration test class.
Added S0undt3ch's skipIf suggestion
Merge pull request #10393 from rallytime/dnsutil_test

Fixed Test
apache module: rm unused import of `os`

************* Module salt.modules.apache
salt/modules/apache.py:12: [W0611(unused-import), ] Unused import os
Merge pull request #10395 from cvrebert/patch-1

apache module: rm unused import of `os`
fix indentation & trailing whitespace in pkg state
rm unused imports in deb_apache module

************* Module salt.modules.deb_apache
salt/modules/deb_apache.py:14: [W0611(unused-import), ] Unused import urllib2
salt/modules/deb_apache.py:12: [W0611(unused-import), ] Unused import re
Merge pull request #10397 from cvrebert/patch-1

rm unused imports in deb_apache module
modules.pkg_resource: avoid shadowing outer version() func
avoid shadowing outer funcs in modules.lxc.list_()

************* Module salt.modules.lxc
salt/modules/lxc.py:294: [W0621(redefined-outer-name), list_] Redefining name 'info' from outer scope (line 468)
salt/modules/lxc.py:297: [W0621(redefined-outer-name), list_] Redefining name 'state' from outer scope (line 409