Add a test for multicard reader support.

Add a test for multicard reader support.

This does not test a specific reader.  It really just tests for the
existance of some removable block device that we can assume comes from
a multicard reader.

This means that USB sticks and SD cards in the internal reader should not
be present because they also provide removable block devices.

BUG=none
TEST=Automatic:  Plug in a multicard reader with media and run this test.

Change-Id: I1573d6ee340e3d2799d68cf04074d12dbe582e57

[petkov, 2010-08-23 20:43:34]

LGTM although this seems like a bit shallow test... I assume you've looked and
this is the best we could do.

http://codereview.chromium.org/3184017/diff/1/3
File client/site_tests/hardware_MultiReader/hardware_MultiReader.py (right):

http://codereview.chromium.org/3184017/diff/1/3#newcode16
client/site_tests/hardware_MultiReader/hardware_MultiReader.py:16: removable =
utils.read_one_line(os.path.join(dev, "removable"))
This will throw an exception if removable is not present -- do you want to catch
it?

[bfreed, 2010-08-23 22:05:50]

Yeah, this test is a bit shallow.  I just did not know how else to map the
external USB device to the removable partition.  So I left it as an assumption
that the removable partition is from the external multi card reader.

I guess such a mapping (from USB device to partition or vice versa) is not
impossible.  But it does appear to be platform specific.

I think it is best to leave it as is.  (Except for current fixes, that is.)

http://codereview.chromium.org/3184017/diff/1/3
File client/site_tests/hardware_MultiReader/hardware_MultiReader.py (right):

http://codereview.chromium.org/3184017/diff/1/3#newcode16
client/site_tests/hardware_MultiReader/hardware_MultiReader.py:16: removable =
utils.read_one_line(os.path.join(dev, "removable"))
No, that is not what I want.
It should be fixed in the version I just uploaded.

On 2010/08/23 20:43:35, petkov wrote:
> This will throw an exception if removable is not present -- do you want to
catch
> it?
>

[petkov, 2010-08-23 22:11:22]

still LGTM

http://codereview.chromium.org/3184017/diff/5001/6002
File client/site_tests/hardware_MultiReader/hardware_MultiReader.py (right):

http://codereview.chromium.org/3184017/diff/5001/6002#newcode19
client/site_tests/hardware_MultiReader/hardware_MultiReader.py:19:
re.sub("/removable", "", path))
I think this is equivalent to

os.path.dirname(path)

[bfreed, 2010-08-23 23:08:11]

http://codereview.chromium.org/3184017/diff/5001/6002
File client/site_tests/hardware_MultiReader/hardware_MultiReader.py (right):

http://codereview.chromium.org/3184017/diff/5001/6002#newcode19
client/site_tests/hardware_MultiReader/hardware_MultiReader.py:19:
re.sub("/removable", "", path))
On 2010/08/23 22:11:22, petkov wrote:
> I think this is equivalent to
> 
> os.path.dirname(path)
> 

Done.