aiidateam / aiida-core

72%

develop: 70%

LAST BUILD ON BRANCH fix_3371_readthedocs

branch: fix_3371_readthedocs

CHANGE BRANCH

x

Reset

• fix_3371_readthedocs
• add-code-of-conduct-1
• aiida_tutorial_05_2018
• develop
• fix_2233_node_to_new_backend
• fix_2384_rest_api
• fix_3152_restapi_test_random_fail
• fix_3578_readthedocs
• fix_for_old_wf_in_sqla_issue_883
• fix_rest_api_identifier_utilities
• master
• merge_master
• release/0.12.5
• release/1.0.0
• release_v0.12.4
• release_v0.12.5
• support/0.12.x
• support/1.0.x
• v0.12.4
• v0.12.5
• v1.0.0
• v1.0.0b5
• v1.0.0b6
• v1.0.1
• workflows

Build # 7150

Build Type

Pull #3374

travis-ci

Committed by web-flow

Commit Message

Fix loading of database environment in ReadTheDocs environment

The environment of ReadTheDocs does not actually have a database and so
the typical configuration and profile loading had to be hotwired. A
special global flag `IN_RTD_DOC_MODE` in `aiida.manage.configuration`
would circumvent loading an actual configuration for a dummy one. Since
the actual database environment never had to be loaded, the fact that
this contained non-sensical database connection parameters was
irrelevant. However, recently the documentation was changed to use the
Sphinx extension to document the `CalcJob` class. This now requires
loading the `aiida.orm` module, which in turn requires the database
environment being loaded.

The fix is to actually load the database environment through the backend
manager, but without performing the schema check. This will work despite
the unusable database configuration because no actual connection is
established in this way.

Pull Request Pull Request #3374: Fix loading of database environment in ReadTheDocs environment

Run Details

5 of 5 new or added lines in 2 files covered. (100.0%)

69647 of 96897 relevant lines covered (71.88%)

2.75 hits per line