Change the default scope for junit to be test scope.

Junit should typically have a test scope as it is not normally part of
production artifacts.  The surefire plugin needs a version (otherwise
maven complains), and corrected indenting for the maven-surefire-plugin
declaration.
Servlet API should be Provided scope.

The Servlet API may be used to compile code, but the container (JBoss,
Tomcat, Jetty) typically provides the implementation at runtime.
REASTEASY-180 Plugin declaration cleanup.

Plugin versions should be declared in PluginManagement, but then you
don't need to mention them again in the build/plugin section (as they
are inherited).
Maven's super-pom already declares the basics (install, deploy,
compiler, etc.) so you don't need to.
The project should declare the encoding using
project.build.sourceEncoding, then you don't need to specify it for
every plugin as they typically refer to that.
servlet-api dependency was declared twice

Once was with provided scope, and the other with test scope.  I kept
the provided scope one and removed the other.
Upgraded SLF4J and tweaked dependencies.

Removed the SLF4J-Simple runtime dependency because it violates the
guidance from SLF4J.  (Libraries should not dictate a runtime logging
implementation.)