agent: Cleanup config

Browse Source

This commit clean up config parsing and testing code to make it a bit more easy to maintain.
- Adds `with_context` from anyhow to include the underlying error. This helps to understand what exactly went wrong.
- Uses ensure and bail as a shorter alternative for `if` checks.
- TestData in test_parse_cmdline is now implements Default to reduce boilerplate code
- Remove `make_err` as it doesn’t make any sense.

Fixes: #2177

Signed-off-by: Maksym Pavlenko <pavlenko.maksym@gmail.com>

...

This commit is contained in:

Maksym Pavlenko 2021-07-02 14:28:43 -07:00

parent 3fe0af6a9b

commit e6b1766f6b

1 changed files with 186 additions and 550 deletions

Show all changes Ignore whitespace when comparing lines Ignore changes in amount of whitespace Ignore changes in whitespace at EOL

Show Stats Download Patch File Download Diff File Expand all files Collapse all files

736

src/agent/src/config.rs

View File

File diff suppressed because it is too large Load Diff