mindersec / minder / 12160307325

58%

Build Type

push

github

Committed by web-flow

Commit Message

Simplify propagation of action settings (#5132)

* Simplify propagation of action settings

Currently, action settings are passed around via a complex dance of
getter/setter functions that span multiple places in the executor.

However, this is not necessary, as we've already read these settings at
the beginning of execution, and can just set them in place where we need
them.

This simplifies this logic and leverages the aggregate profile structure
to set this. It also enforces the setting as part of the action driver
initiatlization.

Signed-off-by: Juan Antonio Osorio <ozz@stacklok.com>

* Fix unit tests

Signed-off-by: Juan Antonio Osorio <ozz@stacklok.com>

* Fix linter issues

Signed-off-by: Juan Antonio Osorio <ozz@stacklok.com>

---------

Signed-off-by: Juan Antonio Osorio <ozz@stacklok.com>

Run Details

29 of 44 new or added lines in 10 files covered. (65.91%)

6 existing lines in 5 files now uncovered.

16458 of 29822 relevant lines covered (55.19%)

38.73 hits per line