search

ru-fix / completable-reactor

CompletableReactor framework makes it easier to create business flows that have concurrently running parts and complex execution branching.
https://ru-fix.github.io/completable-reactor
MIT License
12 stars 2 forks source link

9 add or else transition #11

Closed KuznetsovVlad closed 5 years ago

KuznetsovVlad commented 5 years ago
  1. Есть тест OrElseGraph - где проверяется onElse переход. Пока что он в модуле example. Хотел бы его перенести и дописать немного, вопрос в какой лучше модуль?
  2. Еще вопрос по тестам, я не нашел теста конкретно на ExcecutionBuilder, там изменение, хотелось бы тоже покрыть тестами. Мы функционал ExecutionBuilder через что тестируем?
swarmshine commented 5 years ago

onElse это часть completable-reactor-runtime, так что тесты на поведение onElse предлагаю прямо в этом же модуле разместить. На ExecutionBuilder отдельных тестов не было. Есть тесты на работу всего реактора ru.fix.completable.reactor.runtime.tests.GlCompletableReactorTest ru.fix.completable.reactor.graph.kotlin.KotlinGraphTest и рядом с ними классы лежат. Можно unit закинуть чтобы закрыть функционал самого ExecutionBuilder, я в первую очередь старался закрыть кейсы для реактора в целом и функционал ExecutionBuilder проверялся получается уже косвенно.

swarmshine commented 5 years ago

Модуль example - сделан как чать документации больше для примера использования графа нежели для его тестирования. Для тестовых целей предлагаю использовать reactor-runtime/tests или reactor-graph-kotlin/tests