Ok. Fixed and PR that with
https://github.com/apache/cordova-plugin-dialogs/pull/83.
Admins can merge it
Thanks for the help
On Tue, Aug 23, 2016 at 11:18 PM, Shazron wrote:
> In this case since its a manual test just update the existing testfile.
>
> On Tue, Aug 23, 2016 at
In this case since its a manual test just update the existing testfile.
On Tue, Aug 23, 2016 at 7:54 AM, Deshan Kalupahana <
deshankalupah...@gmail.com> wrote:
> It worked. Thanks.
> Can I change existing tests to support my bug fix or should i write a new
> file?
>
> On Tue, Aug 23, 2016 at
Bugfix pull requests that come with a test to prove they fix the defect are
our absolute favorites!
@purplecabbage
risingj.com
On Tue, Aug 23, 2016 at 7:54 AM, Deshan Kalupahana <
deshankalupah...@gmail.com> wrote:
> It worked. Thanks.
> Can I change existing tests to support my bug fix or
It worked. Thanks.
Can I change existing tests to support my bug fix or should i write a new file?
On Tue, Aug 23, 2016 at 12:23 PM, Shazron wrote:
> A slower way would be to send the pull request and let the CI test it i.e
>
Install cordova-paramedic from its repo url or npmjs. See
https://github.com/apache/cordova-paramedic
In the root of the plugin folder:
cordova-paramedic --plugin . --platform windows
On Mon, Aug 22, 2016 at 11:36 PM, Deshan Kalupahana <
deshankalupah...@gmail.com> wrote:
> Hi,
>
> I'm
A slower way would be to send the pull request and let the CI test it i.e
https://github.com/apache/cordova-plugin-dialogs/pull/82
On Mon, Aug 22, 2016 at 11:50 PM, Shazron wrote:
> Install cordova-paramedic from its repo url or npmjs. See
>
Hi,
I'm currently working on CB-11281, a bug in windows platform to learn
the code base. I fixed that. But i don't know how to unit test that.
The steps in docementation with "org.apache.cordova.test-framework" is
not clear. Help me to run the unit test.
Thanks
--
Deshan Kalupahana,