1105168 - Update insert_call_entry method to remove time.sleep()

Status: RESOLVED WONTFIX

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

Description

[Robert Chira [:RobertC]]

We should remove the time.sleep() from [1] and do something like the suggestion in [2]. I tried doing that when I first made the PR, but the async_script would timeout. I also moved the method in the atoms and tried listening for an onsuccess() event, but that did not work either.

This requires further investigation.


[1] https://github.com/mozilla-b2g/gaia/blob/master/tests/python/gaia-ui-tests/gaiatest/gaia_test.py#L378
[2] https://bugzilla.mozilla.org/show_bug.cgi?id=1096920#c7

Comment 1

[Martijn Wargers (dead)]

The suggestion in https://bugzilla.mozilla.org/show_bug.cgi?id=1096920#c7 is:
"
Robert, could you file a follow-up bug, about the time.sleep(1) workaround?
It should have been possible to write something like this:
self.marionette.execute_async_script('window.wrappedJSObject.CallLogDBManager.add(%s, function() {marionetteScriptFinished()});' % (json.dumps(call)))
..inside def insert_call_entry, so you would not have to use the time.sleep(1) function below.

We should figure out why it doesn't work, ideally.
"

Comment 2

[BMO Automation]

Firefox OS is not being worked on