522476 - add tool support to run mops and abcasm in acceptance tests on winmo

Status: VERIFIED FIXED

(Tamarin Graveyard :: Tools, defect)

Description

[Dan Schaffer]

add support for winmo emulator and winceremoteshell to copy any dependent .abc files onto the device

currently only mops and abcasm contain multiple .abc files.  The winmo test tools only copy the runnable abc file (last argument) to the device/emulator.

Comment 1

[Dan Schaffer]

Attachment: patch for running mops and abcasm on winmo emulator

patch runs mops on winmo emulator in the deep test phase.  the test takes ~5 minutes to run in emulator.  the 1st patch supports winmo emulator, will work on copying support files when running on device next.  possibly bug in the shell where running: avmshell.exe -log file1.abc file2.abc writes file1.log instead of file2.log.  wmemulatorshell.py contains some code to look for .log file with the other abc file.

Comment 2

[Dan Schaffer]

Attachment: slightly improved patch

minor improvements from previous patch.  sets the expected -log output file to be the 1st abc passed to the shell instead of searching for all abcs.

Comment 3

[Brent Baker]

Comment on attachment 407532 [details] [diff] [review]
slightly improved patch

should the mops/mops test be added into the winmo deep testing in testconfig.txt

Comment 4

[Dan Schaffer]

Attachment: patch for ceremoteshell.exe and avmremote.dll multi abc support

support for ceremoteshell.exe and avmremote.dll to copy and run tests with multiple abc files.  I was able to run acceptance and performance tests ok.  For acceptance tests I saw failures in recursion tests, Math (4), and RegExp/eregress_11909.  I will file a new bug to track these failures.

Comment 5

[Dan Schaffer]

I believe the mops/mops will run in winmo deep testing.  It is run in deep on other platforms and removing the winmo specific skip should add it to winmo-emulator-deep.

Comment 6

[Brent Baker]

Comment on attachment 407592 [details] [diff] [review]
patch for ceremoteshell.exe and avmremote.dll multi abc support

patch looks good to my, I am able to run the currently excluded mops and abcasm testcases on a device

adding super-review by RobW

Comment 7

[Dan Schaffer]

pushed fixes.  winmo device support in 2849:4d03cf20e2c5, winmo emulator support in 2961:7dbe2d11bb02