Closed Bug 651335 Opened 14 years ago Closed 8 years ago

devicemanager.py has some hardcoded android data in it

Categories

(Testing :: General, defect)

Product:
Testing
Component:
General
Platform:
x86
Linux
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED INVALID

People

(Reporter: jmaher, Assigned: gbrown)

References

Details

devicemanager.py is designed to be a general tool to communicate with a SUTAgent regardless of what OS the SUTAgent is running on. Obviously there are a few exceptions to this, but those should be handled in a more extensible fashion. In a specific example, the reboot() command references /data/data/com.mozilla.SUTAgentAndroid/files directly. This is bad practice and should at least act more like getAppRoot(). I would prefer a more abstracted approach instead of hacking if/else conditions for each platform.
Depends on: 649681
Blocks: 771963
Assignee: nobody → gbrown
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → INVALID
Component: New Frameworks → General
You need to log in before you can comment on or make changes to this bug.