add default queue_event_player events_when_finished: "queue_event_complete"

[bosh]

If the config provides events_when_complete, those are used instead. The queue_event is included in either case, along with the optional args provided in the config.

Test updated (saw em fail without my code change!) and doc comment added :)

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[bosh]

Alternatively we could provide a default via config_spec.yaml, but since we have the doc strings next to the event posting in code, I like having the name there instead of spread across files. (And if someone were to specify their config with a blank value overriding the default, they might get a crash anyway, vs. the device code forcing a safe default)

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[avanwinkle]

As discussed on the google group, adding a default value to queue events finished will cause undesirable behavior and should be avoided.

If passing the queue event or other params in the callback has a use case, that could be useful thougH!