Wrap the AtomicIntegers with getters, so that access is cleaner and looks nice. Also make them protected, so they don't get accidentally changed.
Remove unused code.
Replace the ugly regex checks with method calls.
Extract the regexes into properties. In the future, if we need to change a regex, we will only have to change it once.
The canned xml and json responses should be replaced with builders, so that we can customize them in tests more easily.
Fix variable names. Make it easier to read.
Merge pull request #919 from izrik/cleanup-mock-identity-service

Cleanup mock identity service
added test for tenanted=false and no service admin roles to check that the right header values are populated
Merge pull request #922 from rackerlabs/DNS-test-tenanted-false-headers

added test for tenanted=false and no service admin roles to check that t...
Updated the XSD to support the new schema entry

For bypassing the tenant check
Adding functionality to test tokens that have no tenant
Creating the test, it fails!
Adding a bypass tenant check roles configure thingy
Adding the logic to here to bypass the tenant check

Also, reformatted the file to match the rest of the formatting.
Adding the configuration option to the Factory
Updating tests to compile for RackspaceAuthentication

Which we should delete anyway
Update the tokens so that they're valid

The old tests would fail sooner and not actually have valid tokens
passed in. Now there's actually at least one role associated with a
token.
Just updating the constructor
since tenant can now be optional...

Handle parts of the logic inside that if statement, so we don't blow up
horribly.
Updated test to work with no tenant

As well as formatting, because the formatting is off.