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

Missing Unit Test for Quote Process, #41

Open
ghost opened this issue Jan 22, 2013 · 1 comment
Open

Missing Unit Test for Quote Process, #41

ghost opened this issue Jan 22, 2013 · 1 comment
Assignees

Comments

@ghost
Copy link

ghost commented Jan 22, 2013

The car insurance demo quote process is missing a working unit test.

Suggest we run as user admin , driver of 45 and a Ford Fiesta and test for a price of 299.

@jamesread
Copy link
Contributor

Derek, we have https://github.com/RedHatUKI/rh6pd/blob/master/rh6pd-processController/src/test/java/org/rh6pd/processController/InsuranceFlowTest.java

Although admittedly this is in the rh6pd-processController project, which is pretty odd. It would be better off in rh6pd-businessRules project..

We also have some unit tests stored in the wrong place at the moment;

https://github.com/RedHatUKI/rh6pd/tree/master/rh6pd-businessRules/src/main/test/org/jbpm/evaluation/carinsurance

  • Should be src/test/... not /src/main/test/...)

@ally-jarrett can you spend some time reviewing both branches and workout what we want to keep out of both of them (and which one is more "modern"?). We should then moved the insurance related tests into the businessRules project under the correct path.

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

No branches or pull requests

2 participants