wrapping NamesapcedJS style classes in a NodeJS style define block
adding 'AssetContentAlteringSourceModule' interface

adding a 'AssetContentAlteringSourceModule' interface to represent
SourceModules that change the reader in some way to allow the
AssetReader to get access to the base reader.  we need to do this since
we need to use the reader to calculate dependencies, but previously the
reader needed to calculate dependencies to print out static dependencies
creating a circular dependency issue
nodejs source mode implements ContentAlteringSourceModule
adding requireAll for all static deps in NamespacedJS source modules
fixing tests after NamespacedJS wrapping changes

fixing tests that are asserting against the wrapped version of
NamespacedJS classes that dont need to be - they can just assert part of
the source module is there since the specific format of the altered
source module should be tested in the NamespacedJSSourceModule tests
removing globalisation of NodeJS classes

removing JIT globalisation of NodeJS classes since its now done by the
requireAll call inside the define block
fixing { that should be [ in requireAll and adding extra test
removing ContentAlteringSourceModule and casting NamespacedJsSourceModule in the reader factory instead
starting to add requireAll JS code
Revert "removing globalisation of NodeJS classes"

This reverts commit c658b056e97c1880d97ccfd16364a93eabe423e2.

Conflicts:
	brjs-core/src/test/java/org/bladerunnerjs/spec/bundling/aspect/AspectBundlingOfMixedSources.java
removing function definiton from ContentPlugin since its now in bootstrap
fixing globalisation of NamespacedJS classes
reintroducing 'AugmentedContentSourceModule' interface
Merge branch 'develop' into static-js-dependency-fixes
passing a AugmentedContentSourceModule in to JsCommentAndCodeBlockStrippingReaderFactory
Merge pull request #620 from BladeRunnerJS/static-js-dependency-fixes

Static js dependency fixes