[tara]orng and run-monkey.sh can't support tara device

UNCONFIRMED
Unassigned

Status

Firefox OS
General
--
major
UNCONFIRMED
5 years ago
4 years ago

People

(Reporter: James Zhang (Spreadtrum), Unassigned)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(4 attachments)

(Reporter)

Description

5 years ago
Created attachment 736796 [details] [diff] [review]
orng patch to support tara

orng and run-monkey.sh can't support tara device, only support unagi.
we modiy the orng.c and script to support both tara and unagi device.
we suggest to use config file to config these different devices.
(Reporter)

Comment 1

5 years ago
Created attachment 736801 [details]
all of the monkey-test script and binary, busybox, gnasp, minidump are opensource
(Reporter)

Comment 2

5 years ago
copy the moneky-test to B2G folder
cd monkey-test && ./run-tara-test.sh    # run tara monkey test
cd monkey-test && ./run-unagi-test.sh  # run unagi monkey test

you can add new run-xxx-test.sh to run new device monke test.
(Reporter)

Comment 3

5 years ago
Created attachment 736802 [details] [diff] [review]
enable crash report and fix tara/unagi receive monkey event bug
(Reporter)

Comment 4

5 years ago
build_symb.sh # build symbol for crash report and minidump
crash_report.sh # adb pull the crash report and screen snapshot
set_busybox.sh # adb push busybox to device
set_gsnap.sh # adb push gsnap to device
unlock.sh # turn on the backlight and unlock
run_test.sh # call other script
(Reporter)

Comment 5

5 years ago
Created attachment 736812 [details]
for example, unagi crash report
(Reporter)

Updated

5 years ago
Severity: normal → major
(Reporter)

Updated

5 years ago
Blocks: 861492
Comment on attachment 736796 [details] [diff] [review]
orng patch to support tara

Review of attachment 736796 [details] [diff] [review]:
-----------------------------------------------------------------

Some unsolicited feedback: I'd prefer if we generalized what this device needed that was a different from what we have there right now (I guess the ABS_Z events) and made that behaviour conditional on device-agnostic properties (like we did, for example, with INPUT_DEVICE_CLASS_TOUCH_MT_SYNC). If we try to have completely different behaviour for different devices the codebase will quickly get unwieldy.

Anyway, great to see people working on making orangutan support different devices.
(Reporter)

Comment 7

5 years ago
(In reply to William Lachance (:wlach) from comment #6)
> Comment on attachment 736796 [details] [diff] [review]
orng patch to support
> tara

Review of attachment 736796 [details] [diff] [review]:
> -----------------------------------------------------------------

Some
> unsolicited feedback: I'd prefer if we generalized what this device needed
> that was a different from what we have there right now (I guess the ABS_Z
> events) and made that behaviour conditional on device-agnostic properties
> (like we did, for example, with INPUT_DEVICE_CLASS_TOUCH_MT_SYNC). If we try
> to have completely different behaviour for different devices the codebase
> will quickly get unwieldy.

Anyway, great to see people working on making
> orangutan support different devices.

Hi William, 

   orng and run-monkey.sh need config file to support different devices, so we can add new devices easily.

   We implement new auto monkey script, support auto code upgrade, build, flash, run monkey and submit the crash report to the server. We are testing the new scrpit now, and I'll submit it soon.
You need to log in before you can comment on or make changes to this bug.