1086571 - Investigate and fix test_call_log_all_calls intermittent failures

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Gaia::UI Tests, defect)

Description

[Robert Chira [:RobertC]]

test_call_log_all_calls is failing intermittently on Jenkins and on local device for different reasons.
The test fails intermittently on both v2.2 and v2.1

http://jenkins1.qa.scl3.mozilla.com/view/UI/job/flame-kk.ui.adhoc/220/HTML_Report/

Comment 1

[Robert Chira [:RobertC]]

Attachment: Link to Github pull-request: https://github.com/mozilla-b2g/gaia/pull/25570

This patch is for the following error:
Traceback (most recent call last):
File "/var/jenkins/1/workspace/flame-kk-319.mozilla-central.ui.functional.smoke/.env/local/lib/python2.7/site-packages/marionette_client-0.8.4-py2.7.egg/marionette/marionette_test.py", line 264, in run
testMethod()
File "/var/jenkins/1/workspace/flame-kk-319.mozilla-central.ui.functional.smoke/tests/python/gaia-ui-tests/gaiatest/tests/functional/dialer/test_call_log_all_calls.py", line 64, in test_call_log_all_calls
self.assertEqual(len(call_list), 2)
AssertionError: 1 != 2

Comment 2

[Robert Chira [:RobertC]]

test_call_log_all_calls started failing recently with the errors present in the following adhoc run:
http://jenkins1.qa.scl3.mozilla.com/view/UI/job/flame-kk.ui.adhoc/235/HTML_Report/

The first digit of the plivo phone number is missing in the call log. Where there any changes done recently in the dialer app which might cause this?

We could not reproduce the issue locally.

Comment 3

[Gabriele Svelto [:gsvelto] (PTO)]

I'm not aware of changes to how we handle the call log that might do this. needinfo'ing :fcampo because he might know more.

Comment 4

[Johan Lorenzo [:jlorenzo]]

I found the change we've seen since October 20th, it's the test itself. It has been modified to check incoming call and it was landed at that date (see bug 1075547 comment 3).

Here's the reason why we don't reproduce locally. In the US, carriers don't add a + for domestic calls, not like in Romania, Canada or France. Then, some providers, like AT&T, don't add the country code (1, in our test case) but some do, like T-Mobile. That's also the reason why this test was intermittent.

Hence, we shouldn't check for the country code in our test for now. Also, to explain that, let's put a comment in the code, about country codes and US carriers.

I opened bug 1088756 to ask UX what we should do with country codes in the US.

Comment 5

[Johan Lorenzo [:jlorenzo]]

Comment on attachment 8511047 [details] [review]
Link to Github pull-request: https://github.com/mozilla-b2g/gaia/pull/25570

LGTM. Let's just check for the phone number minus the country code, and the patch will be okay to me.

Comment 6

[Robert Chira [:RobertC]]

Comment on attachment 8511047 [details] [review]
Link to Github pull-request: https://github.com/mozilla-b2g/gaia/pull/25570

Removed country code and started an adhoc run:
http://jenkins1.qa.scl3.mozilla.com/view/UI/job/flame-kk.ui.adhoc/240/console

Comment 7

[Viorela Ioia [:viorela]]

Comment on attachment 8511047 [details] [review]
Link to Github pull-request: https://github.com/mozilla-b2g/gaia/pull/25570

Looks good, r+

Comment 8

[Johan Lorenzo [:jlorenzo]]

Comment on attachment 8511047 [details] [review]
Link to Github pull-request: https://github.com/mozilla-b2g/gaia/pull/25570

Tested locally. As a country code can be 3 or 4 digits long, this line (https://github.com/mozilla-b2g/gaia/pull/25497/files#diff-6cf85291d551807eb78eb6e6bfbc6a59R35) might be problematic if another SIM card from another country does not display the country code. But I hope bug 1088756 will fix this issue in the meantime.

R+

Comment 9

[Viorela Ioia [:viorela]]

Merged in: https://github.com/mozilla-b2g/gaia/commit/dc2a02165b3411872d64cbc3cb58184bad6f382e
We need to uplift this patch to v2.1

Comment 10

[Robert Chira [:RobertC]]

Attachment: uplift to v2.1

Comment 11

[Viorela Ioia [:viorela]]

Comment on attachment 8512540 [details] [review]
uplift to v2.1

Uplifted to v2.1: https://github.com/mozilla-b2g/gaia/commit/d26ff7da07e910e55855db3d6e1583592e81e797