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 iclelland@google.com designates
 209.85.219.45 as permitted sender)
MIME-Version: 1.0
Sender: iclelland@google.com
In-Reply-To: 
 <CAKP6k2viS6Dtn00Ywm4jGa74PpkKXqwbkLg9sy+pVeX8wLN5_A@mail.gmail.com>
References: 
 <CAMTxO42=6CHt3Mp9N=M10HcU0TAQa6Or0Moj8yM7MZSxePeG8g@mail.gmail.com>
 <CAK_TSXKJpPwC-2wHUfHhgMqoUpdEbNHUmtgYVg7x4-jaPE=M_A@mail.gmail.com>
 <CAKP6k2vOJWt-561NEzHcHpe4494SBxMCB0nMWCS=PXX3nLGKXQ@mail.gmail.com>
 <CAKP6k2viS6Dtn00Ywm4jGa74PpkKXqwbkLg9sy+pVeX8wLN5_A@mail.gmail.com>
From: Ian Clelland <iclelland@chromium.org>
Date: Wed, 8 Jan 2014 15:40:35 -0500
Message-ID: 
 <CAK_TSX+=bhppS+nd6gvd7LuL6JvWTLxU2FeBAB78urDfXz3OAQ@mail.gmail.com>
Subject: Re: IC: current state of IOS master
To: "dev@cordova.apache.org" <dev@cordova.apache.org>
Content-Type: multipart/alternative; boundary=001a1134a71a75e60604ef7b82d8

--001a1134a71a75e60604ef7b82d8
Content-Type: text/plain; charset=UTF-8

They are out of sync, for sure. The mobile-spec tests should be considered
canonical, I think -- that's what is actually being run by the CI server,
and by everyone who is doing manual testing, as far as I know. (Anyone who
knows different: now is the time to chime in :) )

We're very close to being able to split all of the tests out of mobile
spec; we have a test runner which is close to being ready that will pull in
tests from individual plugins. Once that's ready, we'll be able to move the
tests from MS back into plugins.

Most plugins right now have a set of tests that was copied in circa Cordova
2.9/3.0, and so they'll mostly need to be refreshed when we finally move to
plugin-hosted-tests.

(And I think the tests are back to passing now, from the look of the CI
dashboard -- thanks for doing that! :) )

Ian


On Wed, Jan 8, 2014 at 3:23 PM, Jesse <purplecabbage@gmail.com> wrote:

> I reverted the merge and pushed to master.
>
> So should the mobile-spec tests, or the tests in each plugin repo be
> considered the root?  I fear they are out of sync.
>
> @purplecabbage
> risingj.com
>
>
> On Wed, Jan 8, 2014 at 10:54 AM, Jesse <purplecabbage@gmail.com> wrote:
>
> > I did the premature merge for CB-5602, I'll revert it.
> >
> > @purplecabbage
> > risingj.com
> >
> >
> > On Wed, Jan 8, 2014 at 8:47 AM, Ian Clelland <iclelland@chromium.org
> >wrote:
> >
> >> I think that this has to do with CB-5602. It looks like the file api
> tests
> >> were overwritten with a version from the cordova-plugin-file repo, which
> >> would have been rather stale, and the merge caused the whole suite to
> >> contain errors.
> >>
> >> We appear to have a completely different issue in the CI server, because
> >> the tests should have failed on all platforms, not just iOS/release.
> >>
> >> [I've replied to CB-5602 as well; not sure which is the better place for
> >> discussion]
> >>
> >> Ian
> >>
> >>
> >>
> >> On Wed, Jan 8, 2014 at 11:08 AM, David Kemp <drkemp@chromium.org>
> wrote:
> >>
> >> > Unfortunately, our iOS deploy has been broken for a while. I just
> spent
> >> > some time getting it back on its feet and find that master does not
> pass
> >> > tests. I do not have a simple 'blame list' because many commits have
> >> passed
> >> > since the test deployed correctly.
> >> >
> >> > All tests pass on iOS 3.3.x
> >> >
> >> > All tests pass on Android Master and 3.3.x
> >> >
> >> > On iOS master 7 file tests are failing on iOS 6.1.3 and 7.04
> >> >
> >> >
> >> >                "*spec*": "File API DirectoryEntry file.spec.25
> >> > DirectoryEntry.getDirectory: create new dir with space
> >> resolveFileSystemURI
> >> > with encoded URI.",
> >> >
> >> >                        "*exception*": "timeout: timed out after 7500
> >> msec
> >> > waiting for win never called",
> >> >
> >> >                "*spec*": "File API File file.spec.40 should be define
> >> File
> >> > attributes.",
> >> >
> >> >                        "*exception*": "Expected undefined to be
> >> defined.",
> >> >
> >> >                "*spec*": "File API Entry file.spec.63 copyTo:
> directory
> >> > that does not exist.",
> >> >
> >> >                        "*exception*": "TypeError: 'null' is not an
> >> object
> >> > (evaluating 'parent.filesystem.__format__') in
> >> >
> >> >
> >>
> file:///var/mobile/Applications/836872AD-CFED-4BE7-A316-45353151C586/mobilespec.app/www/plugins/org.apache.cordova.file/www/Entry.js
> >> > (line 165)",
> >> >
> >> >                        "*exception*": "timeout: timed out after 7500
> >> msec
> >> > waiting for itCopy never called",
> >> >
> >> >                "*spec*": "File API Entry file.spec.79 moveTo:
> directory
> >> > that does not exist.",
> >> >
> >> >                        "*exception*": "timeout: timed out after 7500
> >> msec
> >> > waiting for itMove never called",
> >> >
> >> >                "*spec*": "File API read method file.spec.82 should
> error
> >> > out on non-existent file.",
> >> >
> >> >                        "*exception*": "timeout: timed out after 7500
> >> msec
> >> > waiting for verifier never called",
> >> >
> >> >                "*spec*": "File API FileWriter file.spec.97 should be
> >> able
> >> > to write and append to file, File object.",
> >> >
> >> >                        "*exception*": "timeout: timed out after 7500
> >> msec
> >> > waiting for verifier",
> >> >
> >> >                "*spec*": "File API Backwards compatibility
> file.spec.109
> >> > should be able to resolve a file:/// URL.",
> >> >
> >> >                        "*exception*": "ReferenceError: Can't find
> >> variable:
> >> > joinURL in
> >> >
> >> >
> >>
> file:///var/mobile/Applications/836872AD-CFED-4BE7-A316-45353151C586/mobilespec.app/www/autotest/tests/file.tests.js
> >> > (line 3599)",
> >> >
> >>
> >
> >
>

--001a1134a71a75e60604ef7b82d8--