cordova-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Joe Bowser <bows...@gmail.com>
Subject Re: [DISCUSS] Cordova-Android 4.0.0 Release
Date Wed, 28 Jan 2015 18:53:21 GMT
I can't remember a single release that had zero failures in mobile-spec, so
I don't know why we are so intent on doing this now.

On Wed Jan 28 2015 at 10:50:54 AM Murat Sutunc <muratsu@microsoft.com>
wrote:

> Just give some data point, running mobile spec I'm getting:
> - 11 failures in 4.4.2
> - 6 failures in 4.0.4
> Ideally we should bring those numbers to 0 to ensure a stable release.
>
> -----Original Message-----
> From: Joe Bowser [mailto:bowserj@gmail.com]
> Sent: Wednesday, January 28, 2015 10:45 AM
> To: dev@cordova.apache.org
> Subject: Re: [DISCUSS] Cordova-Android 4.0.0 Release
>
> On Wed Jan 28 2015 at 10:38:07 AM Andrew Grieve <agrieve@chromium.org>
> wrote:
>
> >
> > - Make CordovaActivity not implement CordovaInterface, but instead
> > provide CordovaInterface via an inner class (to solidify that you
> > can't cast the activity to CordovaInterface and expect that to work -
> > some used to do this but I think we've cleaned it all up now)
> >
> > This literally came out of nowhere.  Why are you trying so hard to
> > remove
> the embedded view use case? What if someone is implementing an activity
> that inherits from another activity like MapActivity?  This API change came
> without any discussion.
>
> All of this can be done in a few days, but I'd also like to see the dust
> > settle a bit before going forward with 4.0.0 release. E.g. At least
> > wait until we do a blog post for 3.7.0 (are you doing this?), and have
> > done a tools release that updates the pinned version to 3.7.0
> >
> >
> If someone else wants to do the blog post on that, that's fine.  And I
> agree that there should be a tools release with 3.7.0 pinned, even though
> 3.7.0 is really just a technicality so we can get 4.0.0 out IMO.
>
>
> >
> > On Wed, Jan 28, 2015 at 12:52 PM, Joe Bowser <bowserj@gmail.com> wrote:
> >
> > > Reminder: failures with plugins are not blockers.  I've run into
> > > that contact issue numerous times when testing with my personal
> > > device.  I recommend making sure that your contacts are completely
> > > clean so that you don't get these weird results.
> > >
> > > The file failures have been happening for quite a while, and those
> > > are
> > not
> > > blockers for the platform release either.  Do these failures happen
> > > on a platform other than ICS?
> > >
> > > On Wed, Jan 28, 2015, 9:06 AM Murat Sutunc <muratsu@microsoft.com>
> > wrote:
> > >
> > > > I’ve ran the mobile-spec tests on android 4.0.3 with 4.0.x and
> > > > there
> > are
> > > > some failures. I’ve searched the jira for issues but wasn’t able
> > > > to
> > find
> > > > any. Has anyone else ran into these issues before?
> > > >
> > > > org.apache.cordova.contacts.tests.tests >> Contacts
> > (navigator.contacts)
> > > > Round trip Contact tests (creating + save + delete + find).
> > > > Contacts.spec.24 Creating, saving, finding a contact should work,
> > > > after which we should not be able to find it, and we should not be
> > > > able to
> > > delete
> > > > it again.
> > > > •       Expected 2 to be 1
> > > > •       Expected 1 to be 0
> > > >          it("contacts.spec.24 Creating, saving, finding a contact
> > should
> > > > work, removing it should work, after which we should not be able
> > > > to
> > find
> > > > it, and we should not be able to delete it again.", function (done) {
> > > >               // Save method is not supported on Windows platform
> > > >               if (isWindows) {
> > > >                   pending();
> > > >                   return;
> > > >               }
> > > >               if (isWindowsPhone8) {
> > > >                   done();
> > > >                   return;
> > > >               }
> > > >               gContactObj = new Contact();
> > > >               gContactObj.name = new ContactName();
> > > >               gContactObj.name.familyName = "DeleteMe";
> > > >               gContactObj.save(function(c_obj) {
> > > >                   var findWin = function(cs) {
> > > >                       expect(cs.length).toBe(1);
> > > >                       // update to have proper saved id
> > > >                       gContactObj = cs[0];
> > > >                       gContactObj.remove(function() {
> > > >                           var findWinAgain = function(seas) {
> > > >                               expect(seas.length).toBe(0);
> > > >                               gContactObj.remove(function() {
> > > >                                   throw("success callback called
> > > > after non-existent Contact object called remove(). Test failed.");
> > > >                               }, function(e) {
> > > >                                   expect(e.code).toBe(ContactErr
> > > > or.UNKNOWN_ERROR);
> > > >                                   done();
> > > >                               });
> > > >                           };
> > > >                           var findFailAgain = function(e) {
> > > >                               throw("find error callback invoked
> > > > after delete, test failed.");
> > > >                           };
> > > >                           var obj = new ContactFindOptions();
> > > >                           obj.filter="DeleteMe";
> > > >                           obj.multiple=true;
> > > >                           navigator.contacts.find(["displayName",
> > "name",
> > > > "phoneNumbers", "emails"], findWinAgain, findFailAgain, obj);
> > > >                       }, function(e) {
> > > >                           throw("Newly created contact's remove
> > function
> > > > invoked error callback. Test failed.");
> > > >                       });
> > > >                   };
> > > >                   var findFail = fail;
> > > >                   var obj = new ContactFindOptions();
> > > >                   obj.filter="DeleteMe";
> > > >                   obj.multiple=true;
> > > >                   navigator.contacts.find(["displayName", "name",
> > > > "phoneNumbers", "emails"], findWin, findFail, obj);
> > > >               }, fail);
> > > >           });
> > > >
> > > > org.apache.cordova.file.tests.test >> file api filereader
> > > > file.spec.81 (couldn’t find a JIRA issue)
> > > > •       Expected `` to be null
> > > >         describe('FileReader', function () {
> > > >             it("file.spec.81 should have correct methods",
> > > > function ()
> > {
> > > >                 var reader = new FileReader();
> > > >                 expect(reader).toBeDefined();
> > > >                 expect(typeof
> > > reader.readAsBinaryString).toBe('function');
> > > >                 expect(typeof reader.readAsDataURL).toBe('
> function');
> > > >                 expect(typeof reader.readAsText).toBe('function');
> > > >                 expect(typeof reader.readAsArrayBuffer).
> > toBe('function');
> > > >                 expect(typeof reader.abort).toBe('function');
> > > >                 //////////// test below fails ////////////
> > > >                 ////////////   '' !== null    ////////////
> > > >                 expect(reader.result).toBe(null);
> > > >             });
> > > >         });
> > > >
> > > > org.apache.cordova.file.tests.tests >> file api parent references
> > > > file.spec.111 (couldn’t find a fire issue):
> > > > •       root.getFile succeeds, it is expected to fail.
> > > > var fileName = "traverse.file.uri";
> > > >                 // create a new file entry
> > > >                 createFile(fileName, function (entry) {
> > > >                     // lookup file system entry
> > > >                     root.getFile('../' + fileName, {
> > > >                         create : false
> > > >                     }, succeed.bind(null, done,
> > > > "root.getFile('../"+fileName+ "')- Unexpected success callback, it
> > should
> > > > not traverse abvoe the root directory"),
> > > >                     function (error) { //.....
> > > >
> > > > org.apache.cordova.file-transfer.tests.tests >> FileTransfer
> > > > methods download filetransfer.spec.6 should get 401 status on http
> > > > basic auth failure
> > > > •       Expected null to be 401
> > > >                 it('filetransfer.spec.6 should get 401 status on
> > > > http basic auth failure', function (done) {
> > > >
> > > >                     // NOTE:
> > > >                     //      using server without credentials
> > > >                     var fileURL = SERVER + '/download_basic_auth';
> > > >
> > > >                     var downloadFail = function (error) {
> > > >                         expect(error.http_status).toBe(401);
> > > >                         expect(error.http_status).not.toBe(404,
> > "Ensure "
> > > > + fileURL + " is in the white list");
> > > >                         done();
> > > >                     };
> > > >
> > > >                     transfer.download(fileURL, localFilePath,
> > > > unexpectedCallbacks.httpWin, downloadFail);
> > > >                 });
> > > >
> > > > org.apache.cordova.geolocation.tests.tests >> getCurrentPosition
> > method
> > > > success callback geolocation.spec.6 should be called with a
> > > > position
> > > object
> > > > •       Expected true to be false
> > > >             it("geolocation.spec.6 should be called with a
> > > > Position object", function (done) {
> > > >                 // this test asks for using geolocation and
> > > > interrupts autotests running.
> > > >                 // That's why we have to pending that for Windows
> > > > Store
> > > > 8.0/8.1 apps
> > > >                 if (isWindowsStore) {
> > > >                     pending();
> > > >                 }
> > > >                 navigator.geolocation.getCurrentPosition(function
> > > > (p)
> > {
> > > >                     expect(p.coords).toBeDefined();
> > > >                     expect(p.timestamp).toBeDefined();
> > > >                     done();
> > > >                 },
> > > >                 fail.bind(null, done),
> > > >                 {
> > > >                     maximumAge: 300000 // 5 minutes maximum age of
> > cached
> > > > position
> > > >                 });
> > > >             });
> > > >
> > > > org.apache.cordova.geolocation.tests.tests >> watchPosition method
> > > > success callback geolocation.spec.8 should be called with a
> > > > position
> > > object
> > > > •       Expected true to be false
> > > >             it("geolocation.spec.8 should be called with a
> > > > Position object", function (done) {
> > > >                 // this test asks for using geolocation and
> > > > interrupts autotests running.
> > > >                 // That's why we have to pending that for Windows
> > > > Store
> > > > 8.0/8.1 apps
> > > >                 if (isWindowsStore) {
> > > >                     pending();
> > > >                 }
> > > >                 successWatch = navigator.geolocation.watchPosition(
> > > >                     function (p) {
> > > >                         expect(p.coords).toBeDefined();
> > > >                         expect(p.timestamp).toBeDefined();
> > > >                         done();
> > > >                     },
> > > >                     fail.bind(null, done),
> > > >                     {
> > > >                         maximumAge: (5 * 60 * 1000) // 5 minutes
> > maximum
> > > > age of cached position
> > > >                     });
> > > >             });
> > > >
> > > >
> > > > -----Original Message-----
> > > > From: Josh Bavari [mailto:jbavari@gmail.com]
> > > > Sent: Wednesday, January 28, 2015 8:30 AM
> > > > To: dev@cordova.apache.org
> > > > Subject: Re: [DISCUSS] Cordova-Android 4.0.0 Release
> > > >
> > > > Joe and team,
> > > >
> > > > I work for Ionic and I've had some involvement with the Cordova
> > > > project since last year. At Ionic, we've released a Crosswalk
> > > > build using
> > Cordova
> > > > Android 4.0 so we can use the cordova crosswalk engine for the
> > > > ionic platform.
> > > >
> > > > I've been working with Ian and Andrew on this to gather more
> > > understanding
> > > > and to get some help along the way. I must say, excellent work,
> > everyone.
> > > >
> > > > As such, we've accumulated quite a bit of users who are actively
> > > > using Cordova Android 4.0. Currently, we've had over 10k test
> > > > trials with it,
> > > and
> > > > I'm happy to say, mostly it's been smooth.
> > > >
> > > > What I've done is made a fork to adjust a few small things, but
> > > > for the most part, we're using 4.0.
> > > >
> > > > I'd love to provide any more feedback that you'd wish.
> > > >
> > > > Thanks again for the awesome work.
> > > >
> > > > On Wed, Jan 28, 2015 at 9:21 AM, Joe Bowser <bowserj@gmail.com>
> wrote:
> > > >
> > > > > Hey
> > > > >
> > > > > So, it's finally here.  I want to see us work more on Pluggable
> > > > > Webviews, and adding the API, but I think it's time that we
> > > > > released what we've been working on for almost a year to our
> > > > > users.  I know that the API isn't exactly the most awesome we
> > > > > can make it, but it works, and I'd rather have it out at 80%
> > > > > than it sitting for a few
> > more
> > > > months in limbo.
> > > > >
> > > > > Are there any major blocking tasks that would prevent a vote
> > > > > thread that anyone knows about, or should we start firing up a
> > > > > release?  I don't think we're going to make our January date,
> > > > > but the first week of February isn't that terrible.
> > > > >
> > > > > Thoughts?
> > > > >
> > > > > Joe
> > > > >
> > > >
> > > >
> > > >
> > > > --
> > > > "Clear thoughts produce clear results."
> > > > Josh Bavari
> > > > Application Developer
> > > > Phone: 405-509-9448
> > > > Cell: 405-812-0496
> > > > Email: jbavari@gmail.com
> > > >
> > > > ------------------------------------------------------------------
> > > > --- To unsubscribe, e-mail: dev-unsubscribe@cordova.apache.org
> > > > For additional commands, e-mail: dev-help@cordova.apache.org
> > > >
> > >
> >
>

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message