Merge lp:~fagerstedt-axel/lava-android-test/ashmemtest into lp:lava-android-test

[Axel Fagerstedt]

Add new test ashmemtest

[Andy Doan (doanac)]

nice job for a first submission :)

A couple of small doc-related things:

1) in tests.rst I'd just call it "ashmemtest" and "command-ashmemtest".

2) in ashmemetest.py could you add a docstring near the top like line 21 of:
 http://bazaar.launchpad.net/~linaro-validation/lava-android-test/trunk/view/head:/lava_android_test/test_definitions/busybox.py

By getting that done correctly, our tests will have nice documenation available at:
  http://lava-android-test.readthedocs.org/en/latest/tests.html

review: Needs Fixing

[lp:~fagerstedt-axel/lava-android-test/ashmemtest]

228. By Axel Fagerstedt on 2013-01-18

Update ashmemtest documentation.

[Axel Fagerstedt (fagerstedt-axel)]

Hi Andy,

Thanks for the comments, updated.

I was not able to figure out how to update my existing commit in bzr so I pushed a second commit into my existing branch. I hope this is the correct way to do it when working with bzr and lp?

[Andy Doan (doanac)]

that's the correct way to update it. I'll merge the code now.

review: Approve

=== modified file 'doc/tests.rst'
--- doc/tests.rst	2012-09-26 21:35:40 +0000
+++ doc/tests.rst	2013-01-18 14:15:23 +0000
@@ -22,6 +22,7 @@
  * `v8`_
  * `wifi`_
  * `0xbench`_
+ * `ashmemtest`_
  * `commands`_
  * `instruments`_
  * `hostshells`_
@@ -87,6 +88,10 @@
 ++++
 .. automodule:: lava_android_test.test_definitions.wifi
 
+ashmemtest
+++++++++
+.. automodule:: lava_android_test.test_definitions.commands.ashmemtest
+
 commands
 ++++++++
 .. automodule:: lava_android_test.test_definitions.commands

=== added file 'lava_android_test/test_definitions/commands/ashmemtest.py'
--- lava_android_test/test_definitions/commands/ashmemtest.py	1970-01-01 00:00:00 +0000
+++ lava_android_test/test_definitions/commands/ashmemtest.py	2013-01-18 14:15:23 +0000
@@ -0,0 +1,32 @@
+# Copyright (c) 2012 Linaro
+
+# Author: Linaro Validation Team <[e-mail]>
+#
+# This file is part of LAVA Android Test.
+#
+#
+# This program is free software: you can redistribute it and/or modify
+# it under the terms of the GNU General Public License as published by
+# the Free Software Foundation, either version 3 of the License, or
+# (at your option) any later version.
+#
+# This program is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+# GNU General Public License for more details.
+#
+# You should have received a copy of the GNU General Public License
+# along with this program.  If not, see <http://www.gnu.org/licenses/>.
+"""
+This is basic unit test for the ashmem driver.
+
+**URL:**  None
+
+**Default Options:** None
+"""
+import lava_android_test.testdef
+
+test_name = 'ashmem'
+
+RUN_ADB_SHELL_STEPS = ['ashmemtest']
+PATTERN = "\s*\[(?P<test_case_id>\w+)\]:\s\w+\s(?P<result>\w+)"