Skip to content

Commit

Permalink
Fixed review finding: Updated skip reason and docstring for irrelevan…
Browse files Browse the repository at this point in the history
…t test case
  • Loading branch information
ckunki committed Oct 10, 2024
1 parent 35515e6 commit 35935ba
Showing 1 changed file with 4 additions and 3 deletions.
7 changes: 4 additions & 3 deletions test/unit/deployment/test_cli.py
Original file line number Diff line number Diff line change
Expand Up @@ -254,14 +254,15 @@ def test_container_file():
"Covered by test_default_values()"


@pytest.mark.skip(reason="Not implemented, yet")
@pytest.mark.skip(reason="Test case to be implemented in derived applications")
def test_container_url():
"""
For the following two arguments to LanguageContainerDeployer.create()
there are no corresponding CLI options defined:
the corresponding CLI options need to defined in actual CLI applications
using ``language_container_deployer_main()``
- container_url: Optional[str] = None,
- container_name: Optional[str] = None):
Hence this test case currently cannot be run.
Hence there is no related test case in the current file.
"""

0 comments on commit 35935ba

Please sign in to comment.