tmt icon indicating copy to clipboard operation
tmt copied to clipboard

Add test for step data update with CLI options

Open happz opened this issue 1 year ago • 1 comments

This functionality has been inside BasePlugin, and today a bug was reported with its handling of flag-like options. Therefore, this patch:

  • extracts the code into a standalone function, complementing the rest of normalization code,
  • fixes the bug,
  • fixes a few more issues discovered, mostly readability,
  • adds a pile of comment WRT various related issues, and
  • adds a unit test.

happz avatar Mar 23 '23 13:03 happz