Skip to content

remove org.gradle.daemon=false (FINERACT-948)#840

Merged
vorburger merged 1 commit intoapache:developfrom
vorburger:FINERACT-948_gradle-daemon
May 8, 2020
Merged

remove org.gradle.daemon=false (FINERACT-948)#840
vorburger merged 1 commit intoapache:developfrom
vorburger:FINERACT-948_gradle-daemon

Conversation

@vorburger
Copy link
Member

@vorburger vorburger commented May 8, 2020

@vorburger
Copy link
Member Author

@ptuomola @awasum @xurror let's try this?

We (anyone, not only me) can always raise a revert PR, if we find it to be more trouble than beneficial...

@awasum
Copy link
Contributor

awasum commented May 8, 2020

Failure:

`

Task :compileIntegrationTestJava
/home/travis/build/apache/fineract/fineract-provider/src/integrationTest/java/org/apache/fineract/integrationtests/StaffTest.java:135: warning: [unchecked] unchecked cast
final List responseActive = (List)StaffHelper.getStaffListWithState(requestSpec, responseSpec, "active");
^
required: List
found: Object
/home/travis/build/apache/fineract/fineract-provider/src/integrationTest/java/org/apache/fineract/integrationtests/StaffTest.java:144: warning: [unchecked] unchecked cast
final List responseInactive = (List)StaffHelper.getStaffListWithState(requestSpec, responseSpec, "inactive");
^
required: List
found: Object
2 warnings
`
See Travis log: https://travis-ci.org/github/apache/fineract/builds/684752992

Seems another Integration Test is Flaky?

@vorburger
Copy link
Member Author

Seems another Integration Test is Flaky?

Yes, but not what you noticed, that one is just a warning - and #842 fixes that.

The IT failure here is FixedDepositTest > testFixedDepositAccountClosureTypeWithdrawal_WITH_HOLD_TAX - I've just created FINERACT-950 and raised #843.

@awasum
Copy link
Contributor

awasum commented May 8, 2020

Thank you.

@vorburger vorburger force-pushed the FINERACT-948_gradle-daemon branch from 94e9022 to 3459fd3 Compare May 8, 2020 18:38
@vorburger vorburger merged commit 29d64d3 into apache:develop May 8, 2020
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.

2 participants