base: Rename TestFiber into SwitchingFiber
authorGiacomo Travaglini <giacomo.travaglini@arm.com>
Fri, 7 Jun 2019 13:10:47 +0000 (14:10 +0100)
committerGiacomo Travaglini <giacomo.travaglini@arm.com>
Sun, 9 Jun 2019 11:03:04 +0000 (11:03 +0000)
This is done since TestFiber is too generic and it is not the only
Fiber's testing subclass in the unit test

Change-Id: Idc386f487091ae9bdadae865090a6719a25583da
Signed-off-by: Giacomo Travaglini <giacomo.travaglini@arm.com>
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/19153
Reviewed-by: Gabe Black <gabeblack@google.com>
Maintainer: Gabe Black <gabeblack@google.com>
Tested-by: kokoro <noreply+kokoro@google.com>
src/base/fiber.test.cc

index 5ed13d4257c5e02bf311a7da9425271842f47481..7e7bfefe8e1966edd8a6a549d1cf4f911db400b5 100644 (file)
 
 #include "base/fiber.hh"
 
-class TestFiber : public Fiber
+class SwitchingFiber : public Fiber
 {
   public:
     const char *name;
     std::vector<Fiber *> next;
 
-    TestFiber(const char *name, std::initializer_list<Fiber *> l);
+    SwitchingFiber(const char *name, std::initializer_list<Fiber *> l);
 
     void checkExpected();
     void main();
 };
 
-extern TestFiber a;
-extern TestFiber b;
-extern TestFiber c;
+extern SwitchingFiber a;
+extern SwitchingFiber b;
+extern SwitchingFiber c;
 
-TestFiber a("A", { &b, &a, Fiber::primaryFiber(), &b, &c });
-TestFiber b("B", { &a, &c });
-TestFiber c("C", { &a, Fiber::primaryFiber(), Fiber::primaryFiber() });
+SwitchingFiber a("A", { &b, &a, Fiber::primaryFiber(), &b, &c });
+SwitchingFiber b("B", { &a, &c });
+SwitchingFiber c("C", { &a, Fiber::primaryFiber(), Fiber::primaryFiber() });
 
-std::vector<TestFiber *>::iterator expectedIt;
-std::vector<TestFiber *> expected({
+std::vector<SwitchingFiber *>::iterator expectedIt;
+std::vector<SwitchingFiber *> expected({
     &a, &b, &a, &a, /* main Fiber, */
     &a, &b, &c, &a, &c,
     /* main Fiber, */ &c, &c
 });
 
-TestFiber::TestFiber(
+SwitchingFiber::SwitchingFiber(
         const char *name, std::initializer_list<Fiber *> l) :
     name(name), next(l)
 {}
 
 void
-TestFiber::checkExpected()
+SwitchingFiber::checkExpected()
 {
     ASSERT_NE(expectedIt, expected.end());
-    TestFiber *e = *expectedIt++;
+    SwitchingFiber *e = *expectedIt++;
     EXPECT_EQ(e, this) << "Expected " << e->name << ", got " << name;
 }
 
 void
-TestFiber::main()
+SwitchingFiber::main()
 {
     checkExpected();
     for (auto &n : next) {