apache / apache/openwhisk-wskdeploy

manifest_parser_test needs more dedicated tests for output parameters...

Open
#620 0 comments 0 reactions 0 assignees View on GitHub
enhancement priority: low tests: unit
Dominant language
Go
Stars
75
Forks
73
PR merge metrics
No merged PRs in 30d

Description

Suggest updating the following test files to include outputs
- tests/dat/manifest_validate_singleline_params.yaml
- add multiple single line parm (types) and add to corresponding unit test.
- tests/dat/manifest_validate_multiline_params.yaml
- add multiple multi line parm (types) and add to corresponding unit test.
- tests/dat/manifest_validate_json_params.yaml
- add multiple json line parm (types) and add to corresponding unit test.

* please note that (for now) JSON output tests are not that interesting/unique, but in the future when we add support for JSON schema they will become more unique (and deserve dedicated unit testing).

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.