Return-Path: X-Original-To: apmail-cordova-dev-archive@www.apache.org Delivered-To: apmail-cordova-dev-archive@www.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 4AAA61727B for ; Thu, 12 Feb 2015 22:00:06 +0000 (UTC) Received: (qmail 38737 invoked by uid 500); 12 Feb 2015 22:00:06 -0000 Delivered-To: apmail-cordova-dev-archive@cordova.apache.org Received: (qmail 38694 invoked by uid 500); 12 Feb 2015 22:00:06 -0000 Mailing-List: contact dev-help@cordova.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@cordova.apache.org Delivered-To: mailing list dev@cordova.apache.org Received: (qmail 38682 invoked by uid 99); 12 Feb 2015 22:00:05 -0000 Received: from athena.apache.org (HELO athena.apache.org) (140.211.11.136) by apache.org (qpsmtpd/0.29) with ESMTP; Thu, 12 Feb 2015 22:00:05 +0000 X-ASF-Spam-Status: No, hits=1.5 required=5.0 tests=HTML_MESSAGE,RCVD_IN_DNSWL_LOW,SPF_PASS X-Spam-Check-By: apache.org Received-SPF: pass (athena.apache.org: domain of purplecabbage@gmail.com designates 209.85.214.182 as permitted sender) Received: from [209.85.214.182] (HELO mail-ob0-f182.google.com) (209.85.214.182) by apache.org (qpsmtpd/0.29) with ESMTP; Thu, 12 Feb 2015 22:00:02 +0000 Received: by mail-ob0-f182.google.com with SMTP id nt9so12955803obb.13 for ; Thu, 12 Feb 2015 13:57:26 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :content-type; bh=L523axGVQWHsXd5Q6ISZGppMpdcmpGaNdUe5z4C4220=; b=XCqW7SMYcVXxUQ6Q2+yS7w9dU0/18fz4OZLoin8EoSkfLYWWQ4GaxoJMZI9ktXtG8S 3RDUPG41Psoz0MNVldVKudcJWL0d5QrBHs1tLxq4LMsAyuj70JGvgTWzRGWepm/Tyd3v oIbxO0+Yt1pCyq4MM+uibTQ4s+X8rat/Shw6xc8uVCIRN95mh3k0wYaq63MDrDnv6JAr G5/vF83djYZMskbQyYJin+MJm/6xkXjArNzKjuElqz0CBm+EGywaUBBB7KxOLaU2hQ1N f2Z85cZ47wkT3JbIX+ezo7NILSEpB5MRtuE+3ybTXSdaUsDTcCC5TZNwABq1/l6ahQPD 8gqw== MIME-Version: 1.0 X-Received: by 10.182.210.197 with SMTP id mw5mr4468979obc.26.1423778246555; Thu, 12 Feb 2015 13:57:26 -0800 (PST) Received: by 10.76.122.105 with HTTP; Thu, 12 Feb 2015 13:57:26 -0800 (PST) In-Reply-To: References: Date: Thu, 12 Feb 2015 13:57:26 -0800 Message-ID: Subject: Re: Adding unit tests to travis for Android/iOS From: Jesse To: "dev@cordova.apache.org" Content-Type: multipart/alternative; boundary=001a11c29be8a00034050eeb347e X-Virus-Checked: Checked by ClamAV on apache.org --001a11c29be8a00034050eeb347e Content-Type: text/plain; charset=UTF-8 Currently cordova-ios, cordova-android, cordova-windows, cordova-wp8 all have travis||appveyor integration in their github mirrors. These tests should definitely be extended to include much much more. For the most part they are currently just testing project creation, but ios and android can run on travisCI emulators so we can include runtime platform tests also. On a similar note, I have been working on plugin tests via what I have been calling cordova-paramedic [1] Essentially cordova-paramedic takes a platform and a plugin, creates the project, installs the plugin, installs the plugin tests, installs cordova-plugin-test-framework and runs the tests. There is also a local server started, and the medic.json file points back to it, so the command line running the tests via the emulator can see the results. I have this working for the device && file plugins on ios[2] via my own forks. We are going to need INFRA to setup the github appveyor+travis integration for EVERY core plugin. I will be writing considerably more about this in the next couple weeks, I am currently working through some issues with wp8 emulator (a vm) being run on a vm in appveyor land, as the emulator and servers are on different virtual networks. [1] ... provides advanced levels of care at the point of illness or injury https://github.com/purplecabbage/cordova-paramedic [2] https://travis-ci.org/purplecabbage/cordova-plugin-device/builds/48971021 { "mobilespec":{ "specs":8, "failures":0, "results":[ ] }, "platform":"ios", "version":"8.1", "timestamp":1422667864, "model":"x86_64" } Results:: ran 8 specs with 0 failures @purplecabbage risingj.com On Thu, Feb 12, 2015 at 1:34 PM, Joe Bowser wrote: > +1. I know Travis has JUnit integration and thanks to Android Studio, the > only easy way to debug Cordova without copying and pasting code out of a > generated project is to open the test project. There should be no excuse > to not write tests since it's harder/more annoying to use the CLI when > working on platforms, since you need to make sure you copy everything you > changed. > > On Thu, Feb 12, 2015, 1:22 PM Murat Sutunc wrote: > > > There has been a lot of discussion about tests lately and I feel the urge > > to jump in and make some suggestions regarding the way we test things. > I'm > > still fairly new in the community and sometimes don't have the whole > > background story, so please if I'm missing something let me know. > > > > Currently we have bunch of platforms with failing tests. I'm assuming > > that, when these tests were first added, they were all passing. My theory > > is that over time platforms moved forward but tests remain stagnant and > now > > bunch of them are failing. I think we should consider running these tests > > automatically to ensure: > > > > a) We keep maintaining unit-tests tests regularly > > > > b) We run all unit tests before checking in code > > > > c) Reduce the friction for new developers > > For unit tests, I think it's a good idea to integrate them into CI > builds. > > This will probably add some extra time on travis/appveyor but it > shouldn't > > take more than 1hr. I think it's a fair trade off to wait some more and > > have an overall more stable Cordova. We can always skip the CI build step > > if it's an urgent fix (security, critical bug, etc). I've checked travis > > and both iOS and Android configs come with SDK tools preinstalled, so > there > > are no blockers to going forward with this. > > > > Thoughts? > > > > > > > --001a11c29be8a00034050eeb347e--