#2134: tests: eliminate typos

Author: amdarr

sketches/sketch_j1d.cc

@@ -1,8 +1,8 @@
 
-// sketched out example interface to VirutalContextCollection, just for
+// sketched out example interface to VirtualContextCollection, just for
 // pedagogical purposes
 template <typename Idx>
-struct VirutalContextCollection {
+struct VirtualContextCollection {
   Idx index();
   Idx min_index();
   Idx max_index();
@@ -32,7 +32,7 @@ struct FinishedMsg : vt::Message {
 };
 
 // the virtual context for a Jacobi1D block
-struct Jacobi1D : VirutalContextCollection<Index1D<int>> {
+struct Jacobi1D : VirtualContextCollection<Index1D<int>> {
   double* v1 = nullptr, *v2 = nullptr;
   double lb = 0.0, rb = 0.0;
   int size = 0;

tests/perf/common/test_harness.cc

@@ -441,7 +441,7 @@ void PerfTestHarness::SpinScheduler() {
 }
 
 void PerfTestHarness::GetMemoryUsage() {
-  // Memory footpring from PerfTestHarness' internal data structs are included
+  // Memory footprint from PerfTestHarness' internal data structs are included
   memory_use_[current_run_].push_back(mem_tracker_.getUsage());
 }
 

tests/unit/active/test_pending_send.cc

@@ -103,7 +103,7 @@ TEST_F(TestPendingSend, test_pending_send_hold) {
   theTerm()->finishedEpoch(ep);
 
   // It should not break out of this loop because of
-  // !theTerm()->isEpochTermianted(ep), thus `k` is used to
+  // !theTerm()->isEpochTerminated(ep), thus `k` is used to
   // break out
   int k = 0;
 
@@ -143,7 +143,7 @@ TEST_F(TestPendingSend, test_pending_broadcast_hold) {
   theTerm()->finishedEpoch(ep);
 
   // It should not break out of this loop because of
-  // !theTerm()->isEpochTermianted(ep), thus `k` is used to
+  // !theTerm()->isEpochTerminated(ep), thus `k` is used to
   // break out
   int k = 0;
 

tests/unit/collection/test_collection_construct_common.h

@@ -74,7 +74,7 @@ struct ConstructHandlers {
   static void handler(CollectionT* col, MessageT* msg) {
     // fmt::print(
     //   "{}: constructed TestCol: idx.x()={}\n",
-    //   theContext()->getNode(), col->getInex().x(), print_ptr(col)
+    //   theContext()->getNode(), col->getIndex().x(), print_ptr(col)
     // );
   }
 };

tests/unit/collection/test_collection_group.extended.cc

@@ -246,7 +246,7 @@ struct TestCollection : Collection<TestCollection, Index1D> {
     );
 
     // Even if msg was sent locally, it is still serialized,
-    // and the handler gots a fresh copy of it.
+    // and the handler gets a fresh copy of it.
     if (from_node == to_node) {
       EXPECT_TRUE(was_serialized);
     }

tests/unit/collection/test_model_weighted_communication_volume.nompi.cc

@@ -179,4 +179,4 @@ TEST_F(TestModelWeightedCommunicationVolume, test_model) {
   }
 }
 
-}}}}} // end namespace vt::tests::unit::veighted::volume
+}}}}} // end namespace vt::tests::unit::weighted::volume

tests/unit/collection/test_reduce_collection_race.cc

@@ -51,12 +51,12 @@ using TestReduceCollectionRace = TestParallelHarnessParam<int>;
 struct MyCol : vt::Collection<MyCol, vt::Index1D> {};
 using ReduceMsg = vt::collective::ReduceTMsg<int>;
 
-static int multipler = 0;
+static int multiplier = 0;
 
 struct ReduceFunctor {
   void operator()(ReduceMsg* msg) {
     auto const num_nodes = theContext()->getNumNodes();
-    auto const num_elems = num_nodes * multipler;
+    auto const num_elems = num_nodes * multiplier;
     fmt::print("reduce finished: val={}, num_elems={}\n", msg->getVal(), num_elems);
     EXPECT_EQ(msg->getVal(), (num_elems * (num_elems-1))/2);
   }
@@ -72,8 +72,8 @@ static void handler(MyCol* col) {
 TEST_P(TestReduceCollectionRace, test_reduce_race_1) {
   auto const num_nodes = theContext()->getNumNodes();
 
-  multipler = GetParam();
-  auto const range = Index1D(multipler * num_nodes);
+  multiplier = GetParam();
+  auto const range = Index1D(multiplier * num_nodes);
   auto proxy = theCollection()->constructCollective<MyCol>(
     range, "test_reduce_race_1"
   );

tests/unit/objgroup/test_objgroup.cc

@@ -377,7 +377,7 @@ struct MyTestObj {
     );
 
     // Even if msg was sent locally, it is still serialized,
-    // and the handler gots a fresh copy of it.
+    // and the handler gets a fresh copy of it.
     if (from_node == to_node) {
       EXPECT_TRUE(was_serialized);
     }

tests/unit/termination/test_term_cleanup.cc

@@ -58,7 +58,7 @@ struct TestTermCleanup :  TestParallelHarness {
   using TestMsgType = TestStaticBytesNormalMsg<64>;
 
   static void handler(TestMsgType* msg) {
-    //fmt::print("recieve msg\n");
+    //fmt::print("receive msg\n");
   }
 };
 

tests/unit/timetrigger/test_time_trigger.extended.cc

@@ -130,7 +130,7 @@ TEST_F(TestTimeTrigger, test_time_trigger_manager_trigger_ready) {
   trigger_manager->triggerReady(current_time + trigger_period_s + 0.01);
   EXPECT_EQ(triggered, 2);
 
-  // test unregisteration of triggers
+  // test unregistration of triggers
   auto prev_triggered = triggered;
   trigger_manager->removeTrigger(id);
   trigger_manager->triggerReady(current_time + trigger_period_s + 0.01);