From: Gabe Black <gabeblack@google.com>
Date: Thu, 29 Nov 2018 01:40:17 +0000 (-0800)
Subject: base: Change the unit test binary names to use .test, and to be consistent.
X-Git-Tag: v19.0.0.0~1353
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=736b9995506669dde16a2f2c47453aed4e1af498;p=gem5.git

base: Change the unit test binary names to use .test, and to be consistent.

Most tests were named *test where * was the base name of the file being
tested, but some were named differently based on, for instance, the
name of the class that file implemented.

This change makes all the test names consistently based off of the file
name they test, and also brings in the new .test convention to make
them easier to read.

Now, if you have a file like fiber.cc you want to test, you'd have a
unit test in a file called fiber.test.cc, and a test called fiber.test
which would generate a binary called fiber.test.opt, fiber.test.debug,
etc.

Change-Id: I61d59016090371a9bae72066e7473a34aecea21f
Reviewed-on: https://gem5-review.googlesource.com/c/14677
Reviewed-by: Giacomo Travaglini <giacomo.travaglini@arm.com>
Maintainer: Gabe Black <gabeblack@google.com>
---

diff --git a/src/base/SConscript b/src/base/SConscript
index 673590371..d1ebbc9d4 100644
--- a/src/base/SConscript
+++ b/src/base/SConscript
@@ -40,26 +40,26 @@ Source('imgwriter.cc')
 Source('bmpwriter.cc')
 Source('callback.cc')
 Source('cprintf.cc', add_tags='gtest lib')
-GTest('cprintftest', 'cprintf.test.cc')
+GTest('cprintf.test', 'cprintf.test.cc')
 Source('debug.cc')
 if env['USE_FENV']:
     Source('fenv.c')
 if env['USE_PNG']:
     Source('pngwriter.cc')
 Source('fiber.cc')
-GTest('fibertest', 'fiber.test.cc', 'fiber.cc')
-GTest('coroutinetest', 'coroutine.test.cc', 'fiber.cc')
+GTest('fiber.test', 'fiber.test.cc', 'fiber.cc')
+GTest('coroutine.test', 'coroutine.test.cc', 'fiber.cc')
 Source('framebuffer.cc')
 Source('hostinfo.cc')
 Source('inet.cc')
 Source('inifile.cc')
-GTest('inifiletest', 'inifile.test.cc', 'inifile.cc', 'str.cc')
+GTest('inifile.test', 'inifile.test.cc', 'inifile.cc', 'str.cc')
 Source('intmath.cc')
 Source('logging.cc')
 Source('match.cc')
 Source('output.cc')
 Source('pixel.cc')
-GTest('pixeltest', 'pixel.test.cc', 'pixel.cc')
+GTest('pixel.test', 'pixel.test.cc', 'pixel.cc')
 Source('pollevent.cc')
 Source('random.cc')
 if env['TARGET_ISA'] != 'null':
@@ -69,7 +69,7 @@ Source('statistics.cc')
 Source('str.cc')
 Source('time.cc')
 Source('trace.cc')
-GTest('trietest', 'trie.test.cc')
+GTest('trie.test', 'trie.test.cc')
 Source('types.cc')
 
 Source('loader/aout_object.cc')
@@ -83,10 +83,10 @@ Source('loader/symtab.cc')
 
 Source('stats/text.cc')
 
-GTest('addr_range_test', 'addr_range.test.cc')
-GTest('AddrRangeMapTest', 'addr_range_map.test.cc')
-GTest('bituniontest', 'bitunion.test.cc')
-GTest('CircleBufTest', 'circlebuf.test.cc')
+GTest('addr_range.test', 'addr_range.test.cc')
+GTest('addr_range_map.test', 'addr_range_map.test.cc')
+GTest('bitunion.test', 'bitunion.test.cc')
+GTest('circlebuf.test', 'circlebuf.test.cc')
 
 DebugFlag('Annotate', "State machine annotation debugging")
 DebugFlag('AnnotateQ', "State machine annotation queue debugging")