search

Tonny1494 / ISII-WORKSHOPS-REALIDAD-AUMENTADA

0 stars 0 forks source link

New business rule required #1

Open lemendoza opened 3 years ago

lemendoza commented 3 years ago

Dear developers,

Based on the impact of the pandemic in the 2020 results, the board of the company has approved a new business rull beginning january 2021: If the customer is male, over 25 and younger than 45 then it is needed to subtract $100 from the base premium.

Therefore, we ask you to modify the premium calculation adding this new rule.

Best regards,

LEMM

allisonEspi commented 3 years ago

Ok, We will modify the GetCarPremium function.

allisonEspi commented 3 years ago

I proceed to categorize the issue

allisonEspi commented 3 years ago

We will proceed to process the request, @Tonny1494 will be the assigned analyst, @allisonEspi will develop the solution , and @eileenbelen will perform the tests.

Tonny1494 commented 3 years ago

the impact analysis has been performed @lemendoza Impact analysis.docx

allisonEspi commented 3 years ago

The solution has been built and include the code: https://github.com/Tonny1494/ISII-WORKSHOPS-REALIDAD-AUMENTADA/blob/1921a820fc402008ea96c625218eb8510b7dd485/ec.edu.espol.workshops.second/src/main/java/ec/edu/espol/workshops/second/CarInsurance.java#L53-L55

lemendoza commented 3 years ago

@Tonny1494 What is the relationship of the presented analysis with the new feature? It presents a very generic description, not related to the feature requested. In addition, the time required for implementation is exaggerated. It is necessary to present a more adequate analysis.

Tonny1494 commented 3 years ago

@lemendoza

@Tonny1494 What is the relationship of the presented analysis with the new feature? It presents a very generic description, not related to the feature requested. In addition, the time required for implementation is exaggerated. It is necessary to present a more adequate analysis.

@lemendoza I have just replaced the document for your approval Impact analysis.docx

eileenbelen commented 3 years ago

System Testing -Realidad Aumentada.docx

lemendoza commented 3 years ago

@Tonny1494 @allisonEspi @eileenbelen I approve the analysis so that you can continue with the workshop. However, the analysis does not support that "just add one line of code to the function" requires 5 hours ...

lemendoza commented 3 years ago

@eileenbelen Review the test plan. In particular, test case 4. According to the new business rule, the value obtained is not the one indicated; therefore, the test case cannot pass.