Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(214)

Unified Diff: util/chip_size.sh

Issue 5136001: Add new testing framework along with a few micro-tests. (Closed) Base URL: svn://coreboot.org/flashrom/trunk/util
Patch Set: General updates, add rough EC unit testing Created 10 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: util/chip_size.sh
diff --git a/util/chip_size.sh b/util/chip_size.sh
new file mode 100755
index 0000000000000000000000000000000000000000..14c54df9ef8cc2beb4a7f510f5dff719a0932bc1
--- /dev/null
+++ b/util/chip_size.sh
@@ -0,0 +1,26 @@
+#!/bin/sh
+#
+# Copyright (C) 2010 Google Inc.
+# Written by David Hendricks for Google Inc.
+#
+# 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 2 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, write to the Free Software
+# Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
+
+REPORTED_SIZE=$(./flashrom ${FLASHROM_PARAM} --get-size 2>/dev/null | tail -n 1)
+ACTUAL_SIZE=$(stat --printf="%s\n" ${BACKUP})
+if [ ${REPORTED_SIZE} -ne ${ACTUAL_SIZE} ]; then
hailfinger 2010/11/23 20:05:34 This only tests that the reported size and the siz
Stefan Reinauer 2010/11/23 21:14:14 Is there anything we can do about that?
dhendrix 2010/11/24 02:48:14 Hmmm, that's a good point. Perhaps we also need t
+ return ${EXIT_FAILURE}
+fi
+
+return ${EXIT_SUCCESS}

Powered by Google App Engine
This is Rietveld 408576698