[ 
https://issues.apache.org/jira/browse/FINERACT-891?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17082549#comment-17082549
 ] 

Michael Vorburger commented on FINERACT-891:
--------------------------------------------

[https://github.com/apache/fineract/pull/762] includes a proposed fix for this 
(I'm not sure it's correct)

> ClientSavingsIntegrationTest has Dead Code due to if (Calendar.DAY_OF_MONTH 
> >= 22)
> ----------------------------------------------------------------------------------
>
>                 Key: FINERACT-891
>                 URL: https://issues.apache.org/jira/browse/FINERACT-891
>             Project: Apache Fineract
>          Issue Type: Bug
>            Reporter: Michael Vorburger
>            Priority: Major
>
> while staring at ClientSavingsIntegrationTest for FINERACT-852, I noticed 
> that in it has x2 Dead Code due to {{if (Calendar.DAY_OF_MONTH >= 22)}} ... 
> that will always be false. I'm guessing that has was meant to check ... I'm 
> not sure, presumably the transactionDate? That doesn't really make much sense 
> either, as that's hard-coded to {{transactionDate.set(Calendar.DAY_OF_MONTH, 
> 22);}} just 2 lines above.
> Someone who better understands what that test does should probably review it.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to