quota.set is broken: set_() takes exactly 1 argument (10 given)

[arnisoph]

Running the module function quota.set with the syntax describe in http://docs.saltstack.com/ref/modules/all/salt.modules.quota.html#salt.modules.quota.set does not work:

[INFO    ] User root Executing command quota.set with jid 20131025205234376365
[DEBUG   ] Command details {'tgt_type': 'glob', 'jid': '20131025205234376365', 'tgt': '<MYHOSTNAME>', 'ret': '', 'user': 'root', 'arg': ['/', 'user=u98779975', 'block-hard-limit=524288000'], 'fun': 'quota.set'}
[WARNING ] TypeError encountered executing quota.set: set_() takes exactly 1 argument (10 given). See debug log for more info.  Possibly a missing arguments issue:  ArgSpec(args=['device'], varargs=None, keywords='kwargs', defaults=None)
[DEBUG   ] TypeError intercepted: set_() takes exactly 1 argument (10 given)
Traceback (most recent call last):
  File "/usr/lib/python2.7/dist-packages/salt/minion.py", line 659, in _thread_return
    return_data = func(*args, **kwargs)
TypeError: set_() takes exactly 1 argument (10 given)
Traceback (most recent call last):
  File "/usr/lib/python2.7/dist-packages/salt/minion.py", line 659, in _thread_return
    return_data = func(*args, **kwargs)
TypeError: set_() takes exactly 1 argument (10 given)
[INFO    ] Returning information for job: 20131025205234376365

The master and the minion run Debian Wheezy 7.2 with the latest packages (salt 0.17.1+dfsg-1bpo70+1dst.1, etc.).

Some unit tests might be useful.

See also http://irclog.perlgeek.de/salt/2013-10-25 (18:28 - 18:49)

[basepi]

For sure, we need tests here. Thanks for the report, we'll get this fixed.

[terminalmage]

Our CLI parser is not recognizing arguments with a dash in them as keyword arguments, so they get lumped in with the positional arguments. Working on a fix now.

[terminalmage]

Unit test added in #8305.