You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@fineract.apache.org by "Mohit Sinha (JIRA)" <ji...@apache.org> on 2019/07/23 07:28:00 UTC

[jira] [Resolved] (FINERACT-780) HookIntegrationTest is flaky

     [ https://issues.apache.org/jira/browse/FINERACT-780?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Mohit Sinha resolved FINERACT-780.
----------------------------------
       Resolution: Fixed
    Fix Version/s: 1.4.0

> HookIntegrationTest is flaky
> ----------------------------
>
>                 Key: FINERACT-780
>                 URL: https://issues.apache.org/jira/browse/FINERACT-780
>             Project: Apache Fineract
>          Issue Type: Bug
>            Reporter: Michael Vorburger.ch
>            Assignee: Mohit Sinha
>            Priority: Major
>             Fix For: 1.4.0
>
>          Time Spent: 20m
>  Remaining Estimate: 0h
>
> Our org.apache.fineract.integrationtests.HookIntegrationTest seems to be "flaky" and occassionally arbitrary fails.
> This seems to have happened e.g. in https://github.com/apache/fineract/pull/557 as well as again in https://github.com/apache/fineract/pull/607.
> I suspect it's related to this in the code of that test:
> {code:java}
> // Subject to https://echo-webhook.herokuapp.com being up
> // See
> // http://www.jamesward.com/2014/06/11/testing-webhooks-was-a-pain-so-i-fixed-the-glitch
> final String payloadURL = "http://echo-webhook.herokuapp.com:80/Z7RXoCBdLSFMDrpn?";
> this.hookHelper.createHook(payloadURL);
> {code}
> In an ideal world, even an integration test should not depend on an external service running on some arbitrary 3rd party website.
> What someone probably could do is to instead start some sort of Echo Server written in Java in-process as part of that test. Something like this may already exist as a library somewhere, or otherwise it probably wouldn't be terribly hard to write it.
> If this happens more often, perhaps just temporarily adding an {{@Igore}} for this test would be in order (until someone can do above).



--
This message was sent by Atlassian JIRA
(v7.6.14#76016)