LAVA Android Test

Merge lp://qastaging/~kejun-zhou/lava-android-test/ime into lp://qastaging/lava-android-test

ime
Merge into trunk

Proposed by Yongqin Liu on 2012-04-25

Status:	Superseded
Proposed branch:	lp://qastaging/~kejun-zhou/lava-android-test/ime
Merge into:	lp://qastaging/lava-android-test
Diff against target:	79 lines (+61/-0) 3 files modified MANIFEST.in (+1/-0) lava_android_test/test_definitions/ime.py (+47/-0) lava_android_test/test_definitions/ime/ime_test.sh (+13/-0)
To merge this branch:	bzr merge lp://qastaging/~kejun-zhou/lava-android-test/ime
Related bugs:	Link a bug report
Related blueprints:	Build android.inputmethodservice Unit Tests and Integrate them in LAVA (High)

Reviewer	Date Requested	Status
Le Chi Thu (community)		Needs Fixing on 2012-05-11
Yongqin Liu		Needs Fixing on 2012-04-25
Kejun ZHOU	2012-04-25	Pending
Review via email: mp+103465@code.qastaging.launchpad.net

This proposal has been superseded by a proposal from 2012-05-23.

Description of the change

add test script wrapper for input method test

Revision history for this message

Yongqin Liu (liuyq0307) wrote on 2012-04-25:

68 + echo "ime = pass"

this line seems not match the pattern defined in
43 +PATTERN = "^\s*(?P<test_case_id>\w+)=(?P<result>\w+)\s*$"

I think you can change to [echo "ime=pass"]

Revision history for this message

Yongqin Liu (liuyq0307) wrote on 2012-04-25:

also, you should modify MANIFEST.in file

review: Needs Fixing

lp://qastaging/~kejun-zhou/lava-android-test/ime updated on 2012-05-03

149. By Ke Jun ZHOU <dp583@stebjsxu0119> on 2012-04-26: [update]for inputmethod service
150. By Ke Jun ZHOU <dp583@stebjsxu0119> on 2012-05-03: update

Revision history for this message

Le Chi Thu (le-chi-thu) wrote on 2012-05-09:

Please describe the purpose of this test and what the test perform. Which input is required and expect conditions /outputs to detect when the test is pass or not.

echo "ime=unexist" should be echo "ime=fail"

review: Needs Fixing

lp://qastaging/~kejun-zhou/lava-android-test/ime updated on 2012-05-09

151. By Ke Jun ZHOU <dp583@stebjsxu0119> on 2012-05-09: Change the lava_android_test/test_definitions/ime/ime_test.sh
-echo "ime=unexist"
+echo "ime=fail"

Revision history for this message

Le Chi Thu (le-chi-thu) wrote on 2012-05-11:

Please describe the purpose of this test and what the test perform. Which input is required and expect conditions /outputs to detect when the test is pass or not.

review: Needs Fixing

lp://qastaging/~kejun-zhou/lava-android-test/ime updated on 2012-05-25