Closed Bug 979631 Opened 11 years ago Closed 11 years ago

testaddons on Fennec fails when more than one addon being tested

Categories

(Add-on SDK Graveyard :: General, defect)

x86
Android
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: jsantell, Assigned: jsantell)

References

Details

Attachments

(1 file)

Currently, if the previous Fennec process is running, we send an adb command to kill it and wait 2 seconds to see if the process still exists -- and throw an error if it does. I'm not sure why it is now, but the process still exists even though Fennec is closed so an error is thrown regardless. This removes the error, ups the wait time (there has to be a better way?) so we can test the fennec testaddons.
Attached file GH PR 1419 (deleted) —
Attachment #8385729 - Flags: review?(evold)
Attachment #8385729 - Flags: review?(evold) → review+
Commits pushed to master at https://github.com/mozilla/addon-sdk https://github.com/mozilla/addon-sdk/commit/9b6c6a276655829fa357281f1d5f4755230fd9c5 Bug 979631 Don't throw an error when waiting for the PID of Fennec to not exist when running testaddons https://github.com/mozilla/addon-sdk/commit/ce2fe61a94759b9d8cc11e5d00417826073f0d54 Merge pull request #1419 from jsantell/979631-fix-fennec-testaddons Fix Bug 979631 Don't throw an error when waiting for the PID of Fennec to not exist when running testaddons, r+=@erikvold
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: