Running a step should be determined by the success of the step it dependsOn

[ceelias]

Currently, dependsOn determines when one step needs another step to run before it which is useful for how we currently write our integrations. I have noticed a few times where a step that other steps dependsOn fails, it continues to run the dependent steps which will inevitably fail.

If a step truly dependsOn another, it should know if it failed or succeeded to determine whether it should run or not.

In a case I see in Knowbe4, the fetch-users and fetch-training-camps steps dependsOn the fetch-groups step. When fetch-groups fails, it causes the other 2 to fail since they are looking for a map with relevant data. There are some workarounds to solve this such as:

1. Set the fetch-groups step to be fatal so the integration terminates. This may not be ideal as there are other steps which don't care about groups.
2. In this case, you could simply do a check to see if the map it expects has any data and if so run it, otherwise skip over everything. This could work but in my opinion isn't the cleanest.

This would require us to keep track of the status of each step to check against.