You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cordova.apache.org by Max Woghiren <ma...@chromium.org> on 2014/03/25 21:57:15 UTC
Many CLI tests failing
David Kemp just sent an email about medic—everyone can access it!
If you take a look, you'll see that there are a bunch of CLI test failures.
If you click "Tools_CLI", then the most recent "Build #" (currently #74),
and finally "stdio" under "Test CLI failed", you'll see a list of failed
tests. There are 21 right now.
Lorin and Josh, they appear to be related to commits you've made in the
last week. Do you mind taking a look?
Thanks!
-Max
Re: Many CLI tests failing
Posted by Lorin Beer <lo...@gmail.com>.
Thanks Max,
took a look, some tests for functionality that isn't implemented made it
into master. I've reverted the offending change.
As policy, I'm behind reverting any commit that fails tests or breaks
functionality. It pollutes everyone's dev process with issues that should
remain isolated. I say this as one of the guilty parties this time.
- Lorin
On Tue, Mar 25, 2014 at 6:36 PM, Max Woghiren <ma...@chromium.org> wrote:
> Josh is having trouble accessing the buildbot page, so here is the
> relevant information.
>
> Lorin's relevant commit:
>
>
> https://git-wip-us.apache.org/repos/asf?p=cordova-cli.git;a=commit;h=79e571953c2d3459b459c02eb50e753308fd453d
>
> Josh's relevant commits:
>
>
> https://git-wip-us.apache.org/repos/asf?p=cordova-cli.git;a=commit;h=07181e4ee3e358b7af7674a4ae68fe06552b2655(platform parsers)
>
> https://git-wip-us.apache.org/repos/asf?p=cordova-cli.git;a=commit;h=edecaf9193e8b23c6b7e209c6893491497021082(create.js)
>
> Lorin's commit led to errors 2-5. The remainder are the result of Josh's
> commits.
>
> The text of the failed tests:
>
> 1) hooker should throw if provided directory is not a cordova project
> Message:
> [31mExpected function to throw Not a Cordova project, can't use hooks. , but it threw Not a Cordova project ("/var/folders/p5/b_w_jqzj2jd64nz22dnv7xzc0000gn/T/e2e-test/hooks_test"), can't use hooks. [0m
> Stacktrace:
> Error: Expected function to throw Not a Cordova project, can't use hooks. , but it threw Not a Cordova project ("/var/folders/p5/b_w_jqzj2jd64nz22dnv7xzc0000gn/T/e2e-test/hooks_test"), can't use hooks.
> at null.<anonymous> (/Users/medic/buildbot/slave_common/Tools_CLI/build/cordova-cli/e2e/hooker.spec.js:53:12)
>
> 2) create command callback should return null if a callback parameter is used
> Message:
> [31mReferenceError: foobar is not defined [0m
> Stacktrace:
> ReferenceError: foobar is not defined
> at null.<anonymous> (/Users/medic/buildbot/slave_common/Tools_CLI/build/cordova-cli/spec/create.spec.js:79:66)
>
> 3) create command callback should return null if a callback parameter is used
> Message:
> [31mtimeout: timed out after 10000 msec waiting for something to happen [0m
> Stacktrace:
> undefined
>
> 4) create command callback should call the callback function if callback parameter is used
> Message:
> [31mReferenceError: foobar is not defined [0m
> Stacktrace:
> ReferenceError: foobar is not defined
> at null.<anonymous> (/Users/medic/buildbot/slave_common/Tools_CLI/build/cordova-cli/spec/create.spec.js:79:66)
>
> 5) create command callback should call the callback function if callback parameter is used
> Message:
> [31mtimeout: timed out after 10000 msec waiting for something to happen [0m
> Stacktrace:
> undefined
>
> 6) create command success should create top-level directory structure appropriate for a cordova-cli project
> Message:
> [31mtimeout: timed out after 10000 msec waiting for spec to complete [0m
> Stacktrace:
> undefined
>
> 7) create command success should create hooks directory
> Message:
> [31mtimeout: timed out after 10000 msec waiting for spec to complete [0m
> Stacktrace:
> undefined
>
> 8) create command success should by default use cordova-app-hello-world as www assets
> Message:
> [31mtimeout: timed out after 10000 msec waiting for spec to complete [0m
> Stacktrace:
> undefined
>
> 9) create command success should try to lazy load custom www location if specified
> Message:
> [31mtimeout: timed out after 10000 msec waiting for spec to complete [0m
> Stacktrace:
> undefined
>
> 10) create command success should add a missing www/config.xml
> Message:
> [31mtimeout: timed out after 10000 msec waiting for spec to complete [0m
> Stacktrace:
> undefined
>
> 11) create command success should not replace an existing www/config.xml
> Message:
> [31mtimeout: timed out after 10000 msec waiting for spec to complete [0m
> Stacktrace:
> undefined
>
> 12) android project parser constructions should throw if provided directory does not contain an AndroidManifest.xml
> Message:
> [31mExpected function to throw The provided path "some/path" is not an Android project. , but it threw CordovaError is not defined [0m
> Stacktrace:
> Error: Expected function to throw The provided path "some/path" is not an Android project. , but it threw CordovaError is not defined
> at null.<anonymous> (/Users/medic/buildbot/slave_common/Tools_CLI/build/cordova-cli/spec/metadata/android_parser.spec.js:71:16)
>
> 13) blackberry10 project parser constructions should throw an exception with a path that is not a native blackberry project
> Message:
> [31mExpected function to throw The provided path "/some/path" is not a Cordova BlackBerry10 project. , but it threw CordovaError is not defined [0m
> Stacktrace:
> Error: Expected function to throw The provided path "/some/path" is not a Cordova BlackBerry10 project. , but it threw CordovaError is not defined
> at null.<anonymous> (/Users/medic/buildbot/slave_common/Tools_CLI/build/cordova-cli/spec/metadata/blackberry_parser.spec.js:85:16)
>
> 14) blackberry10 project parser check_requirements should fire a callback if the blackberry-deploy shell-out fails
> Message:
> [31mReferenceError: CordovaError is not defined [0m
> Stacktrace:
> ReferenceError: CordovaError is not defined
> at /Users/medic/buildbot/slave_common/Tools_CLI/build/cordova-cli/src/metadata/blackberry10_parser.js:50:26
> at Object.<anonymous> (/Users/medic/buildbot/slave_common/Tools_CLI/build/cordova-cli/spec/metadata/blackberry_parser.spec.js:99:29)
> at Function.module.exports.check_requirements (/Users/medic/buildbot/slave_common/Tools_CLI/build/cordova-cli/src/metadata/blackberry10_parser.js:48:19)
> at null.<anonymous> (/Users/medic/buildbot/slave_common/Tools_CLI/build/cordova-cli/spec/metadata/blackberry_parser.spec.js:101:56)
> at null.<anonymous> (/Users/medic/buildbot/slave_common/Tools_CLI/build/cordova-cli/node_modules/jasmine-node/lib/jasmine-node/async-callback.js:45:37)
>
> 15) ios project parser constructions should throw if provided directory does not contain an xcodeproj file
> Message:
> [31mExpected function to throw The provided path is not a Cordova iOS project. , but it threw CordovaError is not defined [0m
> Stacktrace:
> Error: Expected function to throw The provided path is not a Cordova iOS project. , but it threw CordovaError is not defined
> at null.<anonymous> (/Users/medic/buildbot/slave_common/Tools_CLI/build/cordova-cli/spec/metadata/ios_parser.spec.js:60:16)
>
> 16) windows8 project parser constructions should throw if provided directory does not contain a jsproj file
> Message:
> [31mExpected function to throw The provided path "/some/path" is not a Windows 8 project. Error: No .jsproj file. , but it threw CordovaError is not defined [0m
> Stacktrace:
> Error: Expected function to throw The provided path "/some/path" is not a Windows 8 project. Error: No .jsproj file. , but it threw CordovaError is not defined
> at null.<anonymous> (/Users/medic/buildbot/slave_common/Tools_CLI/build/cordova-cli/spec/metadata/windows8_parser.spec.js:72:16)
>
> 17) windows8 project parser check_requirements should fire a callback if there is an error during shelling out
> Message:
> [31mReferenceError: CordovaError is not defined [0m
> Stacktrace:
> ReferenceError: CordovaError is not defined
> at /Users/medic/buildbot/slave_common/Tools_CLI/build/cordova-cli/src/metadata/windows8_parser.js:64:26
> at Object.<anonymous> (/Users/medic/buildbot/slave_common/Tools_CLI/build/cordova-cli/spec/metadata/windows8_parser.spec.js:87:17)
> at Function.module.exports.check_requirements (/Users/medic/buildbot/slave_common/Tools_CLI/build/cordova-cli/src/metadata/windows8_parser.js:61:19)
> at null.<anonymous> (/Users/medic/buildbot/slave_common/Tools_CLI/build/cordova-cli/spec/metadata/windows8_parser.spec.js:89:52)
> at null.<anonymous> (/Users/medic/buildbot/slave_common/Tools_CLI/build/cordova-cli/node_modules/jasmine-node/lib/jasmine-node/async-callback.js:45:37)
>
> 18) wp7 project parser constructions should throw if provided directory does not contain a csproj file
> Message:
> [31mExpected function to throw The provided path "/some/path" is not a Windows Phone 7 project. Error: No .csproj file. , but it threw CordovaError is not defined [0m
> Stacktrace:
> Error: Expected function to throw The provided path "/some/path" is not a Windows Phone 7 project. Error: No .csproj file. , but it threw CordovaError is not defined
> at null.<anonymous> (/Users/medic/buildbot/slave_common/Tools_CLI/build/cordova-cli/spec/metadata/wp7_parser.spec.js:78:16)
>
> 19) wp7 project parser check_requirements should fire a callback if there is an error during shelling out
> Message:
> [31mReferenceError: CordovaError is not defined [0m
> Stacktrace:
> ReferenceError: CordovaError is not defined
> at /Users/medic/buildbot/slave_common/Tools_CLI/build/cordova-cli/src/metadata/wp7_parser.js:60:26
> at Object.<anonymous> (/Users/medic/buildbot/slave_common/Tools_CLI/build/cordova-cli/spec/metadata/wp7_parser.spec.js:92:29)
> at Function.module.exports.check_requirements (/Users/medic/buildbot/slave_common/Tools_CLI/build/cordova-cli/src/metadata/wp7_parser.js:57:19)
> at null.<anonymous> (/Users/medic/buildbot/slave_common/Tools_CLI/build/cordova-cli/spec/metadata/wp7_parser.spec.js:94:47)
> at null.<anonymous> (/Users/medic/buildbot/slave_common/Tools_CLI/build/cordova-cli/node_modules/jasmine-node/lib/jasmine-node/async-callback.js:45:37)
>
> 20) wp8 project parser constructions should throw if provided directory does not contain a csproj file
> Message:
> [31mExpected function to throw The provided path "/some/path" is not a Windows Phone 8 project. Error: No .csproj file. , but it threw CordovaError is not defined [0m
> Stacktrace:
> Error: Expected function to throw The provided path "/some/path" is not a Windows Phone 8 project. Error: No .csproj file. , but it threw CordovaError is not defined
> at null.<anonymous> (/Users/medic/buildbot/slave_common/Tools_CLI/build/cordova-cli/spec/metadata/wp8_parser.spec.js:78:16)
>
> 21) wp8 project parser check_requirements should fire a callback if there is an error during shelling out
> Message:
> [31mReferenceError: CordovaError is not defined [0m
> Stacktrace:
> ReferenceError: CordovaError is not defined
> at /Users/medic/buildbot/slave_common/Tools_CLI/build/cordova-cli/src/metadata/wp8_parser.js:60:26
> at Object.<anonymous> (/Users/medic/buildbot/slave_common/Tools_CLI/build/cordova-cli/spec/metadata/wp8_parser.spec.js:92:29)
> at Function.module.exports.check_requirements (/Users/medic/buildbot/slave_common/Tools_CLI/build/cordova-cli/src/metadata/wp8_parser.js:57:19)
> at null.<anonymous> (/Users/medic/buildbot/slave_common/Tools_CLI/build/cordova-cli/spec/metadata/wp8_parser.spec.js:94:47)
> at null.<anonymous> (/Users/medic/buildbot/slave_common/Tools_CLI/build/cordova-cli/node_modules/jasmine-node/lib/jasmine-node/async-callback.js:45:37)
>
> Please take a look when you have a moment.
>
> Thanks!
> -Max
>
>
Re: Many CLI tests failing
Posted by Andrew Grieve <ag...@chromium.org>.
I think David updated his (and medic's) version of npm and that fixed the
problem. I think the version he had was not installing devDependencies
automatically.
On Fri, Mar 28, 2014 at 11:14 AM, Steven Gill <st...@gmail.com>wrote:
> Medic is now showing plugman tests passing. No idea what is going on. They
> still pass fine for me.
>
> On Mar 28, 2014 6:07 AM, "David Kemp" <dr...@chromium.org> wrote:
>
> > I have just verified the plugman state and on my machine I get the same 7
> > errors that Medic gets.
> > All of them are related to version comparisions.
> >
> >
> >
> >
> > On Thu, Mar 27, 2014 at 6:17 PM, Steven Gill <st...@gmail.com>
> > wrote:
> >
> > > All tests seem to be passing now.
> > >
> > > Medic is showing plugman tests failing after the tizen commits, but
> they
> > > are passing fine for me.
> > >
> > >
> > > On Wed, Mar 26, 2014 at 10:05 AM, Andrew Grieve <agrieve@chromium.org
> > > >wrote:
> > >
> > > > On Wed, Mar 26, 2014 at 9:05 AM, Josh Soref <js...@blackberry.com>
> > > wrote:
> > > >
> > > > > https://github.com/blackberry/cordova-cli/tree/cb_6337
> > > > >
> > > > > Should cover both of my breaks.
> > > > >
> > > > > I¹m waiting for our CI tooling to confirm it¹s happy before I
> submit
> > a
> > > > > pull request for it.
> > > > >
> > > > > Maybe sometime in the near future I¹ll try to rework the tests to
> use
> > > an
> > > > > in-memory-psuedo-filesystem so that we can stop having to add all
> > these
> > > > > stupid lies.
> > > > >
> > > > > Tests shouldn¹t be so brittle that changing code which has no
> > relation
> > > to
> > > > > a test results in a broken test. If we¹re lying and pretending that
> > > > > there¹s e.g. a "lib/dir", then that directory needs to sufficiently
> > > exist
> > > > > ‹ adding code which makes a perfectly reasonable assumption ³this
> > > > > directory which someone claimed exists exists² shouldn¹t break a
> test
> > > > > which pretended the directory existed.
> > > > >
> > > >
> > > > Agree 100% with your sentiments here, and also agree that an
> in-memory
> > > > filesystem is the best way to make our tests decent.
> > > >
> > > > >
> > > > > The CordovaError bit is different (and entirely my fault), I think
> I
> > > must
> > > > > have failed to commit the requires update, because I remember doing
> > the
> > > > > greps for it...
> > > > >
> > > > >
> > > >
> > >
> >
>
Re: Many CLI tests failing
Posted by Steven Gill <st...@gmail.com>.
Medic is now showing plugman tests passing. No idea what is going on. They
still pass fine for me.
On Mar 28, 2014 6:07 AM, "David Kemp" <dr...@chromium.org> wrote:
> I have just verified the plugman state and on my machine I get the same 7
> errors that Medic gets.
> All of them are related to version comparisions.
>
>
>
>
> On Thu, Mar 27, 2014 at 6:17 PM, Steven Gill <st...@gmail.com>
> wrote:
>
> > All tests seem to be passing now.
> >
> > Medic is showing plugman tests failing after the tizen commits, but they
> > are passing fine for me.
> >
> >
> > On Wed, Mar 26, 2014 at 10:05 AM, Andrew Grieve <agrieve@chromium.org
> > >wrote:
> >
> > > On Wed, Mar 26, 2014 at 9:05 AM, Josh Soref <js...@blackberry.com>
> > wrote:
> > >
> > > > https://github.com/blackberry/cordova-cli/tree/cb_6337
> > > >
> > > > Should cover both of my breaks.
> > > >
> > > > I¹m waiting for our CI tooling to confirm it¹s happy before I submit
> a
> > > > pull request for it.
> > > >
> > > > Maybe sometime in the near future I¹ll try to rework the tests to use
> > an
> > > > in-memory-psuedo-filesystem so that we can stop having to add all
> these
> > > > stupid lies.
> > > >
> > > > Tests shouldn¹t be so brittle that changing code which has no
> relation
> > to
> > > > a test results in a broken test. If we¹re lying and pretending that
> > > > there¹s e.g. a "lib/dir", then that directory needs to sufficiently
> > exist
> > > > ‹ adding code which makes a perfectly reasonable assumption ³this
> > > > directory which someone claimed exists exists² shouldn¹t break a test
> > > > which pretended the directory existed.
> > > >
> > >
> > > Agree 100% with your sentiments here, and also agree that an in-memory
> > > filesystem is the best way to make our tests decent.
> > >
> > > >
> > > > The CordovaError bit is different (and entirely my fault), I think I
> > must
> > > > have failed to commit the requires update, because I remember doing
> the
> > > > greps for it...
> > > >
> > > >
> > >
> >
>
Re: Many CLI tests failing
Posted by David Kemp <dr...@chromium.org>.
I have just verified the plugman state and on my machine I get the same 7
errors that Medic gets.
All of them are related to version comparisions.
On Thu, Mar 27, 2014 at 6:17 PM, Steven Gill <st...@gmail.com> wrote:
> All tests seem to be passing now.
>
> Medic is showing plugman tests failing after the tizen commits, but they
> are passing fine for me.
>
>
> On Wed, Mar 26, 2014 at 10:05 AM, Andrew Grieve <agrieve@chromium.org
> >wrote:
>
> > On Wed, Mar 26, 2014 at 9:05 AM, Josh Soref <js...@blackberry.com>
> wrote:
> >
> > > https://github.com/blackberry/cordova-cli/tree/cb_6337
> > >
> > > Should cover both of my breaks.
> > >
> > > I¹m waiting for our CI tooling to confirm it¹s happy before I submit a
> > > pull request for it.
> > >
> > > Maybe sometime in the near future I¹ll try to rework the tests to use
> an
> > > in-memory-psuedo-filesystem so that we can stop having to add all these
> > > stupid lies.
> > >
> > > Tests shouldn¹t be so brittle that changing code which has no relation
> to
> > > a test results in a broken test. If we¹re lying and pretending that
> > > there¹s e.g. a "lib/dir", then that directory needs to sufficiently
> exist
> > > ‹ adding code which makes a perfectly reasonable assumption ³this
> > > directory which someone claimed exists exists² shouldn¹t break a test
> > > which pretended the directory existed.
> > >
> >
> > Agree 100% with your sentiments here, and also agree that an in-memory
> > filesystem is the best way to make our tests decent.
> >
> > >
> > > The CordovaError bit is different (and entirely my fault), I think I
> must
> > > have failed to commit the requires update, because I remember doing the
> > > greps for it...
> > >
> > >
> >
>
Re: Many CLI tests failing
Posted by Steven Gill <st...@gmail.com>.
All tests seem to be passing now.
Medic is showing plugman tests failing after the tizen commits, but they
are passing fine for me.
On Wed, Mar 26, 2014 at 10:05 AM, Andrew Grieve <ag...@chromium.org>wrote:
> On Wed, Mar 26, 2014 at 9:05 AM, Josh Soref <js...@blackberry.com> wrote:
>
> > https://github.com/blackberry/cordova-cli/tree/cb_6337
> >
> > Should cover both of my breaks.
> >
> > I¹m waiting for our CI tooling to confirm it¹s happy before I submit a
> > pull request for it.
> >
> > Maybe sometime in the near future I¹ll try to rework the tests to use an
> > in-memory-psuedo-filesystem so that we can stop having to add all these
> > stupid lies.
> >
> > Tests shouldn¹t be so brittle that changing code which has no relation to
> > a test results in a broken test. If we¹re lying and pretending that
> > there¹s e.g. a "lib/dir", then that directory needs to sufficiently exist
> > ‹ adding code which makes a perfectly reasonable assumption ³this
> > directory which someone claimed exists exists² shouldn¹t break a test
> > which pretended the directory existed.
> >
>
> Agree 100% with your sentiments here, and also agree that an in-memory
> filesystem is the best way to make our tests decent.
>
> >
> > The CordovaError bit is different (and entirely my fault), I think I must
> > have failed to commit the requires update, because I remember doing the
> > greps for it...
> >
> >
>
Re: Many CLI tests failing
Posted by Andrew Grieve <ag...@chromium.org>.
On Wed, Mar 26, 2014 at 9:05 AM, Josh Soref <js...@blackberry.com> wrote:
> https://github.com/blackberry/cordova-cli/tree/cb_6337
>
> Should cover both of my breaks.
>
> I¹m waiting for our CI tooling to confirm it¹s happy before I submit a
> pull request for it.
>
> Maybe sometime in the near future I¹ll try to rework the tests to use an
> in-memory-psuedo-filesystem so that we can stop having to add all these
> stupid lies.
>
> Tests shouldn¹t be so brittle that changing code which has no relation to
> a test results in a broken test. If we¹re lying and pretending that
> there¹s e.g. a "lib/dir", then that directory needs to sufficiently exist
> ‹ adding code which makes a perfectly reasonable assumption ³this
> directory which someone claimed exists exists² shouldn¹t break a test
> which pretended the directory existed.
>
Agree 100% with your sentiments here, and also agree that an in-memory
filesystem is the best way to make our tests decent.
>
> The CordovaError bit is different (and entirely my fault), I think I must
> have failed to commit the requires update, because I remember doing the
> greps for it...
>
>
Re: Many CLI tests failing
Posted by Josh Soref <js...@blackberry.com>.
https://github.com/blackberry/cordova-cli/tree/cb_6337
Should cover both of my breaks.
I¹m waiting for our CI tooling to confirm it¹s happy before I submit a
pull request for it.
Maybe sometime in the near future I¹ll try to rework the tests to use an
in-memory-psuedo-filesystem so that we can stop having to add all these
stupid lies.
Tests shouldn¹t be so brittle that changing code which has no relation to
a test results in a broken test. If we¹re lying and pretending that
there¹s e.g. a "lib/dir", then that directory needs to sufficiently exist
‹ adding code which makes a perfectly reasonable assumption ³this
directory which someone claimed exists exists² shouldn¹t break a test
which pretended the directory existed.
The CordovaError bit is different (and entirely my fault), I think I must
have failed to commit the requires update, because I remember doing the
greps for it...
Re: Many CLI tests failing
Posted by David Kemp <dr...@google.com>.
We will have to look into why some people can't get to the machine. It's
not that the server is down or slow, its being blocked for some people.
On Mar 25, 2014 9:37 PM, "Max Woghiren" <ma...@chromium.org> wrote:
> Josh is having trouble accessing the buildbot page, so here is the relevant
> information.
>
> Lorin's relevant commit:
>
>
> https://git-wip-us.apache.org/repos/asf?p=cordova-cli.git;a=commit;h=79e571953c2d3459b459c02eb50e753308fd453d
>
> Josh's relevant commits:
>
>
> https://git-wip-us.apache.org/repos/asf?p=cordova-cli.git;a=commit;h=07181e4ee3e358b7af7674a4ae68fe06552b2655(platform
> parsers)
>
> https://git-wip-us.apache.org/repos/asf?p=cordova-cli.git;a=commit;h=edecaf9193e8b23c6b7e209c6893491497021082(create.js)
>
> Lorin's commit led to errors 2-5. The remainder are the result of Josh's
> commits.
>
> The text of the failed tests:
>
> 1) hooker should throw if provided directory is not a cordova project
> Message:
> [31mExpected function to throw Not a Cordova project, can't use
> hooks. , but it threw Not a Cordova project
> ("/var/folders/p5/b_w_jqzj2jd64nz22dnv7xzc0000gn/T/e2e-test/hooks_test"),
> can't use hooks. [0m
> Stacktrace:
> Error: Expected function to throw Not a Cordova project, can't
> use hooks. , but it threw Not a Cordova project
> ("/var/folders/p5/b_w_jqzj2jd64nz22dnv7xzc0000gn/T/e2e-test/hooks_test"),
> can't use hooks.
> at null.<anonymous>
>
> (/Users/medic/buildbot/slave_common/Tools_CLI/build/cordova-cli/e2e/hooker.spec.js:53:12)
>
> 2) create command callback should return null if a callback parameter is
> used
> Message:
> [31mReferenceError: foobar is not defined [0m
> Stacktrace:
> ReferenceError: foobar is not defined
> at null.<anonymous>
>
> (/Users/medic/buildbot/slave_common/Tools_CLI/build/cordova-cli/spec/create.spec.js:79:66)
>
> 3) create command callback should return null if a callback parameter is
> used
> Message:
> [31mtimeout: timed out after 10000 msec waiting for something to
> happen [0m
> Stacktrace:
> undefined
>
> 4) create command callback should call the callback function if
> callback parameter is used
> Message:
> [31mReferenceError: foobar is not defined [0m
> Stacktrace:
> ReferenceError: foobar is not defined
> at null.<anonymous>
>
> (/Users/medic/buildbot/slave_common/Tools_CLI/build/cordova-cli/spec/create.spec.js:79:66)
>
> 5) create command callback should call the callback function if
> callback parameter is used
> Message:
> [31mtimeout: timed out after 10000 msec waiting for something to
> happen [0m
> Stacktrace:
> undefined
>
> 6) create command success should create top-level directory
> structure appropriate for a cordova-cli project
> Message:
> [31mtimeout: timed out after 10000 msec waiting for spec to complete
> [0m
> Stacktrace:
> undefined
>
> 7) create command success should create hooks directory
> Message:
> [31mtimeout: timed out after 10000 msec waiting for spec to complete
> [0m
> Stacktrace:
> undefined
>
> 8) create command success should by default use
> cordova-app-hello-world as www assets
> Message:
> [31mtimeout: timed out after 10000 msec waiting for spec to complete
> [0m
> Stacktrace:
> undefined
>
> 9) create command success should try to lazy load custom www
> location if specified
> Message:
> [31mtimeout: timed out after 10000 msec waiting for spec to complete
> [0m
> Stacktrace:
> undefined
>
> 10) create command success should add a missing www/config.xml
> Message:
> [31mtimeout: timed out after 10000 msec waiting for spec to complete
> [0m
> Stacktrace:
> undefined
>
> 11) create command success should not replace an existing www/config.xml
> Message:
> [31mtimeout: timed out after 10000 msec waiting for spec to complete
> [0m
> Stacktrace:
> undefined
>
> 12) android project parser constructions should throw if provided
> directory does not contain an AndroidManifest.xml
> Message:
> [31mExpected function to throw The provided path "some/path" is
> not an Android project. , but it threw CordovaError is not defined [0m
> Stacktrace:
> Error: Expected function to throw The provided path "some/path"
> is not an Android project. , but it threw CordovaError is not defined
> at null.<anonymous>
>
> (/Users/medic/buildbot/slave_common/Tools_CLI/build/cordova-cli/spec/metadata/android_parser.spec.js:71:16)
>
> 13) blackberry10 project parser constructions should throw an
> exception with a path that is not a native blackberry project
> Message:
> [31mExpected function to throw The provided path "/some/path" is
> not a Cordova BlackBerry10 project. , but it threw CordovaError is not
> defined [0m
> Stacktrace:
> Error: Expected function to throw The provided path "/some/path"
> is not a Cordova BlackBerry10 project. , but it threw CordovaError is
> not defined
> at null.<anonymous>
>
> (/Users/medic/buildbot/slave_common/Tools_CLI/build/cordova-cli/spec/metadata/blackberry_parser.spec.js:85:16)
>
> 14) blackberry10 project parser check_requirements should fire a
> callback if the blackberry-deploy shell-out fails
> Message:
> [31mReferenceError: CordovaError is not defined [0m
> Stacktrace:
> ReferenceError: CordovaError is not defined
> at
> /Users/medic/buildbot/slave_common/Tools_CLI/build/cordova-cli/src/metadata/blackberry10_parser.js:50:26
> at Object.<anonymous>
>
> (/Users/medic/buildbot/slave_common/Tools_CLI/build/cordova-cli/spec/metadata/blackberry_parser.spec.js:99:29)
> at Function.module.exports.check_requirements
>
> (/Users/medic/buildbot/slave_common/Tools_CLI/build/cordova-cli/src/metadata/blackberry10_parser.js:48:19)
> at null.<anonymous>
>
> (/Users/medic/buildbot/slave_common/Tools_CLI/build/cordova-cli/spec/metadata/blackberry_parser.spec.js:101:56)
> at null.<anonymous>
>
> (/Users/medic/buildbot/slave_common/Tools_CLI/build/cordova-cli/node_modules/jasmine-node/lib/jasmine-node/async-callback.js:45:37)
>
> 15) ios project parser constructions should throw if provided
> directory does not contain an xcodeproj file
> Message:
> [31mExpected function to throw The provided path is not a
> Cordova iOS project. , but it threw CordovaError is not defined [0m
> Stacktrace:
> Error: Expected function to throw The provided path is not a
> Cordova iOS project. , but it threw CordovaError is not defined
> at null.<anonymous>
>
> (/Users/medic/buildbot/slave_common/Tools_CLI/build/cordova-cli/spec/metadata/ios_parser.spec.js:60:16)
>
> 16) windows8 project parser constructions should throw if provided
> directory does not contain a jsproj file
> Message:
> [31mExpected function to throw The provided path "/some/path" is
> not a Windows 8 project. Error: No .jsproj file. , but it threw
> CordovaError is not defined [0m
> Stacktrace:
> Error: Expected function to throw The provided path "/some/path"
> is not a Windows 8 project. Error: No .jsproj file. , but it threw
> CordovaError is not defined
> at null.<anonymous>
>
> (/Users/medic/buildbot/slave_common/Tools_CLI/build/cordova-cli/spec/metadata/windows8_parser.spec.js:72:16)
>
> 17) windows8 project parser check_requirements should fire a
> callback if there is an error during shelling out
> Message:
> [31mReferenceError: CordovaError is not defined [0m
> Stacktrace:
> ReferenceError: CordovaError is not defined
> at
> /Users/medic/buildbot/slave_common/Tools_CLI/build/cordova-cli/src/metadata/windows8_parser.js:64:26
> at Object.<anonymous>
>
> (/Users/medic/buildbot/slave_common/Tools_CLI/build/cordova-cli/spec/metadata/windows8_parser.spec.js:87:17)
> at Function.module.exports.check_requirements
>
> (/Users/medic/buildbot/slave_common/Tools_CLI/build/cordova-cli/src/metadata/windows8_parser.js:61:19)
> at null.<anonymous>
>
> (/Users/medic/buildbot/slave_common/Tools_CLI/build/cordova-cli/spec/metadata/windows8_parser.spec.js:89:52)
> at null.<anonymous>
>
> (/Users/medic/buildbot/slave_common/Tools_CLI/build/cordova-cli/node_modules/jasmine-node/lib/jasmine-node/async-callback.js:45:37)
>
> 18) wp7 project parser constructions should throw if provided
> directory does not contain a csproj file
> Message:
> [31mExpected function to throw The provided path "/some/path" is
> not a Windows Phone 7 project. Error: No .csproj file. , but it threw
> CordovaError is not defined [0m
> Stacktrace:
> Error: Expected function to throw The provided path "/some/path"
> is not a Windows Phone 7 project. Error: No .csproj file. , but it
> threw CordovaError is not defined
> at null.<anonymous>
>
> (/Users/medic/buildbot/slave_common/Tools_CLI/build/cordova-cli/spec/metadata/wp7_parser.spec.js:78:16)
>
> 19) wp7 project parser check_requirements should fire a callback if
> there is an error during shelling out
> Message:
> [31mReferenceError: CordovaError is not defined [0m
> Stacktrace:
> ReferenceError: CordovaError is not defined
> at
> /Users/medic/buildbot/slave_common/Tools_CLI/build/cordova-cli/src/metadata/wp7_parser.js:60:26
> at Object.<anonymous>
>
> (/Users/medic/buildbot/slave_common/Tools_CLI/build/cordova-cli/spec/metadata/wp7_parser.spec.js:92:29)
> at Function.module.exports.check_requirements
>
> (/Users/medic/buildbot/slave_common/Tools_CLI/build/cordova-cli/src/metadata/wp7_parser.js:57:19)
> at null.<anonymous>
>
> (/Users/medic/buildbot/slave_common/Tools_CLI/build/cordova-cli/spec/metadata/wp7_parser.spec.js:94:47)
> at null.<anonymous>
>
> (/Users/medic/buildbot/slave_common/Tools_CLI/build/cordova-cli/node_modules/jasmine-node/lib/jasmine-node/async-callback.js:45:37)
>
> 20) wp8 project parser constructions should throw if provided
> directory does not contain a csproj file
> Message:
> [31mExpected function to throw The provided path "/some/path" is
> not a Windows Phone 8 project. Error: No .csproj file. , but it threw
> CordovaError is not defined [0m
> Stacktrace:
> Error: Expected function to throw The provided path "/some/path"
> is not a Windows Phone 8 project. Error: No .csproj file. , but it
> threw CordovaError is not defined
> at null.<anonymous>
>
> (/Users/medic/buildbot/slave_common/Tools_CLI/build/cordova-cli/spec/metadata/wp8_parser.spec.js:78:16)
>
> 21) wp8 project parser check_requirements should fire a callback if
> there is an error during shelling out
> Message:
> [31mReferenceError: CordovaError is not defined [0m
> Stacktrace:
> ReferenceError: CordovaError is not defined
> at
> /Users/medic/buildbot/slave_common/Tools_CLI/build/cordova-cli/src/metadata/wp8_parser.js:60:26
> at Object.<anonymous>
>
> (/Users/medic/buildbot/slave_common/Tools_CLI/build/cordova-cli/spec/metadata/wp8_parser.spec.js:92:29)
> at Function.module.exports.check_requirements
>
> (/Users/medic/buildbot/slave_common/Tools_CLI/build/cordova-cli/src/metadata/wp8_parser.js:57:19)
> at null.<anonymous>
>
> (/Users/medic/buildbot/slave_common/Tools_CLI/build/cordova-cli/spec/metadata/wp8_parser.spec.js:94:47)
> at null.<anonymous>
>
> (/Users/medic/buildbot/slave_common/Tools_CLI/build/cordova-cli/node_modules/jasmine-node/lib/jasmine-node/async-callback.js:45:37)
>
> Please take a look when you have a moment.
>
> Thanks!
> -Max
>
Re: Many CLI tests failing
Posted by Josh Soref <js...@blackberry.com>.
Max wrote:
> Josh is having trouble accessing the buildbot page, so here is the relevant information.
Thanks. Looking.
Re: Many CLI tests failing
Posted by Max Woghiren <ma...@chromium.org>.
Josh is having trouble accessing the buildbot page, so here is the relevant
information.
Lorin's relevant commit:
https://git-wip-us.apache.org/repos/asf?p=cordova-cli.git;a=commit;h=79e571953c2d3459b459c02eb50e753308fd453d
Josh's relevant commits:
https://git-wip-us.apache.org/repos/asf?p=cordova-cli.git;a=commit;h=07181e4ee3e358b7af7674a4ae68fe06552b2655(platform
parsers)
https://git-wip-us.apache.org/repos/asf?p=cordova-cli.git;a=commit;h=edecaf9193e8b23c6b7e209c6893491497021082(create.js)
Lorin's commit led to errors 2-5. The remainder are the result of Josh's
commits.
The text of the failed tests:
1) hooker should throw if provided directory is not a cordova project
Message:
[31mExpected function to throw Not a Cordova project, can't use
hooks. , but it threw Not a Cordova project
("/var/folders/p5/b_w_jqzj2jd64nz22dnv7xzc0000gn/T/e2e-test/hooks_test"),
can't use hooks. [0m
Stacktrace:
Error: Expected function to throw Not a Cordova project, can't
use hooks. , but it threw Not a Cordova project
("/var/folders/p5/b_w_jqzj2jd64nz22dnv7xzc0000gn/T/e2e-test/hooks_test"),
can't use hooks.
at null.<anonymous>
(/Users/medic/buildbot/slave_common/Tools_CLI/build/cordova-cli/e2e/hooker.spec.js:53:12)
2) create command callback should return null if a callback parameter is used
Message:
[31mReferenceError: foobar is not defined [0m
Stacktrace:
ReferenceError: foobar is not defined
at null.<anonymous>
(/Users/medic/buildbot/slave_common/Tools_CLI/build/cordova-cli/spec/create.spec.js:79:66)
3) create command callback should return null if a callback parameter is used
Message:
[31mtimeout: timed out after 10000 msec waiting for something to
happen [0m
Stacktrace:
undefined
4) create command callback should call the callback function if
callback parameter is used
Message:
[31mReferenceError: foobar is not defined [0m
Stacktrace:
ReferenceError: foobar is not defined
at null.<anonymous>
(/Users/medic/buildbot/slave_common/Tools_CLI/build/cordova-cli/spec/create.spec.js:79:66)
5) create command callback should call the callback function if
callback parameter is used
Message:
[31mtimeout: timed out after 10000 msec waiting for something to
happen [0m
Stacktrace:
undefined
6) create command success should create top-level directory
structure appropriate for a cordova-cli project
Message:
[31mtimeout: timed out after 10000 msec waiting for spec to complete [0m
Stacktrace:
undefined
7) create command success should create hooks directory
Message:
[31mtimeout: timed out after 10000 msec waiting for spec to complete [0m
Stacktrace:
undefined
8) create command success should by default use
cordova-app-hello-world as www assets
Message:
[31mtimeout: timed out after 10000 msec waiting for spec to complete [0m
Stacktrace:
undefined
9) create command success should try to lazy load custom www
location if specified
Message:
[31mtimeout: timed out after 10000 msec waiting for spec to complete [0m
Stacktrace:
undefined
10) create command success should add a missing www/config.xml
Message:
[31mtimeout: timed out after 10000 msec waiting for spec to complete [0m
Stacktrace:
undefined
11) create command success should not replace an existing www/config.xml
Message:
[31mtimeout: timed out after 10000 msec waiting for spec to complete [0m
Stacktrace:
undefined
12) android project parser constructions should throw if provided
directory does not contain an AndroidManifest.xml
Message:
[31mExpected function to throw The provided path "some/path" is
not an Android project. , but it threw CordovaError is not defined [0m
Stacktrace:
Error: Expected function to throw The provided path "some/path"
is not an Android project. , but it threw CordovaError is not defined
at null.<anonymous>
(/Users/medic/buildbot/slave_common/Tools_CLI/build/cordova-cli/spec/metadata/android_parser.spec.js:71:16)
13) blackberry10 project parser constructions should throw an
exception with a path that is not a native blackberry project
Message:
[31mExpected function to throw The provided path "/some/path" is
not a Cordova BlackBerry10 project. , but it threw CordovaError is not
defined [0m
Stacktrace:
Error: Expected function to throw The provided path "/some/path"
is not a Cordova BlackBerry10 project. , but it threw CordovaError is
not defined
at null.<anonymous>
(/Users/medic/buildbot/slave_common/Tools_CLI/build/cordova-cli/spec/metadata/blackberry_parser.spec.js:85:16)
14) blackberry10 project parser check_requirements should fire a
callback if the blackberry-deploy shell-out fails
Message:
[31mReferenceError: CordovaError is not defined [0m
Stacktrace:
ReferenceError: CordovaError is not defined
at /Users/medic/buildbot/slave_common/Tools_CLI/build/cordova-cli/src/metadata/blackberry10_parser.js:50:26
at Object.<anonymous>
(/Users/medic/buildbot/slave_common/Tools_CLI/build/cordova-cli/spec/metadata/blackberry_parser.spec.js:99:29)
at Function.module.exports.check_requirements
(/Users/medic/buildbot/slave_common/Tools_CLI/build/cordova-cli/src/metadata/blackberry10_parser.js:48:19)
at null.<anonymous>
(/Users/medic/buildbot/slave_common/Tools_CLI/build/cordova-cli/spec/metadata/blackberry_parser.spec.js:101:56)
at null.<anonymous>
(/Users/medic/buildbot/slave_common/Tools_CLI/build/cordova-cli/node_modules/jasmine-node/lib/jasmine-node/async-callback.js:45:37)
15) ios project parser constructions should throw if provided
directory does not contain an xcodeproj file
Message:
[31mExpected function to throw The provided path is not a
Cordova iOS project. , but it threw CordovaError is not defined [0m
Stacktrace:
Error: Expected function to throw The provided path is not a
Cordova iOS project. , but it threw CordovaError is not defined
at null.<anonymous>
(/Users/medic/buildbot/slave_common/Tools_CLI/build/cordova-cli/spec/metadata/ios_parser.spec.js:60:16)
16) windows8 project parser constructions should throw if provided
directory does not contain a jsproj file
Message:
[31mExpected function to throw The provided path "/some/path" is
not a Windows 8 project. Error: No .jsproj file. , but it threw
CordovaError is not defined [0m
Stacktrace:
Error: Expected function to throw The provided path "/some/path"
is not a Windows 8 project. Error: No .jsproj file. , but it threw
CordovaError is not defined
at null.<anonymous>
(/Users/medic/buildbot/slave_common/Tools_CLI/build/cordova-cli/spec/metadata/windows8_parser.spec.js:72:16)
17) windows8 project parser check_requirements should fire a
callback if there is an error during shelling out
Message:
[31mReferenceError: CordovaError is not defined [0m
Stacktrace:
ReferenceError: CordovaError is not defined
at /Users/medic/buildbot/slave_common/Tools_CLI/build/cordova-cli/src/metadata/windows8_parser.js:64:26
at Object.<anonymous>
(/Users/medic/buildbot/slave_common/Tools_CLI/build/cordova-cli/spec/metadata/windows8_parser.spec.js:87:17)
at Function.module.exports.check_requirements
(/Users/medic/buildbot/slave_common/Tools_CLI/build/cordova-cli/src/metadata/windows8_parser.js:61:19)
at null.<anonymous>
(/Users/medic/buildbot/slave_common/Tools_CLI/build/cordova-cli/spec/metadata/windows8_parser.spec.js:89:52)
at null.<anonymous>
(/Users/medic/buildbot/slave_common/Tools_CLI/build/cordova-cli/node_modules/jasmine-node/lib/jasmine-node/async-callback.js:45:37)
18) wp7 project parser constructions should throw if provided
directory does not contain a csproj file
Message:
[31mExpected function to throw The provided path "/some/path" is
not a Windows Phone 7 project. Error: No .csproj file. , but it threw
CordovaError is not defined [0m
Stacktrace:
Error: Expected function to throw The provided path "/some/path"
is not a Windows Phone 7 project. Error: No .csproj file. , but it
threw CordovaError is not defined
at null.<anonymous>
(/Users/medic/buildbot/slave_common/Tools_CLI/build/cordova-cli/spec/metadata/wp7_parser.spec.js:78:16)
19) wp7 project parser check_requirements should fire a callback if
there is an error during shelling out
Message:
[31mReferenceError: CordovaError is not defined [0m
Stacktrace:
ReferenceError: CordovaError is not defined
at /Users/medic/buildbot/slave_common/Tools_CLI/build/cordova-cli/src/metadata/wp7_parser.js:60:26
at Object.<anonymous>
(/Users/medic/buildbot/slave_common/Tools_CLI/build/cordova-cli/spec/metadata/wp7_parser.spec.js:92:29)
at Function.module.exports.check_requirements
(/Users/medic/buildbot/slave_common/Tools_CLI/build/cordova-cli/src/metadata/wp7_parser.js:57:19)
at null.<anonymous>
(/Users/medic/buildbot/slave_common/Tools_CLI/build/cordova-cli/spec/metadata/wp7_parser.spec.js:94:47)
at null.<anonymous>
(/Users/medic/buildbot/slave_common/Tools_CLI/build/cordova-cli/node_modules/jasmine-node/lib/jasmine-node/async-callback.js:45:37)
20) wp8 project parser constructions should throw if provided
directory does not contain a csproj file
Message:
[31mExpected function to throw The provided path "/some/path" is
not a Windows Phone 8 project. Error: No .csproj file. , but it threw
CordovaError is not defined [0m
Stacktrace:
Error: Expected function to throw The provided path "/some/path"
is not a Windows Phone 8 project. Error: No .csproj file. , but it
threw CordovaError is not defined
at null.<anonymous>
(/Users/medic/buildbot/slave_common/Tools_CLI/build/cordova-cli/spec/metadata/wp8_parser.spec.js:78:16)
21) wp8 project parser check_requirements should fire a callback if
there is an error during shelling out
Message:
[31mReferenceError: CordovaError is not defined [0m
Stacktrace:
ReferenceError: CordovaError is not defined
at /Users/medic/buildbot/slave_common/Tools_CLI/build/cordova-cli/src/metadata/wp8_parser.js:60:26
at Object.<anonymous>
(/Users/medic/buildbot/slave_common/Tools_CLI/build/cordova-cli/spec/metadata/wp8_parser.spec.js:92:29)
at Function.module.exports.check_requirements
(/Users/medic/buildbot/slave_common/Tools_CLI/build/cordova-cli/src/metadata/wp8_parser.js:57:19)
at null.<anonymous>
(/Users/medic/buildbot/slave_common/Tools_CLI/build/cordova-cli/spec/metadata/wp8_parser.spec.js:94:47)
at null.<anonymous>
(/Users/medic/buildbot/slave_common/Tools_CLI/build/cordova-cli/node_modules/jasmine-node/lib/jasmine-node/async-callback.js:45:37)
Please take a look when you have a moment.
Thanks!
-Max