Adjustments and updates, Adding basic unit testing

[namachieli]

Fixes

• Adjusted ./bin/template.jinja so that token is not set to required: true Token is now set as a required parameter in actions, regardless of pack config. #60
• Fix python 2.7/3.6 compatibility for ./actions/run.py run.py incompatible logic in python 3.x #63 RuntimeError: dictionary changed size during iteration #58 Code seems to have compatibility issues with 3.7 #59

Additions

• CODEOWNER file
• Basic unit testing and structure for expanded unit testing

After updating ./bin/template.jinja, ./bin/generate_openapi.py was re-ran to remove required from the parameter token for all actions generated.

Reviews would be greatly appreciated!

[nmaludy]

Not a huge fan of having this override here, previously when i had to override for a single method i implemented this here: https://github.com/StackStorm-Exchange/stackstorm-slack/blob/master/bin/generate_openapi.py#L19

Would be probably good to add a "global" override or something here: https://github.com/StackStorm-Exchange/stackstorm-slack/blob/master/bin/generate_openapi.py#L156

[namachieli]

Ok, I will take a look at that, thanks!

[namachieli]

Added a simple loop and a constant, removed the previous change to ./bin/template.jinja
Regenerated actions and now token['required'] is being overridden to false

stackstorm-slack/bin/generate_openapi.py

Line 47 in 9407835

GLOBAL_PARAM_OVERRIDES = {

stackstorm-slack/bin/generate_openapi.py

Line 167 in 9407835

for p_override_key, p_override_value in six.iteritems(GLOBAL_PARAM_OVERRIDES):

Does this approach it more how you were seeing it?