commit 2c169cbc538c6c3864bfe8fea2c2c35b40e48d19
author Shawn Nematbakhsh <shawnn@chromium.org> Tue Oct 29 16:23:58 2013 -0700
committer chrome-internal-fetch <chrome-internal-fetch@google.com> Wed Oct 30 23:10:28 2013 +0000
tree ca5d57ac00c2769d3052b565acdc432c704e2a3a
parent 28c2df3a96db52e424d5a93a781b13e39260c229

cros_generate_breakpad_symbols: Add directory exclusion list

Add new parameter to cros_generate_breakpad_symbols to exclude elf files
residing in a given directory and its subdirectories.

TEST=pass cros_generate_breakpad_symbols_unittest.py
BUG=chrome-os-partner:23671

Change-Id: Iea28dc210fd82ca26b606f8f567d546fc5645ee2
Reviewed-on: https://chromium-review.googlesource.com/175032
Reviewed-by: Mike Frysinger <vapier@chromium.org>
Commit-Queue: Shawn Nematbakhsh <shawnn@chromium.org>
Tested-by: Shawn Nematbakhsh <shawnn@chromium.org>

scripts/cros_generate_breakpad_symbols_unittest.py

diff --git a/scripts/cros_generate_breakpad_symbols_unittest.py b/scripts/cros_generate_breakpad_symbols_unittest.py
index b9c0d66..dfc8f19 100755
--- a/scripts/cros_generate_breakpad_symbols_unittest.py
+++ b/scripts/cros_generate_breakpad_symbols_unittest.py
@@ -187,6 +187,14 @@
       self.assertEquals(gen_mock.call_count, 2)
       self.assertExists(dummy_file)
 
+  def testExclusionList(self, gen_mock):
+    """Verify files in directories of the exclusion list are excluded"""
+    exclude_dirs = ['bin', 'usr', 'fake/dir/fake']
+    with parallel_unittest.ParallelMock():
+      ret = cros_generate_breakpad_symbols.GenerateBreakpadSymbols(
+          self.board, sysroot=self.board_dir, exclude_dirs=exclude_dirs)
+      self.assertEquals(ret, 0)
+      self.assertEquals(gen_mock.call_count, 1)
 
 class GenerateSymbolTest(cros_test_lib.MockTempDirTestCase):