refactor: simplify integration status implementation and check

syndesisio / syndesis-rest

The API for Syndesis - a flexible, customizable, cloud-hosted platform that provides core integration capabilities as a service. It leverages Red Hat's existing product architecture using OpenShift Online/Dedicated and Fuse Integration Services.

https://syndesis-staging.b6ff.rh-idev.openshiftapps.com/api/v1/

Apache License 2.0

6 stars 17 forks source link

refactor: simplify integration status implementation and check #729

Closed zregvart closed 7 years ago

zregvart commented 7 years ago

Integration::getStatus returns a Optional<IntegrationStatus> that is non empty in all cases with a single property (currentState) populated. This simplifies that method to return just the IntegrationRevisionState. Also simplifies and makes more explicit the logic of counting active integrations of a user.

jimmibot commented 7 years ago

@zregvart, thanks! @KurtStam, please review this.

pure-bot[bot] commented 7 years ago

Pull request approved by @KurtStam - applying approved label