Fix the order of assertEqual arguments(keystoneclient, kvs, etc)

assertEqual method's arguments should be in
('expected', 'actual') order.

Change-Id: Ic429c9264a823c3b03ca3b5c898b4aeb96891504
Partial-Bug: #1277104
Fix test_provider_token_expiration_validation transient failure

The keystone.tests.test_token_provider.TestTokenProvider.
test_provider_token_expiration_validation test would fail if the
tests weren't run before the expiration period passed. This could
happen if the system was slow and the previous tests took a long
time causing a potential transient failure.

Change-Id: I123aadd55770002fe1c9ed9b17669d5212198aa3
Closes-Bug: #1278550
remove the unused variable in test_sql_upgrade

There is an unused variable in tests/test_sql_upgrade.py. It needs
to be removed.

Change-Id: Ia50a4e91319d1a4dce9515c8b910de28fa68fcb5
Add slowest output to tox runs (testr)

Add --slowest for all tox test runs that don't use other various
test commands.

Change-Id: I9f9fdcdda8895ba33c817f87b566794dbcd6ccb9
Use oslo db.sqlalchemy.session.EngineFacade.from_config

Rather than passing the CONF dict to EngineFacade, Keystone can
use the new from_config factory function.

Change-Id: I60f30254431b512988a0f1811b3c119207b91ad7
Do not expose internal data on UnexpectedError

UnexpectedError now works much-more akin to SecurityError and
obscures internal data from being leaked unless running in debug
mode.

Change-Id: I8c8895f5c9db9f51dec014da24f7e86d607d0363
Closes-Bug: #1291047
Remove unused db_sync from extensions

The db_sync in the extensions isn't used, either by the tests or
by keystone-manage db_sync, so remove them.

Change-Id: Ia0598e8dae81e258a05767f47525d5c11acce8db
Uses generator expressions instead of filter

Change-Id: I3685a8ecb4d34258e44a1480c4f78aaaf4dca27c
Implements: bp keystone-py3kcompat
Start using to oslotest

oslotest provides a BaseTestCase class then enables proper capturing of
stdout, stderr and logging streams during a test run. We will take
advantage of this capability in the near future.

Change-Id: I16761053f631e709224eee813dc0f423f06c8870
update sample conf

Change-Id: Ic80e79ff412a0156a94b2992efb9ec9487b41a2c
Partial-Bug: 1293781
Related-Bug: 1233365
Merge "Do not expose internal data on UnexpectedError"
is_revoked check all viable subtrees

An indentation error coupled with an early return lead to cases
where a subtree containing a pertinent revocation event was not
searched, and a revoked token would report as unrevoked.

Closes-bug: #1294292
Change-Id: If56ae64d8a30b461563ee8be002544117fb14215
Merge "update sample conf"
Add support for parallel testr workers in Keystone

Full support for parallel testr workers now works in keystone. The
major item to be aware of is that if non-SQLite backends or a LIVE
LDAP backend is being used for testing, the concurrency must be set
to 1 to avoid conflicts. To artificially set the concurrency set
the env variable ``TEST_RUN_CONCURRENCY`` to the desired number of
testr workers. The default is to run one worker. To use 1 per
available core, set ``TEST_RUN_CONCURRENCY`` to 0.

Each worker uses a subdirectory of the test temp dir  based upon
the PID. When the worker exists (via the use of atexit), the
worker-specific PID directory is automatically removed.

The first time concurrent tests are run in a given environment,
ensure that old .testrepository directories are cleaned up as they
may introduce bad learning data on how to bin the tests based upon
test runs that occured without concurrency.

Closes-Bug: #1240052
Change-Id: I5c3385a431774ce574266a82065ed6237e8b3665
Merge "is_revoked check all viable subtrees"
Merge "Use oslo db.sqlalchemy.session.EngineFacade.from_config"
Use assertIn in test_v3_catalog

assertEqual was used where assertIn should have been. assertIn
will provide a more useful error message.

Change-Id: I97ccbb8579ea726fd76c520d0573fdcbf5a261d7
Filter SAML2 assertion parameters with certain prefix.

Add ``assertion_prefix`` option that filters environment parameters
that will be passed to the RuleProcessor object. Parameters' names
must start with ``assertion_prefix`` value.
If not configured, ``assertion_prefix`` defaults to an empty string,
and all environment parameters are passed to the RuleProcessor.

Change-Id: I2696bbadcfff9745d8edca6c896c13fda49d636e
Closes-Bug: #1293436
Add a space after the hash for block comments

Automatic enforcement of this rule is added in:
https://review.openstack.org/78119

bp more-code-style-automation

Change-Id: I167d8e2e648eaf553c586f225939daddfaeddda6
Removes the use of mutables as default args

Automatic enforcement of this rule is added in:
https://review.openstack.org/78119

bp more-code-style-automation

Change-Id: I184105a456e9beae3cb3d6716b974ba52ef8bdad