Mailing-List: contact callback-dev-help@incubator.apache.org; run by ezmlm
Precedence: bulk
Reply-To: callback-dev@incubator.apache.org
Received-SPF: pass (nike.apache.org: domain of bowserj@gmail.com designates
 74.125.82.43 as permitted sender)
MIME-Version: 1.0
In-Reply-To: 
 <CABiQX1WYr9aTqx4O4mPAvyNVqT8tw5eMgq4VtNxbEFoYqN0AXg@mail.gmail.com>
References: 
 <CABiQX1X6p8qsFaj+JBG1BRGSYNVuuVF6wqV=g5SJav-=LBfmSQ@mail.gmail.com>
	<CABiQX1VNDjOGFaF4sJ8ykgt9uW1yhBB+HRyWeNc=1kHTf+kpMA@mail.gmail.com>
	<CABiQX1WYr9aTqx4O4mPAvyNVqT8tw5eMgq4VtNxbEFoYqN0AXg@mail.gmail.com>
Date: Thu, 20 Sep 2012 20:32:56 -0700
Message-ID: 
 <CAOBL_k5ta-_mgk2Dh=rh5vhragi_5knAQsYUKmS492FF32HZXg@mail.gmail.com>
Subject: Re: jake showing a failing test
From: Joe Bowser <bowserj@gmail.com>
To: Andrew Grieve <agrieve@google.com>
Cc: callback-dev@incubator.apache.org
Content-Type: text/plain; charset=ISO-8859-1

Hey

I try and avoid changing anything in incubator-cordova-js, and I limit
my changes to lib/android.  As far as tests, they should be automatic
if you have your node setup working properly.  I know that Fil did
some changes on the cordova-js part of the geolocation code due to a
bug in the Cordova-Android version of that plugin (not the built-in
one), so maybe the test needs some more changes.

Also, we agreed earlier that those lines of JS need to be deleted
entirely for speed.

Joe

On Thu, Sep 20, 2012 at 8:13 PM, Andrew Grieve <agrieve@google.com> wrote:
> Okay, fixed the test by converting some wait()s to waitFor()s.
>
>
> On Thu, Sep 20, 2012 at 10:56 PM, Andrew Grieve <agrieve@google.com> wrote:
>>
>> Well, sorry for the false accusation :(. Looks like the test is just
>> flakey:
>>
>> geolocation getCurrentPosition position acquisition
>>   it should fire error callback with TIMEOUT code after timeout period has
>> elapsed and no position is available
>>   Error: Expected spy error to have been called with [ { code : 3, message
>> : 'Position retrieval timed out.' } ] but it was never called.
>>     at new jasmine.ExpectationResult (/Users/agrieve/Google
>> Drive/macfiles/git/incubator-cordova-js/thirdparty/jasmine/jasmine.js:102:32)
>>     at jasmine.Matchers.matcherFn_ [as toHaveBeenCalledWith]
>> (/Users/agrieve/Google
>> Drive/macfiles/git/incubator-cordova-js/thirdparty/jasmine/jasmine.js:1171:29)
>>     at geo.getCurrentPosition.timeout (eval at <anonymous>
>> (/Users/agrieve/Google
>> Drive/macfiles/git/incubator-cordova-js/test/runner.js:84:21))
>>     at jasmine.Block.execute (/Users/agrieve/Google
>> Drive/macfiles/git/incubator-cordova-js/thirdparty/jasmine/jasmine.js:1001:15)
>>     at jasmine.Queue.next_ (/Users/agrieve/Google
>> Drive/macfiles/git/incubator-cordova-js/thirdparty/jasmine/jasmine.js:1790:31)
>>     at jasmine.Queue.next_.onComplete (/Users/agrieve/Google
>> Drive/macfiles/git/incubator-cordova-js/thirdparty/jasmine/jasmine.js:1786:18)
>>     at Object.jasmine.WaitsBlock.execute [as _onTimeout]
>> (/Users/agrieve/Google
>> Drive/macfiles/git/incubator-cordova-js/thirdparty/jasmine/jasmine.js:2231:5)
>>     at Timer.list.ontimeout (timers.js:101:19)
>>
>> it fails for me most times, but does pass every once-in-a-while. I'll have
>> a look and see if I can figure out what's up with it.
>>
>>
>> On Thu, Sep 20, 2012 at 10:46 PM, Andrew Grieve <agrieve@google.com>
>> wrote:
>>>
>>> Hey Joe,
>>>
>>> Looks like your "Fixing CB-1498" commit to the js repo causes jake to
>>> fail a unit test. I'll fix it up, but might be a sign that maybe you didn't
>>> run the tests before checking in? It would also be helpful  if you could
>>> rebase before committing and write a description and [platform] in the
>>> commit message instead of just the bug id.
>>>
>>
>