unit tests don't enable Foreign Key for Sqlite

Reviewed: https://review.openstack.org/126030
Committed: https://git.openstack.org/cgit/openstack/keystone/commit/?id=ed2b65a5ddc8536470797c39467b20e934b02dc0
Submitter: Zuul
Branch: master

commit ed2b65a5ddc8536470797c39467b20e934b02dc0
Author: David Stanek <email address hidden>
Date: Fri Oct 3 18:22:41 2014 +0000

    Force SQLite to properly deal with foreign keys

    This will help with testing since SQLite will start enforcing the
    foreign key relationships.

    We will still have a problem with migrations for tables that refer to
    each other. SQLite can't alter tables and sqlalchemy-migrates tmp table
    strategy for migrations fails in this situation.

    This patch did:
    1. Add FK support for the tests. Disable it by default.
    2. Make sure the Fk is disabled for test_sql_upgrade and
    identity.backens.test_sql

    Partial-Bug: #1744195

    Co-Authored-By: wangxiyuan<email address hidden>
    Change-Id: I276af7c0125dc2cb2c54215d54491665db1caa22

Fix proposed to branch: master
Review: https://review.openstack.org/558029

Fix proposed to branch: master
Review: https://review.openstack.org/558173

Fix proposed to branch: master
Review: https://review.openstack.org/558184

Fix proposed to branch: master
Review: https://review.openstack.org/558192

Fix proposed to branch: master
Review: https://review.openstack.org/558193

Change abandoned by wangxiyuan (<email address hidden>) on branch: master
Review: https://review.openstack.org/558192

Change abandoned by wangxiyuan (<email address hidden>) on branch: master
Review: https://review.openstack.org/558184

Change abandoned by wangxiyuan (<email address hidden>) on branch: master
Review: https://review.openstack.org/558173

Fix proposed to branch: master
Review: https://review.openstack.org/591946

Reviewed: https://review.openstack.org/558029
Committed: https://git.openstack.org/cgit/openstack/keystone/commit/?id=46a28313fcb2078870024874f51d9539ce4c364e
Submitter: Zuul
Branch: master

commit 46a28313fcb2078870024874f51d9539ce4c364e
Author: wangxiyuan <email address hidden>
Date: Sat Mar 31 12:19:02 2018 +0800

    Enable Foreign keys for sql backend unit test

    Enable the sqlite foreign keys function for unit test.

    This patch is the first part to solve sql backend test issues.

    Change-Id: I5d29d05e64b76ff6530c9af5ee39a2df1b26aa03
    Partial-Bug: #1744195

Reviewed: https://review.openstack.org/558193
Committed: https://git.openstack.org/cgit/openstack/keystone/commit/?id=012dac29b8dd6ac77e707f916fc4c0ca57d58b14
Submitter: Zuul
Branch: master

commit 012dac29b8dd6ac77e707f916fc4c0ca57d58b14
Author: wangxiyuan <email address hidden>
Date: Mon Apr 2 19:28:33 2018 +0800

    Enable foreign keys for unit test

    The unit test uses sqlite for test which closes db foreign keys
    function by default. This patch enabled the sqlite foreign keys
    function for unit test by default.

    The "project" table is a self referencing FK table(id <-> domain_id
    column). So when the FK is enabled, there must exists a root record
    before insert data to this table. It's <<keystone.domain.root>>.

    Usually, the <<keystone.domain.root>> recored is inserted into the
    table once operators run "keystone-manage db_sync" command when
    deploy Keystone. But the unit test code doesn't run this command,
    it initialise the db schema by reading sqlalchemy object model, so
    the <<keystone.domain.root>> record is missed. Then we can't create
    any project record, it'll raise FK error.

    So in this patch, before creating any projects in the test, we must
    ensure the <<keystone.domain.root>> record exists first.

    Change-Id: I565d12395ca39a58ba90faf8641a9e02d986aeb9
    Closes-Bug: #1744195

Reviewed: https://review.openstack.org/591946
Committed: https://git.openstack.org/cgit/openstack/keystone/commit/?id=c3f590bc13efcdddb3fa82e8d116239f4e8d8b1f
Submitter: Zuul
Branch: master

commit c3f590bc13efcdddb3fa82e8d116239f4e8d8b1f
Author: wangxiyuan <email address hidden>
Date: Wed Aug 15 15:43:57 2018 +0800

    Add a test for idp and federated user cascade deleting

    If a idp is deleted, the related federated user should be
    cascade deleted as well.

    Change-Id: I2c9b4052413f9a31ffc22c5f3b1bee30dda2c42a
    Partial-bug: #1744195

This issue was fixed in the openstack/keystone 15.0.0.0rc1 release candidate.