Make sure we cleanup what was generated by the architecture.

tools/testing/test_case.py

Index: tools/testing/test_case.py
===================================================================
--- tools/testing/test_case.py	(revision 431)
+++ tools/testing/test_case.py	(working copy)
@@ -154,3 +154,7 @@
 
   def GetName(self):
     return self.path[-1]
+
+  def Cleanup(self):
+    if not self.context.keep_temporary_files:

[zundel, 2011/10/14 13:57:22]

What do you think about moving this method to TestCase.Cleanup and sharing code
with StandardTestCase.

We can test for the presence of run_arch with:

if run_arch in self and not self.context.keep_temporary_files:
 ...

[ngeoffray, 2011/10/14 14:08:39]

I'd rather not touch TestCase. The run_arch kind of looks like detail to me,
which TestCase should not deal with. Would you prefer an class in between
TestCase and all the other test cases?

[zundel, 2011/10/14 14:13:02]

No, I don't think adding a class in between would be much good just for this one
case. 

[ngeoffray, 2011/10/14 14:25:06]

OK, committing then. If it turns out there are too many things that can be
shared in TestCase, we can reconsider.

+      self.run_arch.Cleanup()