Commit5748d37

committed

test(unit): increase V4 CLI coverage

1 parent8bf53c8 commit5748d37Copy full SHA for 5748d37

File tree

+42

-0

lines changed

+42

-0

lines changed

Lines changed: 10 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -21,6 +21,16 @@ def fake_object(fake_manager):`
`21`	`21`	`returnhelpers.FakeObject(fake_manager, {"attr1":"foo","alist": [1,2,3]})`
`22`	`22`
`23`	`23`
	`24`	`+@pytest.fixture`
	`25`	`+deffake_object_no_id(fake_manager):`
	`26`	`+returnhelpers.FakeObjectWithoutId(fake_manager, {})`
	`27`	`+`
	`28`	`+`
	`29`	`+@pytest.fixture`
	`30`	`+deffake_object_long_repr(fake_manager):`
	`31`	`+returnhelpers.FakeObjectWithLongRepr(fake_manager, {"test":"a"*100})`
	`32`	`+`
	`33`	`+`
`24`	`34`	`@pytest.fixture`
`25`	`35`	`deffake_object_with_parent(fake_manager_with_parent):`
`26`	`36`	`returnhelpers.FakeObject(`

Lines changed: 9 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -15,6 +15,15 @@ class FakeObject(base.RESTObject):`
`15`	`15`	`pass`
`16`	`16`
`17`	`17`
	`18`	`+classFakeObjectWithoutId(base.RESTObject):`
	`19`	`+_id_attr=None`
	`20`	`+`
	`21`	`+`
	`22`	`+classFakeObjectWithLongRepr(base.RESTObject):`
	`23`	`+_id_attr=None`
	`24`	`+_repr_attr="test"`
	`25`	`+`
	`26`	`+`
`18`	`27`	`classOtherFakeObject(FakeObject):`
`19`	`28`	`_id_attr="foo"`
`20`	`29`

Lines changed: 23 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -1,14 +1,17 @@`
`1`	`1`	`importargparse`
`2`	`2`	`importio`
`3`	`3`	`importos`
	`4`	`+importsys`
`4`	`5`	`importtempfile`
`5`	`6`	`fromcontextlibimportredirect_stderr# noqa: H302`
	`7`	`+fromunittestimportmock`
`6`	`8`
`7`	`9`	`importpytest`
`8`	`10`
`9`	`11`	`importgitlab.base`
`10`	`12`	`fromgitlabimportcli`
`11`	`13`	`fromgitlab.exceptionsimportGitlabError`
	`14`	`+fromgitlab.v4importcliasv4_cli`
`12`	`15`
`13`	`16`
`14`	`17`	`@pytest.mark.parametrize(`
`@@ -146,3 +149,23 @@ def test_v4_parser():`
`146`	`149`	`)`
`147`	`150`	`actions=user_subparsers.choices["create"]._option_string_actions`
`148`	`151`	`assertactions["--name"].required`
	`152`	`+`
	`153`	`+`
	`154`	`+@pytest.mark.skipif(sys.version_info< (3,8),reason="added in 3.8")`
	`155`	`+deftest_legacy_display_without_fields_warns(fake_object_no_id):`
	`156`	`+printer=v4_cli.LegacyPrinter()`
	`157`	`+`
	`158`	`+withmock.patch("builtins.print")asmocked:`
	`159`	`+printer.display(fake_object_no_id,obj=fake_object_no_id)`
	`160`	`+`
	`161`	`+assert"No default fields to show"inmocked.call_args.args[0]`
	`162`	`+`
	`163`	`+`
	`164`	`+@pytest.mark.skipif(sys.version_info< (3,8),reason="added in 3.8")`
	`165`	`+deftest_legacy_display_with_long_repr_truncates(fake_object_long_repr):`
	`166`	`+printer=v4_cli.LegacyPrinter()`
	`167`	`+`
	`168`	`+withmock.patch("builtins.print")asmocked:`
	`169`	`+printer.display(fake_object_long_repr,obj=fake_object_long_repr)`
	`170`	`+`
	`171`	`+assertlen(mocked.call_args.args[0])<80`

Comments

(0)