Merge lp://staging/~qzhang/lava-test/fix-900694 into lp://staging/lava-test/0.0

Proposed by Spring Zhang
Status: Merged
Merged at revision: 105
Proposed branch: lp://staging/~qzhang/lava-test/fix-900694
Merge into: lp://staging/lava-test/0.0
Diff against target: 30 lines (+5/-8)
1 file modified
lava_test/test_definitions/ltp.py (+5/-8)
To merge this branch: bzr merge lp://staging/~qzhang/lava-test/fix-900694
Reviewer Review Type Date Requested Status
Le Chi Thu (community) Approve
Spring Zhang (community) Needs Resubmitting
Review via email: mp+84709@code.staging.launchpad.net

Description of the change

fix "no attribute 'appendtoall'" in ltp suite

To post a comment you must log in.
106. By Spring Zhang

fix the error 'TypeError: parse() takes exactly 1 argument (2 given)'

Revision history for this message
Spring Zhang (qzhang) wrote :

tested and pass

review: Needs Resubmitting
Revision history for this message
Le Chi Thu (le-chi-thu) wrote :

should use artifacts.stdout_pathname instead of hard coded to the "testoutput.log"

review: Needs Fixing
107. By Spring Zhang

remove hardcode due to ChiThu's comments

Revision history for this message
Spring Zhang (qzhang) wrote :

Thanks for suggestion, corrected.

review: Needs Resubmitting
Revision history for this message
Le Chi Thu (le-chi-thu) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches