Skip to content

Commit

Permalink
Merge remote-tracking branch 'upstream/dev' into spark-rapids-tools-253
Browse files Browse the repository at this point in the history
  • Loading branch information
parthosa committed Oct 11, 2023
2 parents 1e4eeb4 + fb6d20f commit 1c6c2a2
Showing 1 changed file with 12 additions and 0 deletions.
12 changes: 12 additions & 0 deletions user_tools/tests/spark_rapids_tools_ut/test_tool_argprocessor.py
Original file line number Diff line number Diff line change
Expand Up @@ -157,6 +157,18 @@ def test_cluster_props_no_eventlogs(self, get_ut_data_dir, tool_name, csp, prop_
assert tool_args['runtimePlatform'] == CspEnv(csp)
self.validate_args_w_savings_enabled(tool_name, tool_args)

@pytest.mark.parametrize('tool_name', ['qualification', 'profiling'])
@register_triplet_test([ArgValueCase.IGNORE, ArgValueCase.UNDEFINED, ArgValueCase.UNDEFINED])
def test_cluster_props_no_eventlogs_on_prem(self, capsys, tool_name):
# Missing eventlogs is not accepted for onPrem
with pytest.raises(SystemExit) as pytest_wrapped_e:
AbsToolUserArgModel.create_tool_args(tool_name,
platform='onprem')
assert pytest_wrapped_e.type == SystemExit
captured = capsys.readouterr()
# Verify there is no URL in error message
assert 'https://' not in captured.err

@pytest.mark.skip(reason='Unit tests are not completed yet')
def test_arg_cases_coverage(self):
args_keys = [
Expand Down

0 comments on commit 1c6c2a2

Please sign in to comment.