I wonder if there should be an 
```
else 
  raise some error 
```
in here, otherwise some part of the app might try to call set_outcome! expecting some error, but not getting on .... is that what the ! indicates.
might be wrong about this