Skip to content

Commit

Permalink
🐛 FIX: Add broker_* parameters to default test config.
Browse files Browse the repository at this point in the history
In the `TestManager`, set the `broker_*` parameters by adding
them to the `_DEFAULT_PROFILE_INFO`.

The default values were copied from the config schema. They could
be extracted directly, but since some keys (e.g. `AIIDADB_*`)
are named differently in the `profile_info` compared to the
config schema, this may be unreliable.

Previously, these were set to `None` by default, which caused the
config validation to fail because only string values are allowed.
  • Loading branch information
Dominik Gresch committed Mar 25, 2021
1 parent a8ee41d commit b1c05b7
Show file tree
Hide file tree
Showing 2 changed files with 16 additions and 0 deletions.
6 changes: 6 additions & 0 deletions aiida/manage/tests/__init__.py
Original file line number Diff line number Diff line change
Expand Up @@ -39,6 +39,12 @@
'repo_dir': 'test_repo',
'config_dir': '.aiida',
'root_path': '',
'broker_protocol': 'amqp',
'broker_username': 'guest',
'broker_password': 'guest',
'broker_host': '127.0.0.1',
'broker_port': 5672,
'broker_virtual_host': ''
}


Expand Down
10 changes: 10 additions & 0 deletions tests/manage/tests/test_pytest_fixtures.py
Original file line number Diff line number Diff line change
@@ -0,0 +1,10 @@
# -*- coding: utf-8 -*-
"""Tests for the AiiDA pytest fixtures."""

from aiida.manage.configuration import get_config
from aiida.manage.configuration.config import Config


def test_profile_config(aiida_profile): # pylint: disable=unused-argument
"""Check that the config file created with the test profile passes validation."""
Config.from_file(get_config().filepath)

0 comments on commit b1c05b7

Please sign in to comment.