Skip to content
Snippets Groups Projects
Commit fd1a2b1b authored by Bastien Abadie's avatar Bastien Abadie
Browse files

Cover the 3 cases

parent b7906fc5
No related branches found
No related tags found
1 merge request!20Load configuration from API or local file
Pipeline #77953 passed
...@@ -34,10 +34,29 @@ def test_init_var_ponos_data_given(monkeypatch): ...@@ -34,10 +34,29 @@ def test_init_var_ponos_data_given(monkeypatch):
def test_init_var_worker_version_id_missing(monkeypatch): def test_init_var_worker_version_id_missing(monkeypatch):
monkeypatch.setattr(sys, "argv", ["worker"])
monkeypatch.delenv("WORKER_VERSION_ID") monkeypatch.delenv("WORKER_VERSION_ID")
worker = BaseWorker() worker = BaseWorker()
worker.configure()
assert worker.worker_version_id is None assert worker.worker_version_id is None
assert worker.is_read_only is True assert worker.is_read_only is True
assert worker.config == {} # default empty case
def test_init_var_worker_local_file(monkeypatch, tmp_path):
# Build a dummy yaml config file
config = tmp_path / "config.yml"
config.write_text("---\nlocalKey: abcdef123")
monkeypatch.setattr(sys, "argv", ["worker", "-c", str(config)])
monkeypatch.delenv("WORKER_VERSION_ID")
worker = BaseWorker()
worker.configure()
assert worker.worker_version_id is None
assert worker.is_read_only is True
assert worker.config == {"localKey": "abcdef123"} # Use a local file for devs
config.unlink()
def test_cli_default(mocker, mock_worker_version_api): def test_cli_default(mocker, mock_worker_version_api):
...@@ -46,6 +65,8 @@ def test_cli_default(mocker, mock_worker_version_api): ...@@ -46,6 +65,8 @@ def test_cli_default(mocker, mock_worker_version_api):
assert not spy.called assert not spy.called
assert logger.level == logging.NOTSET assert logger.level == logging.NOTSET
assert not hasattr(worker, "api_client") assert not hasattr(worker, "api_client")
mocker.patch.object(sys, "argv", ["worker"])
worker.configure() worker.configure()
assert spy.called assert spy.called
...@@ -53,6 +74,9 @@ def test_cli_default(mocker, mock_worker_version_api): ...@@ -53,6 +74,9 @@ def test_cli_default(mocker, mock_worker_version_api):
assert not worker.args.verbose assert not worker.args.verbose
assert logger.level == logging.NOTSET assert logger.level == logging.NOTSET
assert worker.api_client assert worker.api_client
assert worker.worker_version_id == "12341234-1234-1234-1234-123412341234"
assert worker.is_read_only is False
assert worker.config == {"someKey": "someValue"} # from API
logger.setLevel(logging.NOTSET) logger.setLevel(logging.NOTSET)
...@@ -74,5 +98,6 @@ def test_cli_arg_verbose_given(mocker, mock_worker_version_api): ...@@ -74,5 +98,6 @@ def test_cli_arg_verbose_given(mocker, mock_worker_version_api):
assert worker.api_client assert worker.api_client
assert worker.worker_version_id == "12341234-1234-1234-1234-123412341234" assert worker.worker_version_id == "12341234-1234-1234-1234-123412341234"
assert worker.is_read_only is False assert worker.is_read_only is False
assert worker.config == {"someKey": "someValue"} # from API
logger.setLevel(logging.NOTSET) logger.setLevel(logging.NOTSET)
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment