Bsc1172340

[noelpower]

No description provided.

[jreidinger]

Is there really ldap-server and krb-server CLI commands? I do not see it. If you just do not support CLI at all and there there is no good replacement, you can omit that "help" parameter and it will just write CLI not supported.

[Firstyear]

Sorry I don't understand this comment. Can you explain what you mean here?

[jreidinger]

Well, what I mean here. That this help text is showed when user print some command line arguments so e.g.:

yast2 auth-cli help

say use Use 'auth-cli' directly. but such command does not exists. We use it often to provide existing CLI tool that user can use e.g. for firewall we recommend firewall-cli to use. But if I get it here is not direct replacement for functionality that auth-cli provide that works on command line. So in that case I suggest to use solution we use e.g. in storage-ng where is also no suitable CLI tool and just say CLI not available - yast/yast-storage-ng#1170 (comment)

so simply you can remove that msg and "help" key from map and it will print just "CLI not supported"

[coveralls]

Pull Request Test Coverage Report for Build 722258648

• 0 of 22 (0.0%) changed or added relevant lines in 3 files are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage decreased (-0.5%) to 11.609%

---

Changes Missing Coverage	Covered Lines	Changed/Added Lines	%
src/clients/krb-server.rb	0	5	0.0%
src/clients/ldap-server.rb	0	5	0.0%
src/lib/authserver/cli/auth-cli.rb	0	12	0.0%

Totals
Change from base Build 646759445:	-0.5%
Covered Lines:	57
Relevant Lines:	491

---

💛 - Coveralls

[yast-bot]

✔️ Public Jenkins job #14 successfully finished
✔️ Created OBS submit request #883276