Skip to content

Commit

Permalink
Remove obsolete test for MEP config
Browse files Browse the repository at this point in the history
As of Nov, 2024, the MEP configuration is separate from the SEP/UEP.  This test
is effectively accounted for within that entire structure and set of tests.

Ref: f12b50b
  • Loading branch information
khk-globus committed Jan 16, 2025
1 parent 4e7d012 commit f6935a5
Showing 1 changed file with 0 additions and 17 deletions.
17 changes: 0 additions & 17 deletions compute_endpoint/tests/unit/test_cli_behavior.py
Original file line number Diff line number Diff line change
Expand Up @@ -583,23 +583,6 @@ def test_multi_user_config_enforces_no_engine(mock_command_ensure, ep_name, run_
assert "engine\n" in rc.stderr, (rc.stdout, rc.stderr)


def test_multi_user_instantiates_no_engine(mock_command_ensure, ep_name, run_line):
"""different from *enforces* no engine because of historical Config()
implementation that default instantiates an Executor, if it's not specified"""
ep_dir = mock_command_ensure.endpoint_config_dir / ep_name
ep_dir.mkdir(parents=True)
data = {"config": ""}

config = {"multi_user": True}
data["config"] = yaml.safe_dump(config)

to_patch = "globus_compute_endpoint.endpoint.config.config.GlobusComputeEngine"
with mock.patch(to_patch) as mock_gce_cls:
rc = run_line(f"start {ep_name}", stdin=json.dumps(data), assert_exit_code=1)

assert not mock_gce_cls.called, (rc.stdout, rc.stderr)


@pytest.mark.parametrize("use_uuid", (True, False))
@mock.patch(f"{_MOCK_BASE}get_config")
@mock.patch(f"{_MOCK_BASE}Endpoint.get_endpoint_id")
Expand Down

0 comments on commit f6935a5

Please sign in to comment.