Skip to content

Conversation

yann-eugone
Copy link
Contributor

The idea behind "routing" is : "the ability to pick the appropriate component for a given subject".

Routing is a "generic" behaviour which can be leveraged at many places in the app.

I started with an implementation for ItemProcessor & ItemWriter.

@codecov
Copy link

codecov bot commented Jul 11, 2021

Codecov Report

Merging #20 (abf58ba) into 0.x (d8d73b5) will increase coverage by 4.91%.
The diff coverage is 100.00%.

Impacted file tree graph

@@             Coverage Diff              @@
##                0.x      #20      +/-   ##
============================================
+ Coverage     83.52%   88.44%   +4.91%     
- Complexity      477      517      +40     
============================================
  Files            66       75       +9     
  Lines          1360     1515     +155     
============================================
+ Hits           1136     1340     +204     
+ Misses          224      175      -49     
Impacted Files Coverage Δ
src/batch/src/Finder/CallbackFinder.php 100.00% <100.00%> (ø)
src/batch/src/Finder/ClassMapFinder.php 100.00% <100.00%> (ø)
.../batch/src/Job/Item/Processor/RoutingProcessor.php 100.00% <100.00%> (ø)
src/batch/src/Job/Item/Writer/RoutingWriter.php 100.00% <100.00%> (ø)
src/batch/src/Test/Finder/DummyFinder.php 100.00% <100.00%> (ø)
...src/Test/Job/Item/Processor/TestDebugProcessor.php 100.00% <100.00%> (ø)
...batch/src/Test/Job/Item/Reader/TestDebugReader.php 100.00% <100.00%> (ø)
src/batch/src/Test/Job/Item/TestDebugComponent.php 100.00% <100.00%> (ø)
...batch/src/Test/Job/Item/Writer/TestDebugWriter.php 100.00% <100.00%> (ø)
... and 12 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d8d73b5...abf58ba. Read the comment docs.

@yann-eugone yann-eugone merged commit ad5dcbb into 0.x Jul 13, 2021
@yann-eugone yann-eugone deleted the component-routing branch July 14, 2021 09:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant