yahoo-eng-team team mailing list archive

[wangxiyuan <wangxiyuan@xxxxxxxxxx>]

Public bug reported:

Now Keystone runs unit test with sqlite backend by default. But sqlite doesn't enable Foreign Key function by default.
It means that we lost the case for all FK related test.
Even after enabled FK for the test. There still some problems:
1. We lost root domain (<<keystone.domian.root>>) when setup tests. So that a lot of tests will fail.
2. Some tests, such as test_sql_upgrade and identity.backens.test_sql, use global db engine. It means that the test will meet conflict if the FK is enabled for the global db engine.

So we should refactor our test code to satisfy sqilte FK function.
There are some steps:
1. Add FK support for the tests. Disable it by default
2. Enable FK for the test one by one.
3. Make sure the Fk is disabled for test_sql_upgrade and identity.backens.test_sql

** Affects: keystone
     Importance: Undecided
         Status: New

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to OpenStack Identity (keystone).
https://bugs.launchpad.net/bugs/1744195

Title:
  unit tests don't enable Foreign Key for Sqlite

Status in OpenStack Identity (keystone):
  New

Bug description:
  Now Keystone runs unit test with sqlite backend by default. But sqlite doesn't enable Foreign Key function by default.
  It means that we lost the case for all FK related test.
  Even after enabled FK for the test. There still some problems:
  1. We lost root domain (<<keystone.domian.root>>) when setup tests. So that a lot of tests will fail.
  2. Some tests, such as test_sql_upgrade and identity.backens.test_sql, use global db engine. It means that the test will meet conflict if the FK is enabled for the global db engine.

  So we should refactor our test code to satisfy sqilte FK function.
  There are some steps:
  1. Add FK support for the tests. Disable it by default
  2. Enable FK for the test one by one.
  3. Make sure the Fk is disabled for test_sql_upgrade and identity.backens.test_sql

To manage notifications about this bug go to:
https://bugs.launchpad.net/keystone/+bug/1744195/+subscriptions