Manage wusa 3010 return code

[tlemarchand]

...and add additional comment in state in case of return code different from 0.

What does this PR do?

This PR fixes a bug.
If a .msu file installation requires reboot, currently wusa state will fail with an un-catched CommandExecutionError, even if installation is done and requires only a reboot.
Also, this PR adds an additional comment in case of CommandExecutionError, so that we can easily know that a reboot is needed, or in case of failure what is the problem.

What issues does this PR fix or reference?

Previous Behavior

---

ID: KB4054590
Function: wusa.installed
Result: False
Comment: An exception occurred in this state: Traceback (most recent call last):
File "C:\salt\bin\lib\site-packages\salt\state.py", line 1919, in call
**cdata['kwargs'])
File "C:\salt\bin\lib\site-packages\salt\loader.py", line 1918, in wrapper
return f(*args, **kwargs)
File "C:\salt\bin\lib\site-packages\salt\states\win_wusa.py", line 90, in installed
salt'wusa.install'
File "C:\salt\bin\lib\site-packages\salt\modules\win_wusa.py", line 143, in install
raise CommandExecutionError('Unknown error: {0}'.format(ret_code))
CommandExecutionError: Unknown error: 3010

New Behavior

Tests written?

No

Commits signed with GPG?

No

[tlemarchand]

Hi,

Is it possible to move forward on this ?

[twangboy]

@tlemarchand Could you rebase this against master. And please add some tests.

[tlemarchand]

@tlemarchand Could you rebase this against master. And please add some tests.

@twangboy , I rebased against master, fixed existing tests and added a new test for the 3010 return code.