Skip to content

Conversation

@Tim-Linschoten
Copy link
Member

Now extracts the input names and output events if annotations are present for the implementation.
Ensured these fields are used in the compatability function.
Updated the getImplementation to comply to our Baker interface language setup so it supports both Java & Scala correctly.

…mentations if the output is available on the implementation.

Fixed the LocalInteractionsSpec to do validations (the validation was not executed).
…F code.

Added output biding for runtime events that have the @FireEvent annotation.
Moved the annotations to a seperate package so the runtime can reuse the same annotations.
Fixed the output binding for interaction instances.
Added correct error messages for the compatibility check.
…tor object that complies to our typesafe interface instead of the InteractionExecution.Descriptor.
Added binding of the interaction input names if available.
# Conflicts:
#	bakery/baker-state/src/main/scala/com/ing/bakery/baker/BakerService.scala
#	bakery/state/src/test/scala/com/ing/bakery/baker/StateRuntimeSpec.scala
@codecov-commenter
Copy link

Codecov Report

Merging #926 (477050f) into master (d03bac5) will decrease coverage by 0.03%.
The diff coverage is 75.51%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #926      +/-   ##
==========================================
- Coverage   81.95%   81.92%   -0.04%     
==========================================
  Files         108      109       +1     
  Lines        3602     3629      +27     
  Branches      106      106              
==========================================
+ Hits         2952     2973      +21     
- Misses        650      656       +6     
Impacted Files Coverage Δ
...g/bakery/interaction/RemoteInteractionClient.scala 100.00% <ø> (ø)
.../bakery/interaction/RemoteInteractionService.scala 86.11% <ø> (ø)
...ain/scala/com/ing/baker/runtime/model/BakerF.scala 37.39% <0.00%> (ø)
...r/runtime/serialization/InteractionExecution.scala 100.00% <ø> (ø)
...n/scala/com/ing/baker/runtime/akka/AkkaBaker.scala 74.62% <50.00%> (ø)
...la/com/ing/baker/runtime/model/InteractionsF.scala 67.34% <67.85%> (+0.68%) ⬆️
...om/ing/baker/runtime/serialization/JsonCodec.scala 100.00% <100.00%> (ø)
...ing/baker/runtime/serialization/JsonDecoders.scala 88.88% <100.00%> (+0.42%) ⬆️
...ing/baker/runtime/serialization/JsonEncoders.scala 100.00% <100.00%> (ø)
... and 1 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 d03bac5...477050f. Read the comment docs.

jacum
jacum previously approved these changes May 17, 2021
@Tim-Linschoten Tim-Linschoten merged commit ff31c21 into master May 18, 2021
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.

4 participants