how about we change this dependency injection to just rely on arguments passed to this action?

```
    def branch_tracker
      @branch_tracker ||= (options[:branch_tracker] || BranchTracker)
    end
```

this way we reduce the need to go around various objects and still retain the ability to stub it.

`options` would be initialized in the upmost class (AbstractAction)