Closed Bug 491724 Opened 16 years ago Closed 16 years ago

add pyspy cmdline tests

Categories

(Tamarin Graveyard :: Virtual Machine, defect)

Product:
Tamarin Graveyard
Component:
Virtual Machine
Platform:
x86
macOS
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
VERIFIED FIXED

People

(Reporter: dschaffe, Assigned: dschaffe)

Details

Attachments

(1 file, 4 obsolete files)

patch
23.18 KB, patch
brbaker
: review+
Details | Diff | Splinter Review
add memstats and pyspy tests to cmdline testsuite - memstats runs $AVM -memstats and parses basic output to show gross stats are output - pyspy test only runs on windows, sets MEMORY_PROFILER env var, runs test, executes pyspy script separately, verifies vm output contains memory stats probably good idea to add new tests to smokes to make sure test is stable
Flags: flashplayer-triage+
Attached file patch (obsolete) —
Assignee: nobody → dschaffe
Status: NEW → ASSIGNED
Attachment #376062 - Flags: review?(brbaker)
Attached patch patch 2 (obsolete) — — Splinter Review
Attachment #376062 - Attachment is obsolete: true
Attachment #376062 - Flags: review?(brbaker)
Attached patch patch #3 (obsolete) — — Splinter Review
Attachment #376072 - Attachment is obsolete: true
Attached patch addition to patch #3 (obsolete) — — Splinter Review
Attachment #376307 - Flags: review?(brbaker)
Attachment #376076 - Flags: review?(brbaker)
Attached patch patch — — Splinter Review
Attachment #376076 - Attachment is obsolete: true
Attachment #376307 - Attachment is obsolete: true
Attachment #376454 - Flags: review?(brbaker)
Attachment #376076 - Flags: review?(brbaker)
Attachment #376307 - Flags: review?(brbaker)
sandbox testing the patch. also had to specify --builtinabc ${GLOBALABC} for windows, windows64 using native python
Comment on attachment 376454 [details] [diff] [review] patch Make sure to include a license header in the memstats.as testfile
Attachment #376454 - Flags: review?(brbaker) → review+
submitted in tamarin-redux 1865 83405cbf9278
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
closing bug by marking verified/fixed.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: