mirror
/
libx52
mirror of https://github.com/nirenjan/libx52.git
1
0
Fork 0
Code Releases Wiki Activity

Move LED tests in x52test to the end 

x52test doesn't perform any checks to see if the connected device
supports setting individual LEDs. Therefore, if an X52 (non-Pro) was
connected when running x52test, it would fail when running the LED
tests, and not perform any of the subsequent tests.

By moving the LED tests to the end, this allows the other tests to run
on a non-Pro X52. Although the tests would still fail, it won't actually
break anything.

Addresses #19.
debian-packaging
nirenjan 2020-05-18 14:59:24 -07:00
parent be1f7e0d5a
commit 45f009ac90
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
utils/test/x52_test.c
View File

@ -85,11 +85,11 @@ static void signal_handler(int sig)
#define TESTS \ #define TESTS \
X(brightness, bri, "Test brightness scale (~ 1m)") \ X(brightness, bri, "Test brightness scale (~ 1m)") \
X(leds, led, "Test LED states (~ 45s)") \
X(mfd_text, mfd1, "Test MFD string display (~ 30s)") \ X(mfd_text, mfd1, "Test MFD string display (~ 30s)") \
X(mfd_display, mfd2, "Test MFD displays all characters (~ 2m 15s)") \ X(mfd_display, mfd2, "Test MFD displays all characters (~ 2m 15s)") \
X(blink_n_shift, blink, "Test the blink and shift commands (< 10s)") \ X(blink_n_shift, blink, "Test the blink and shift commands (< 10s)") \
X(clock, clock, "Test the clock commands (~1m)") X(clock, clock, "Test the clock commands (~1m)") \
X(leds, led, "Test LED states (~ 45s)")
enum { enum {
#define X(en, kw, desc) TEST_BIT_ ## en, #define X(en, kw, desc) TEST_BIT_ ## en,