Page MenuHomeDevCentral

Check shell command receives enough arguments
ClosedPublic

Authored by dereckson on Mar 25 2018, 14:03.
Tags
None
Referenced Files
F11701239: D1496.diff
Fri, Sep 12, 16:18
Unknown Object (File)
Mon, Sep 8, 01:14
Unknown Object (File)
Tue, Sep 2, 10:08
Unknown Object (File)
Tue, Sep 2, 01:43
Unknown Object (File)
Sat, Aug 30, 09:16
Unknown Object (File)
Wed, Aug 27, 08:35
Unknown Object (File)
Mon, Aug 25, 22:22
Unknown Object (File)
Sat, Aug 23, 05:33
Subscribers
None

Details

Summary

shell allows handlers with an arbitray number of required arguments.

This change handles the case where such an argument is missing.

Test Plan
~/.shell.yml
handlers:
  tools:
    server: "dev.tools.wmflabs.org"
    command: ['become', '{{%s}}']
    interactive: True
$ shell tools
shell: Required argument is missing.
$ shell tools translate
tools.translate@tools-bastion-02:~$ 

Diff Detail

Repository
rOPS Nasqueron Operations
Lint
Lint Passed
Unit
No Test Coverage
Branch
feature/shell (branched from master)
Build Status
Buildable 2354
Build 2602: arc lint + arc unit

Event Timeline

dereckson created this revision.
This revision is now accepted and ready to land.Mar 25 2018, 14:04
This revision was automatically updated to reflect the committed changes.