Avoid use of str() with exceptions

The new Message class does not allow str() so all locales can be safely
translated. It requires use of unicode() instead.

Change-Id: I1fd9142b0211cecc746578f4080cb2ddfa663b0d
Related-Bug: #1274638
Don't skip tests for some bugs

Skipping tests for a bug is a bad idea since if the bug is fixed
it's a manual process to find the affected test rather than being
found automatically by the tests.

Change-Id: I8109459bc825c5dc042e386432cb3496871f9862
Remove unnecessary check to see if trustee exists

Previously we check if the returned trustee value is None, this
is already done at the backend level.

Change-Id: I801bed09300782e289ee505612df09f7270fcb4e
Clean up database fixtures

 - Issues setting up the DB will now break early instead of causing
   issues later in tests.
 - Databases are now removed at the end of the test run.

Change-Id: I1dc5feafa9ae88fc0f2a4dd9459e7e34e2c4ce73
Add tests for oauth consumer normalize fields

There were no tests that verified that fields in oauth consumers
were normalized as expected. The tests show that the fields aren't
being normalized as expected on update.

Change-Id: I3bc95766b61d8dc8c85b32f628b275721255e95d
Related-Bug: #1276857
Correctly normalize consumer fields on update

The self._normalize_dict() function in ConsumerCrudV3 creates a
copy of the consumer passed in and doesn't modify it, so this
line of code didn't do anything. The code is changed to use the
result as the new consumer rather than discarding it.

Closes-Bug: #1276857
Change-Id: Iec7573001ad73f0dc9192a4e74b1c35adc2cda4c
Cleanup oauth tests

In a lot of places this code was using dict.get() where [] makes
more sense because the code then assumes that the field was
present and had a value. Using .get() just means that if the code
changes the error would be an unexpected TypeError on a different
line than where the problem actually was.

Also, building of simple url strings was simplified to remove the
dict so that they could be kept on a single line.

Change-Id: I98b5631e02fe1456f0f583c4ec55c9ac0f8c4c35
Provide clearer error when deleting enabled domain

Currently the message is confusing, because it reads like an instruction
to delete an enabled domain, rather than an instruction that you can
only delete disabled domains.  Hopefully this wording is a little clearer
for users.

Change-Id: I5189cd1bbae0e485d03660be19f20879110a3b3e
Closes-Bug: #1279823
LDAP: document enabled_emulation

 - Add documentation to the Configuration page for enabled_emulation
 - Fix a comment about the default value used for enabled_emulation

Change-Id: I3fdb4e9ecfe8bd43f8046d6e99a18941bc938487
Closes-Bug: #1275907
List trusts, incorrect self link.

Fix the base_url method in trust controller to build off
of the one in V3 controller.

Closes-Bug: #1279398

Change-Id: If9b3114c9b2b9978237988fa3b52e0bede783057
Add lockfile and kombu as requirements for keystone

Sphinx is failing to build because we do not list
lockfile or kombu in either requirements file.

Change-Id: Ic31870c283a18958cd61e51bc56ed771732ee753
Use WebOb directly in ec2_token middleware

When https://review.openstack.org/#/c/71494/ was introduced, it
forgot to change ec2_token as well. This caused errors when building
docs.

Change-Id: I051519c815ed5ae60ec07203073131db1e031e10
Treat sphinx warnings as errors

Change-Id: I003c9a210704c12423dc038f442df3913d133a6a
Set sensible default for keystone's paste

The https://review.openstack.org/#/c/66247/ change introduced
the requirement to set config_file before keystone would start
up. Restore the old setting as default to allow a smooth
transition for users.

Change-Id: I935c5144728d04c59abaee9a515e249ef0d24477
Merge "Avoid use of str() with exceptions"
Merge "Provide clearer error when deleting enabled domain"
Make live LDAP user DN match the default from devstack

The default LDAP user DN in devstack uses the "cn" attribute in
the RDN.  The live LDAP test default configuration is supposed to
match the default devstack config, but the RDN is incorrectly
using the "dc" attribute.  This patch corrects the live LDAP
default config to match devstack.

Change-Id: Iee49606526047105b4e60142af32b4ec81a19444
Closes-Bug: 1280509
Merge "Remove unnecessary check to see if trustee exists"
Merge "Add tests for oauth consumer normalize fields"
Merge "Clean up database fixtures"