Add missing return to allow pipe()'s to be chained together per #791
for config.get callbacks, pass closures

* instead of references to functions. Resolves Issue #784
* update tests to go with watch updates
cleanup duplicated loading log messages

* outbound: removed unused ‘var server’
* updated 3 more callbacks to be closures
* removed 12 loginfo(‘loading….’) from plugins
Merge pull request #788 from msimerson/fs-watch-cb

for config.get callbacks, pass closures
tests/plugins/* cleanups

* remove lots of unused boilerplate
* move constants importing into fixtures/plugins (and removed to dozen+
plugins)
* purged unused requires
* updated tests that were still manually adding stuff to plugins and
connections that have since been added to the test figures.
Merge pull request #790 from msimerson/test-cleanups

tests/plugins/* cleanups
link updates (folding long lines)
cfreader/binary, flat, ini, json, yaml
config.js: lint & ws
configfile: removed formats to cfreader/*

* whitespace / link updates
* enable tests/cfreader
* moved ini tests to tests/cfreader/ini
access: enable helo.check in test that tests it
moved caching to configfile (from cfreader/*)
cfreader/binary: improved testing

test length, and compare with fs loaded file.
YAML overrides for JSON change

* avoid duplication in cfreader/*
* move file existence tests into configfile
moved empty config into cfreader.type.empty()

instead of configfile.empty_config()
add tests for configfile.get_filetype_reader
configfile: comment out logger