sleep function using JavaScript units

Bug #1250113 reported by Víctor R. Ruiz
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
The Webapps-core project
Triaged
Wishlist
Unassigned

Bug Description

In tests/autopilot/webapps_click_packages_autopilot/tests/__init__.py this method is defined:

    def update_autopilot_proxy_for_pid(self, pid):
        self.app_proxy = None
        for x in range(10):
            self.app_proxy = get_proxy_object_for_existing_process(pid=int(pid))
            if not self.app_proxy is None:
                break
            sleep(1000)
            self.assertThat(x, NotEquals(9))
        self.assertThat(self.app_proxy, NotEquals(None))

Line 54 sleep(1000) is using JavaScript units (miliseconds), instead of Python units (seconds).

Maybe this function could be rewritten using Eventually.

Víctor R. Ruiz (vrruiz)
description: updated
affects: webapps-demo → webapps-core
David Barth (dbarth)
Changed in webapps-core:
importance: Undecided → Wishlist
status: New → Triaged
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.