Closed
Bug 938256
Opened 9 years ago
Closed 9 years ago
mozPhoneNumberService fuzzyMatch is always returning true
Categories
(Core :: DOM: Device Interfaces, defect)
Tracking
()
RESOLVED
FIXED
1.3 C2/1.4 S2(17jan)
People
(Reporter: jmcf, Assigned: mikehenrty)
References
Details
(Whiteboard: [systemsfe])
Attachments
(1 file)
Test Case: var req = navigator.mozPhoneNumberService.fuzzyMatch('983456789', '638883076'); req.onsuccess = function() { if(req.result === true) { window.console.log('Failed'); } else { window.console.log('Pass!'); } req.onerror = function() { alert(req.error.name); } Actually I have not found any test case that returns false, thus it seems there is some integration problem that makes it to return always true.
Assignee | ||
Comment 1•9 years ago
|
||
WIP
Updated•9 years ago
|
Whiteboard: [systemsfe]
Assignee | ||
Comment 2•9 years ago
|
||
Its green 20 out of 20: https://travis-ci.org/mozilla-b2g/gaia/builds/16899869 I think we should add these integration tests.
Assignee | ||
Comment 3•9 years ago
|
||
Comment on attachment 8334374 [details] [review] Add fuzzyMatch integration tests Julien, since you know a thing or two about the phoneNumberService, would you mind having a look at these integration tests?
Attachment #8334374 -
Flags: review?(felash)
Assignee | ||
Comment 4•9 years ago
|
||
Jose, are you still able to reproduce this? Based on the integration tests, fuzzyMatch seems to be working.
Flags: needinfo?(jmcf)
Updated•9 years ago
|
Target Milestone: --- → 1.3 C2/1.4 S2(17jan)
Comment 5•9 years ago
|
||
Comment on attachment 8334374 [details] [review] Add fuzzyMatch integration tests r=me but please fix the comments first :)
Attachment #8334374 -
Flags: review?(felash) → review+
Comment 6•9 years ago
|
||
(In reply to Julien Wajsberg [:julienw] from comment #5) > Comment on attachment 8334374 [details] [review] > Add fuzzyMatch integration tests > > r=me but please fix the comments first :) These files fail `make hint APP=sms` apps/sms/test/marionette/phone_number_service_test.js: line 3, col 5, Redefinition of 'assert'. apps/sms/test/marionette/phone_number_service_test.js: line 8, col 1, 'marionette' is not defined. apps/sms/test/marionette/phone_number_service_test.js: line 9, col 16, 'marionette' is not defined. apps/sms/test/marionette/phone_number_service_test.js: line 40, col 11, 'marionetteScriptFinished' is not defined. apps/sms/test/marionette/phone_number_service_test.js: line 6, col 24, 'TARGET_APP_MANIFEST' is defined but never used.
Assignee | ||
Comment 7•9 years ago
|
||
Rick, My fault! Thanks for your fix! By the way, how does your patch fix redefinition of 'assert'? .../phone_number_service_test.js: line 3, col 5, Redefinition of 'assert'.
Depends on: 959840
Comment 8•9 years ago
|
||
(In reply to Michael Henretty [:mhenretty] from comment #7) > Rick, > > My fault! Thanks for your fix! > > By the way, how does your patch fix redefinition of 'assert'? > .../phone_number_service_test.js: line 3, col 5, Redefinition of 'assert'. You can keep this one, I'm removing "assert" from the list of globals in bug 937431.
Comment 9•9 years ago
|
||
(In reply to Michael Henretty [:mhenretty] from comment #7) > Rick, > > My fault! Thanks for your fix! > > By the way, how does your patch fix redefinition of 'assert'? > .../phone_number_service_test.js: line 3, col 5, Redefinition of 'assert'. I removed it from the jshintrc that put in the marionette dir, but looks like julien has a different plan ;)
Reporter | ||
Comment 10•9 years ago
|
||
(In reply to Michael Henretty [:mhenretty] from comment #4) > Jose, are you still able to reproduce this? Based on the integration tests, > fuzzyMatch seems to be working. Yes, I did a simple test and seems to be working thanks!
Flags: needinfo?(jmcf)
Comment 11•9 years ago
|
||
oki, then I'm marking this fixed because we landed some integration tests.
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•