From 2f6751e48a37699cfd999e0afd8731ca3962611c Mon Sep 17 00:00:00 2001
From: Nick Brassel <nick@tzarc.org>
Date: Tue, 8 Mar 2022 16:58:05 +1100
Subject: Asymmetric encoders, encoder tests. (#16068)

---
 quantum/encoder/tests/encoder_tests.cpp | 36 ++++++++++++++++-----------------
 1 file changed, 18 insertions(+), 18 deletions(-)

(limited to 'quantum/encoder/tests/encoder_tests.cpp')

diff --git a/quantum/encoder/tests/encoder_tests.cpp b/quantum/encoder/tests/encoder_tests.cpp
index 1888fdab8d..b7c18aeec0 100644
--- a/quantum/encoder/tests/encoder_tests.cpp
+++ b/quantum/encoder/tests/encoder_tests.cpp
@@ -30,12 +30,12 @@ struct update {
     bool   clockwise;
 };
 
-uint8_t uidx = 0;
+uint8_t updates_array_idx = 0;
 update  updates[32];
 
 bool encoder_update_kb(uint8_t index, bool clockwise) {
-    updates[uidx % 32] = {index, clockwise};
-    uidx++;
+    updates[updates_array_idx % 32] = {index, clockwise};
+    updates_array_idx++;
     return true;
 }
 
@@ -47,15 +47,15 @@ bool setAndRead(pin_t pin, bool val) {
 class EncoderTest : public ::testing::Test {};
 
 TEST_F(EncoderTest, TestInit) {
-    uidx = 0;
+    updates_array_idx = 0;
     encoder_init();
     EXPECT_EQ(pinIsInputHigh[0], true);
     EXPECT_EQ(pinIsInputHigh[1], true);
-    EXPECT_EQ(uidx, 0);
+    EXPECT_EQ(updates_array_idx, 0);
 }
 
 TEST_F(EncoderTest, TestOneClockwise) {
-    uidx = 0;
+    updates_array_idx = 0;
     encoder_init();
     // send 4 pulses. with resolution 4, that's one step and we should get 1 update.
     setAndRead(0, false);
@@ -63,26 +63,26 @@ TEST_F(EncoderTest, TestOneClockwise) {
     setAndRead(0, true);
     setAndRead(1, true);
 
-    EXPECT_EQ(uidx, 1);
+    EXPECT_EQ(updates_array_idx, 1);
     EXPECT_EQ(updates[0].index, 0);
     EXPECT_EQ(updates[0].clockwise, true);
 }
 
 TEST_F(EncoderTest, TestOneCounterClockwise) {
-    uidx = 0;
+    updates_array_idx = 0;
     encoder_init();
     setAndRead(1, false);
     setAndRead(0, false);
     setAndRead(1, true);
     setAndRead(0, true);
 
-    EXPECT_EQ(uidx, 1);
+    EXPECT_EQ(updates_array_idx, 1);
     EXPECT_EQ(updates[0].index, 0);
     EXPECT_EQ(updates[0].clockwise, false);
 }
 
 TEST_F(EncoderTest, TestTwoClockwiseOneCC) {
-    uidx = 0;
+    updates_array_idx = 0;
     encoder_init();
     setAndRead(0, false);
     setAndRead(1, false);
@@ -97,7 +97,7 @@ TEST_F(EncoderTest, TestTwoClockwiseOneCC) {
     setAndRead(1, true);
     setAndRead(0, true);
 
-    EXPECT_EQ(uidx, 3);
+    EXPECT_EQ(updates_array_idx, 3);
     EXPECT_EQ(updates[0].index, 0);
     EXPECT_EQ(updates[0].clockwise, true);
     EXPECT_EQ(updates[1].index, 0);
@@ -107,38 +107,38 @@ TEST_F(EncoderTest, TestTwoClockwiseOneCC) {
 }
 
 TEST_F(EncoderTest, TestNoEarly) {
-    uidx = 0;
+    updates_array_idx = 0;
     encoder_init();
     // send 3 pulses. with resolution 4, that's not enough for a step.
     setAndRead(0, false);
     setAndRead(1, false);
     setAndRead(0, true);
-    EXPECT_EQ(uidx, 0);
+    EXPECT_EQ(updates_array_idx, 0);
     // now send last pulse
     setAndRead(1, true);
-    EXPECT_EQ(uidx, 1);
+    EXPECT_EQ(updates_array_idx, 1);
     EXPECT_EQ(updates[0].index, 0);
     EXPECT_EQ(updates[0].clockwise, true);
 }
 
 TEST_F(EncoderTest, TestHalfway) {
-    uidx = 0;
+    updates_array_idx = 0;
     encoder_init();
     // go halfway
     setAndRead(0, false);
     setAndRead(1, false);
-    EXPECT_EQ(uidx, 0);
+    EXPECT_EQ(updates_array_idx, 0);
     // back off
     setAndRead(1, true);
     setAndRead(0, true);
-    EXPECT_EQ(uidx, 0);
+    EXPECT_EQ(updates_array_idx, 0);
     // go all the way
     setAndRead(0, false);
     setAndRead(1, false);
     setAndRead(0, true);
     setAndRead(1, true);
     // should result in 1 update
-    EXPECT_EQ(uidx, 1);
+    EXPECT_EQ(updates_array_idx, 1);
     EXPECT_EQ(updates[0].index, 0);
     EXPECT_EQ(updates[0].clockwise, true);
 }
-- 
cgit v1.2.3