Skip to content
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

The assurances of the voter can be obtained from a cast vote if canup… #313

Open
wants to merge 20 commits into
base: develop
Choose a base branch
from
Open

The assurances of the voter can be obtained from a cast vote if canup… #313

wants to merge 20 commits into from

Conversation

valentinbujdoso
Copy link
Contributor

…date is true #242

@valentinbujdoso
Copy link
Contributor Author

Im not clearly understand how can I "fake" it. could you explaint it a little bite detailed?

@magwas
Copy link
Member

magwas commented Apr 4, 2019

Im not clearly understand how can I "fake" it. could you explaint it a little bite detailed?

https://blog.pragmatists.com/test-doubles-fakes-mocks-and-stubs-1a7491dfa3da

@valentinbujdoso
Copy link
Contributor Author

So something like this?

Copy link
Member

@magwas magwas left a 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.

@magwas
Copy link
Member

magwas commented Apr 9, 2019

Please provide full code coverage. Yes, even for the code I helped you with.

…e_obtained_from_a_cast_vote_if_canupdateis_true_242
@magwas
Copy link
Member

magwas commented Apr 11, 2019

only an unimplemented test is needed for ADAAssuranceManager.getAssurances()

List<String> getAssurances();

static AssuranceManager getAssuranceManager() {
return new ADAAssuranceManager();
Copy link
Contributor

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.

src/main/java/org/rulez/demokracia/pdengine/Voteable.java Outdated Show resolved Hide resolved
@valentinbujdoso
Copy link
Contributor Author

I will take a look tomorrow

…e_obtained_from_a_cast_vote_if_canupdateis_true_242
@valentinbujdoso
Copy link
Contributor Author

@magwas I created the test but it give back quite strange errors

@magwas
Copy link
Member

magwas commented Apr 16, 2019

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
@valentinbujdoso
Copy link
Contributor Author

The import javax.xml.ws.WebServiceContext; cannot be resolved.
What should I do?

@magwas
Copy link
Member

magwas commented Apr 17, 2019

seems like jax-ws is missing. window/install new software.
Is it with the eclipse in the docker image or your own setup?

@szirbucz szirbucz mentioned this pull request Apr 29, 2019
@magwas
Copy link
Member

magwas commented Jun 4, 2019

its going in in #333 @valentinbujdoso is it paid yet for you?

@valentinbujdoso
Copy link
Contributor Author

@magwas:
Elnezest, hogy el vagyok veszve, a majusom nagyon suru volt, es a juniusom meg talan meg surubb lesz, de utana jelentkezem munkara keszen.
Ezt meg nem rendeztuk, ha visszatertem az elok korebe beszelunk rola.

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.

3 participants