Re: [battery] getBattery() test case feedback

Zhiqiang

Thanks for the update. 

One clarification:

> This pull request is no acceptable per Ms2ger and my review.

I assume this should read that the "pull request is now acceptable", correct?

Dom, have you had a chance to review the idlharness pull request?

> https://github.com/w3c/testharness.js/pull/161


Thanks

regards, Frederick

Frederick Hirsch
Chair, W3C Device APIs WG (DAP)

www.fjhirsch.com
@fjhirsch

> On Dec 4, 2015, at 3:00 AM, Zhang, Zhiqiang <zhiqiang.zhang@intel.com> wrote:
> 
> Hi,
> 
>> From: Kostiainen, Anssi
>> Sent: Wednesday, November 18, 2015 7:43 PM
>> 
>>> On 18 Nov 2015, at 13:13, Dominique Hazael-Massieux <dom@w3.org>
>> wrote:
>>> 
>>> On 28/08/2015 08:55, Zhang, Zhiqiang wrote:
>>>>> I'm wondering whether the testharness.js built-in promise_*() are of
>>>>> any help or whether we'd need to patch testharness.js to add
>>>>> support. I'd guess testharness.js should be able to properly test
>> functions that return promises.
>>>>> jgraham who's the original author of testharness.js was on the Moz
>>>>> bug, perhaps he knows.
>>>> 
>>>> Hmm, seems this is a problem of idlharness.js rather than
>>>> testharness.js, as the idlharness.js hasn't considered the Promise related
>> tech.
>>> 
>>> I have a pending pull request on idlharness that I think would help  (at least
>> a bit) with this:
>>> https://github.com/w3c/testharness.js/pull/161
> 
> This pull request is no acceptable per Ms2ger and my review.
> 
>> 
>> Dom - excellent, thank you!
>> 
>> My expectation is Zhiqiang will revise the Battery Status API test suite and
>> provide feedback, when the testharness.js PR has landed.
> 
> I just have a quick testing on Firefox Nightly and Chrome, and make a test report at
> 
> https://w3c.github.io/test-results/battery-status/all.html
> 
> ... for your info.
> 
> I also make a pull request at
> 
> https://github.com/w3c/web-platform-tests/pull/2412
> 
> ... to deprecate battery-created-manual.html, which is only useful on devices that expose the BatteryManager
> interface, but lack a backend implementation. This test is no longer needed, because at least we have 2
> implementations now, Chrome and Firefox Nightly.
> 
> Please take a look at it.
> 
> Thanks,
> Zhiqiang
> 

Received on Tuesday, 5 January 2016 13:29:17 UTC