LAVA Android Test

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

  1. ime
  2. Merge into trunk
Proposed by Kejun ZHOU
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
Reviewer Review Type Date Requested Status
Kejun ZHOU (community) Needs Resubmitting
Le Chi Thu Pending
Yongqin Liu Pending
Review via email: mp+106954@code.qastaging.launchpad.net

This proposal supersedes a proposal from 2012-04-25.

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

Commit message

This test is for checking which input methods are running in the Android system. The test runs the tool /system/bin/ime to show all the input method services.

Description of the change

This test is for checking which input methods are running in the Android system. The test runs the tool /system/bin/ime to show all the input method services.

To post a comment you must log in.
Revision history for this message
Yongqin Liu (liuyq0307) wrote : Posted in a previous version of this proposal

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 : Posted in a previous version of this proposal

also, you should modify MANIFEST.in file

review: Needs Fixing
Revision history for this message
Le Chi Thu (le-chi-thu) wrote : Posted in a previous version of this proposal

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
Revision history for this message
Le Chi Thu (le-chi-thu) wrote : Posted in a previous version of this proposal

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
Revision history for this message
Kejun ZHOU (kejun-zhou) wrote :

The MANIFEST.in had been modify at 2012-04-26 03:16:07 UTC

include lava_android_test/test_definitions/gatortest/*
+include lava_android_test/test_definitions/ime/*

review: Needs Resubmitting
lp://qastaging/~kejun-zhou/lava-android-test/ime updated
152. By Ke Jun ZHOU <dp583@stebjsxu0119>

Change imp.py to:
PATTERN = "^\s*(?P<test_case_id>ime)=(?P<result>\w+)\s*$"

Unmerged revisions

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