commit 16181d1d69f2590ac6919eccc4ee413c0f7989dc
author George Liu <liuxiwei@ieisystem.com> Thu Aug 22 20:05:20 2024 +0800
committer George Liu <liuxiwei@ieisystem.com> Wed Aug 28 09:41:50 2024 +0800
tree f1bc52da81860b8c53724a10f2b8aeba56ae493b
parent f05925599f7ad2b07e326eb3413a488ce21bd1c6

clang-tidy: Enable readability-static-accessed-through-instance check

Checks for member expressions that access static members through
instances, and replaces them with uses of the appropriate
qualified-id.

Signed-off-by: George Liu <liuxiwei@ieisystem.com>
Change-Id: I24de8ac8897de839594c6c71319febd0209bbc2e

test/utest-group-priority.cpp

diff --git a/test/utest-group-priority.cpp b/test/utest-group-priority.cpp
index f47e5f2..9d31905 100644
--- a/test/utest-group-priority.cpp
+++ b/test/utest-group-priority.cpp
@@ -59,7 +59,7 @@
     assertedGroups.insert(&mygroup);
 
     std::map<LedName, Layout::LedAction> map =
-        manager.getNewMap(assertedGroups);
+        phosphor::led::Manager::getNewMap(assertedGroups);
 
     assertMap(map, {
                        {"led1", Layout::Action::On},
@@ -107,7 +107,7 @@
     assertedGroups.insert(&groupC);
 
     std::map<LedName, Layout::LedAction> map =
-        manager.getNewMap(assertedGroups);
+        phosphor::led::Manager::getNewMap(assertedGroups);
 
     assertMap(map, {
                        {"led1", Layout::Action::On},
@@ -190,7 +190,7 @@
     // Off Service Action
     assertedGroups.insert(&groups->at(groupOffServiceAction));
 
-    assertMap(manager.getNewMap(assertedGroups),
+    assertMap(phosphor::led::Manager::getNewMap(assertedGroups),
               {
                   {pwr, Layout::Action::Off},
                   {fault, Layout::Action::Off},
@@ -198,7 +198,7 @@
 
     // On Ok
     assertedGroups.insert(&groups->at(groupSysOnOk));
-    assertMap(manager.getNewMap(assertedGroups),
+    assertMap(phosphor::led::Manager::getNewMap(assertedGroups),
               {
                   {pwr, Layout::Action::On},
                   {fault, Layout::Action::Off},
@@ -206,7 +206,7 @@
 
     // Off Fault
     assertedGroups.insert(&groups->at(groupSysOffFault));
-    assertMap(manager.getNewMap(assertedGroups),
+    assertMap(phosphor::led::Manager::getNewMap(assertedGroups),
               {
                   {pwr, Layout::Action::Off},
                   {fault, Layout::Action::On},
@@ -214,7 +214,7 @@
 
     // Off Fault
     assertedGroups.insert(&groups->at(groupSysOffFault));
-    assertMap(manager.getNewMap(assertedGroups),
+    assertMap(phosphor::led::Manager::getNewMap(assertedGroups),
               {
                   {pwr, Layout::Action::Off},
                   {fault, Layout::Action::On},
@@ -222,7 +222,7 @@
 
     // On Fault
     assertedGroups.insert(&groups->at(groupSysOnFault));
-    assertMap(manager.getNewMap(assertedGroups),
+    assertMap(phosphor::led::Manager::getNewMap(assertedGroups),
               {
                   {pwr, Layout::Action::On},
                   {fault, Layout::Action::On},
@@ -230,7 +230,7 @@
 
     // Off Locate
     assertedGroups.insert(&groups->at(groupSysOffLocate));
-    assertMap(manager.getNewMap(assertedGroups),
+    assertMap(phosphor::led::Manager::getNewMap(assertedGroups),
               {
                   {pwr, Layout::Action::Off},
                   {fault, Layout::Action::Blink},
@@ -238,7 +238,7 @@
 
     // On Locate
     assertedGroups.insert(&groups->at(groupSysOnLocate));
-    assertMap(manager.getNewMap(assertedGroups),
+    assertMap(phosphor::led::Manager::getNewMap(assertedGroups),
               {
                   {pwr, Layout::Action::On},
                   {fault, Layout::Action::Blink},
@@ -336,7 +336,7 @@
 
     // Sleep
     assertedGroups.insert(&groups->at(gBBUSleep));
-    assertMap(manager.getNewMap(assertedGroups),
+    assertMap(phosphor::led::Manager::getNewMap(assertedGroups),
               {
                   {bbu_ok, Layout::Action::Off},
                   {bbu_fault, Layout::Action::Off},
@@ -346,7 +346,7 @@
 
     // Fault
     assertedGroups.insert(&groups->at(gBBUFault));
-    assertMap(manager.getNewMap(assertedGroups),
+    assertMap(phosphor::led::Manager::getNewMap(assertedGroups),
               {
                   {bbu_ok, Layout::Action::Off},
                   {bbu_fault, Layout::Action::On},
@@ -356,7 +356,7 @@
 
     // EOL
     assertedGroups.insert(&groups->at(gBBUEOL));
-    assertMap(manager.getNewMap(assertedGroups),
+    assertMap(phosphor::led::Manager::getNewMap(assertedGroups),
               {
                   {bbu_ok, Layout::Action::Off},
                   {bbu_fault, Layout::Action::On},
@@ -366,7 +366,7 @@
 
     // On + Locate
     assertedGroups.insert(&groups->at(gBBUOnLocate));
-    assertMap(manager.getNewMap(assertedGroups),
+    assertMap(phosphor::led::Manager::getNewMap(assertedGroups),
               {
                   {bbu_ok, Layout::Action::On},
                   {bbu_fault, Layout::Action::Blink},