Mailing-List: contact dev-help@cordova.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@cordova.apache.org
Received-SPF: pass (nike.apache.org: domain of bowserj@gmail.com designates
 209.85.213.179 as permitted sender)
MIME-Version: 1.0
References: 
 <BDD863A8AA703D4D8A2639FB639A448255555D@shsmsx102.ccr.corp.intel.com>
 <CABiQX1W82xdNaPnhwZnZk4jMufAQ=mJ-95Jdodxq7-q+E23W_Q@mail.gmail.com>
 <CAOBL_k6nk4S9UK=5iW94SsDUR4uGqYC5F5EXefiV2LXEWCDH7w@mail.gmail.com>
 <CABiQX1VXSxQmP-5KUZztLZGE6UYrSezchhWTWU-r27iqHrtAVw@mail.gmail.com>
 <CAOBL_k70Azvftdzn+DAFFFjL2AFDEZ2BALXDovnwQcTry4Qg-A@mail.gmail.com>
 <BDD863A8AA703D4D8A2639FB639A4482555E3A@shsmsx102.ccr.corp.intel.com>
 <CABiQX1WxzU=+a6Cpisg6C6Srr_C3=pbPNeeDGkbrHKBhVGiR-g@mail.gmail.com>
 <CABiQX1U+ZKE3XogMDEOMuGsKsmDAdnMZBR2MQ6dJc6ATn8Eb=w@mail.gmail.com>
 <BDD863A8AA703D4D8A2639FB639A4482556DDA@shsmsx102.ccr.corp.intel.com>
 <CAEeF2TcwEDxGo_NPj0rPS4tNZcAipABSfdk=rmBJKYKi60r5jA@mail.gmail.com>
From: Joe Bowser <bowserj@gmail.com>
Date: Thu, 08 Jan 2015 16:57:42 +0000
Message-ID: 
 <CAOBL_k5SzMyFVnH=cYja=g=yydFyPEuuJ0931YPot00njPpRAQ@mail.gmail.com>
Subject: Re: How to test the cordova-android 4.0 by mobile spec
To: dev@cordova.apache.org
Content-Type: multipart/alternative; boundary=20cf302667fc4481d3050c26f08e

--20cf302667fc4481d3050c26f08e
Content-Type: text/plain; charset=UTF-8

That said, the tests failing should be filed as an issue.  I think this is
related to Andrew's recent refactor of the Splashscreen, removing it from
the core platform.  These changes are relatively minor, but can become huge
w.r.t. docs and test coverage.

On Thu Jan 08 2015 at 8:52:53 AM Michal Mocny <mmocny@chromium.org> wrote:

> It should be, but given that its on a branch (fewer passing eyeballs), the
> pace of changes on 4.0.x, and that the failing test in question is a manual
> test, things are easily overlooked.  Thanks for bringing it up.
>
> Once 3.7 ships and 4.0 moves to master for next major release, I expect the
> tests will get more eyeballs.
>
> On Wed, Jan 7, 2015 at 8:28 PM, Fu, Junwei <junwei.fu@intel.com> wrote:
>
> > Thank you  very much, it's fine after update your commit.
> >
> > Is the mobilespec being tested for Cordova 4.0, Because there are still
> > some other issue such as the splash screen can't show in splashscreen
> > manual tests.
> >
> > -----Original Message-----
> > From: agrieve@google.com [mailto:agrieve@google.com] On Behalf Of Andrew
> > Grieve
> > Sent: Thursday, January 08, 2015 2:00 AM
> > To: Andrew Grieve
> > Cc: dev
> > Subject: Re: How to test the cordova-android 4.0 by mobile spec
> >
> > Went to try this out and decided it's a bad idea. Much better is to just
> > fix createmobilespec.js to use the proper platform versions when
> generating
> > the .js. I've done this and checked it in.
> >
> > On Wed, Jan 7, 2015 at 10:45 AM, Andrew Grieve <agrieve@chromium.org>
> > wrote:
> >
> > > Hmm, I hadn't tried the key event manual tests with 4.0.x in MS. I do
> > > think that would break based on the version being MSTEST, since there
> > > is now code to make an exec() to CoreAndroid based on the
> > platformVersion.
> > >
> > > Maybe let's just set the plugin bridge name back to "App" on 4.0.x so
> > > that the js exec() will always work (but still leave the class name as
> > > "CoreAndroid")
> > >
> > > On Tue, Jan 6, 2015 at 10:06 PM, Fu, Junwei <junwei.fu@intel.com>
> wrote:
> > >
> > >> Yes, I set up mobilespec following
> > >> https://github.com/apache/cordova-mobile-spec .
> > >>
> > >> -----Original Message-----
> > >> From: Joe Bowser [mailto:bowserj@gmail.com]
> > >> Sent: Wednesday, January 07, 2015 10:54 AM
> > >> To: dev@cordova.apache.org
> > >> Subject: Re: How to test the cordova-android 4.0 by mobile spec
> > >>
> > >> So, it's not using the cordova that's installed locally via npm? I'll
> > >> have to check that again.
> > >>
> > >> On Tue Jan 06 2015 at 6:33:14 PM Andrew Grieve <agrieve@chromium.org>
> > >> wrote:
> > >>
> > >> > I've just been checkout out 4.0.x branch of cordova-android, then
> > >> > running the script. I think the script requires that
> > >> > cordova-mobile-spec, cordova-android, and cordova-js all be sibling
> > >> directories.
> > >> >
> > >> > On Tue, Jan 6, 2015 at 7:58 PM, Joe Bowser <bowserj@gmail.com>
> wrote:
> > >> >
> > >> > > I haven't been able to get mobile-spec working reliably since it
> > >> > > was refactored and we were forced to use the createmobilespec.sh
> > >> > > script
> > >> (i.e.
> > >> > > Tests don't run on Android 2.3) How did you manage to get it to
> > >> > > work with 4.0.x?
> > >> > >
> > >> > > On Tue Jan 06 2015 at 6:24:51 AM Andrew Grieve
> > >> > > <agrieve@chromium.org>
> > >> > > wrote:
> > >> > >
> > >> > > > I think you had it right. use cordova-mobile-spec@master and
> > >> > > > cordova-android@4.0.x. Failing tests are likely issues (in
> > >> > > > android or
> > >> > in
> > >> > > > the tests). Just fixed last night that the JS on 4.0.x was not
> > >> > > > properly registering with the CoreAndroid plugin due to it
> > >> > > > being renamed, so
> > >> > that
> > >> > > > could be the cause of the key event test failing.
> > >> > > >
> > >> > > > On Tue, Jan 6, 2015 at 3:47 AM, Fu, Junwei
> > >> > > > <junwei.fu@intel.com>
> > >> > wrote:
> > >> > > >
> > >> > > > > Hi,
> > >> > > > >
> > >> > > > > Because There are no cordova-mobile-spec  4.0 branch, so I
> > >> > > > > used cordova-mobile-spec master branch to test
> > >> > > > > cordova-android 4.0, but
> > >> > the
> > >> > > > key
> > >> > > > > event test cases failed. And the splash screen test cases
> > >> > > > > failed with cordova-mobile-spec 3.6 branch, I don't know how
> > >> > > > > to test the cordova-android 4.0 by mobile spec.
> > >> > > > >
> > >> > > > > Thanks,
> > >> > > > > Junwei.
> > >> > > > >
> > >> > > >
> > >> > >
> > >> >
> > >>
> > >
> > >
> >
>

--20cf302667fc4481d3050c26f08e--