From 736b9995506669dde16a2f2c47453aed4e1af498 Mon Sep 17 00:00:00 2001 From: Gabe Black Date: Wed, 28 Nov 2018 17:40:17 -0800 Subject: [PATCH] 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 Maintainer: Gabe Black --- src/base/SConscript | 20 ++++++++++---------- 1 file changed, 10 insertions(+), 10 deletions(-) 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") -- 2.30.2