-
Notifications
You must be signed in to change notification settings - Fork 13
The assurances of the voter can be obtained from a cast vote if canup… #313
base: develop
Are you sure you want to change the base?
The assurances of the voter can be obtained from a cast vote if canup… #313
Conversation
…e_obtained_from_a_cast_vote_if_canupdateis_true_242
src/main/java/org/rulez/demokracia/pdengine/SessionFactoryManager.java
Outdated
Show resolved
Hide resolved
src/test/java/org/rulez/demokracia/pdengine/UnimplementedTests.java
Outdated
Show resolved
Hide resolved
src/main/java/org/rulez/demokracia/pdengine/SessionFactoryManager.java
Outdated
Show resolved
Hide resolved
src/main/java/org/rulez/demokracia/pdengine/SessionFactoryManager.java
Outdated
Show resolved
Hide resolved
src/test/java/org/rulez/demokracia/pdengine/UnimplementedTests.java
Outdated
Show resolved
Hide resolved
src/test/java/org/rulez/demokracia/pdengine/UnimplementedTests.java
Outdated
Show resolved
Hide resolved
Im not clearly understand how can I "fake" it. could you explaint it a little bite detailed? |
…e_obtained_from_a_cast_vote_if_canupdateis_true_242
https://blog.pragmatists.com/test-doubles-fakes-mocks-and-stubs-1a7491dfa3da |
src/test/java/org/rulez/demokracia/pdengine/UnimplementedTests.java
Outdated
Show resolved
Hide resolved
src/main/java/org/rulez/demokracia/pdengine/FakeADAAssuranceProvider.java
Outdated
Show resolved
Hide resolved
src/main/java/org/rulez/demokracia/pdengine/SessionFactoryManager.java
Outdated
Show resolved
Hide resolved
src/main/java/org/rulez/demokracia/pdengine/SessionFactoryManager.java
Outdated
Show resolved
Hide resolved
src/test/java/org/rulez/demokracia/pdengine/UnimplementedTests.java
Outdated
Show resolved
Hide resolved
So something like this? |
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.
see poc/assurancemanager branch in my repo to understand how dependency injection (old school) works.
Build on that code.
src/main/java/org/rulez/demokracia/pdengine/FakeADAAssuranceProvider.java
Outdated
Show resolved
Hide resolved
…e_obtained_from_a_cast_vote_if_canupdateis_true_242
Please provide full code coverage. Yes, even for the code I helped you with. |
…e_obtained_from_a_cast_vote_if_canupdateis_true_242
only an unimplemented test is needed for ADAAssuranceManager.getAssurances() |
List<String> getAssurances(); | ||
|
||
static AssuranceManager getAssuranceManager() { | ||
return new ADAAssuranceManager(); |
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.
Test it to kill zombie.
I will take a look tomorrow |
…e_obtained_from_a_cast_vote_if_canupdateis_true_242
@magwas I created the test but it give back quite strange errors |
you have received a pull request yesterday with the resolution of the conflicts. |
…_can_be_obtained_from_a_cast_vote_if_canupdateis_true_242_java10_merge merge with current edemo/develop
…e_obtained_from_a_cast_vote_if_canupdateis_true_242
The import javax.xml.ws.WebServiceContext; cannot be resolved. |
seems like jax-ws is missing. window/install new software. |
its going in in #333 @valentinbujdoso is it paid yet for you? |
@magwas: |
…date is true #242