Stop using NOLINT for setup and teardown methods

Signed-off-by: Mateusz Jablonski <[email protected]>

Author: JablonskiMateusz

opencl/test/unit_test/built_ins/built_in_tests.cpp

@@ -58,7 +58,7 @@ class BuiltInTests
       public ContextFixture,
       public ::testing::Test {
 
-    using BuiltInFixture::SetUp;
+    using BuiltInFixture::setUp;
     using ContextFixture::setUp;
 
   public:
@@ -73,12 +73,12 @@ class BuiltInTests
         ClDeviceFixture::setUp();
         cl_device_id device = pClDevice;
         ContextFixture::setUp(1, &device);
-        BuiltInFixture::SetUp(pDevice);
+        BuiltInFixture::setUp(pDevice);
     }
 
     void TearDown() override {
         allBuiltIns.clear();
-        BuiltInFixture::TearDown();
+        BuiltInFixture::tearDown();
         ContextFixture::tearDown();
         ClDeviceFixture::tearDown();
     }

opencl/test/unit_test/fixtures/built_in_fixture.cpp

@@ -1,5 +1,5 @@
 /*
- * Copyright (C) 2018-2021 Intel Corporation
+ * Copyright (C) 2018-2022 Intel Corporation
  *
  * SPDX-License-Identifier: MIT
  *
@@ -16,7 +16,7 @@
 
 using namespace NEO;
 
-void BuiltInFixture::SetUp(Device *pDevice) {
+void BuiltInFixture::setUp(Device *pDevice) {
     // create an instance of the builtins
     pBuiltIns = pDevice->getBuiltIns();
     pBuiltIns->setCacheingEnableState(false);
@@ -38,7 +38,7 @@ void BuiltInFixture::SetUp(Device *pDevice) {
     gEnvironment->igcPushDebugVars(igcDebugVars);
 }
 
-void BuiltInFixture::TearDown() {
+void BuiltInFixture::tearDown() {
     gEnvironment->igcPopDebugVars();
     gEnvironment->fclPopDebugVars();
 }

opencl/test/unit_test/fixtures/built_in_fixture.h

@@ -14,8 +14,8 @@ class Device;
 
 class BuiltInFixture {
   public:
-    void SetUp(NEO::Device *pDevice); // NOLINT(readability-identifier-naming)
-    void TearDown();                  // NOLINT(readability-identifier-naming)
+    void setUp(NEO::Device *pDevice);
+    void tearDown();
 
     NEO::BuiltIns *pBuiltIns = nullptr;
 };

opencl/test/unit_test/helpers/hardware_commands_helper_tests.cpp

@@ -36,15 +36,15 @@ void HardwareCommandsTest::SetUp() {
     cl_device_id device = pClDevice;
     ContextFixture::setUp(1, &device);
     ASSERT_NE(nullptr, pContext);
-    BuiltInFixture::SetUp(pDevice);
+    BuiltInFixture::setUp(pDevice);
     ASSERT_NE(nullptr, pBuiltIns);
 
     mockKernelWithInternal = std::make_unique<MockKernelWithInternals>(*pClDevice, pContext);
 }
 
 void HardwareCommandsTest::TearDown() {
     mockKernelWithInternal.reset(nullptr);
-    BuiltInFixture::TearDown();
+    BuiltInFixture::tearDown();
     ContextFixture::tearDown();
     ClDeviceFixture::tearDown();
 }

opencl/test/unit_test/helpers/hardware_commands_helper_tests.h

@@ -29,7 +29,7 @@ struct HardwareCommandsTest : ClDeviceFixture,
                               BuiltInFixture,
                               ::testing::Test {
 
-    using BuiltInFixture::SetUp;
+    using BuiltInFixture::setUp;
     using ContextFixture::setUp;
 
     void SetUp() override;