-
Notifications
You must be signed in to change notification settings - Fork 19
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Foo 1527 more unit tests #999
Conversation
Codecov Report
@@ Coverage Diff @@
## issue942 #999 +/- ##
==============================================
+ Coverage 74.06% 74.13% +0.06%
+ Complexity 17 5 -12
==============================================
Files 212 208 -4
Lines 4342 4260 -82
Branches 295 295
==============================================
- Hits 3216 3158 -58
+ Misses 1021 993 -28
- Partials 105 109 +4 |
…st into FOO-1527-more-unit-tests
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please pull. did some minor renames already...
also, you may want to "mvn spotless:apply" before pushing, or use the githooks provided (see /use_local_hooks.sh)
factcast-factus/src/main/java/org/factcast/factus/applier/DefaultProjector.java
Outdated
Show resolved
Hide resolved
factcast-factus/src/main/java/org/factcast/factus/applier/DefaultProjector.java
Show resolved
Hide resolved
factcast-factus/src/test/java/org/factcast/factus/DefaultFactusTest.java
Outdated
Show resolved
Hide resolved
factcast-factus/src/test/java/org/factcast/factus/DefaultFactusTest.java
Show resolved
Hide resolved
factcast-factus/src/test/java/org/factcast/factus/DefaultFactusTest.java
Outdated
Show resolved
Hide resolved
factcast-factus/src/test/java/org/factcast/factus/applier/DefaultProjectorTest.java
Outdated
Show resolved
Hide resolved
factcast-factus/src/test/java/org/factcast/factus/lock/LockedTestManagedProjection.java
Outdated
Show resolved
Hide resolved
factcast-factus/src/test/java/org/factcast/factus/lock/LockedTestAggregate.java
Outdated
Show resolved
Hide resolved
factcast-factus/src/test/java/org/factcast/factus/lock/LockedTestManagedProjection.java
Outdated
Show resolved
Hide resolved
factcast-factus/src/test/java/org/factcast/factus/lock/LockedTestSnapshotProjection.java
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I tried to address some of your remarks. However, I got a bit lost what is resolved already and what is not, can you resolve your comments when there is nothing left to do? Also please resolve if my changes fixed what you found.
// projection, | ||
// the lock would be broken | ||
assertThat(locked.specs()) | ||
.isEqualTo(specs); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
same same
eventSerializer, | ||
new DuplicateHandlerProjection())) | ||
// ASSERT | ||
.isInstanceOf(UnsupportedOperationException.class) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Something fancy like UnsatisfiedLinkError or LayerInstantiationException? :P
I agree, I suggest to create your own exception type though
all resolved, adding more tiny tests |
No description provided.