BootloaderTest: Test GET_SERIAL_NUMBER

Author: matthijskooijman

BootloaderTest/BootloaderTest.ino

@@ -324,6 +324,29 @@ test(070_get_hardware_info) {
   assertEqual(flash_size, AVAILABLE_FLASH_SIZE);
 }
 
+test(075_get_serial_number) {
+  uint8_t data[9];
+  assertTrue(run_transaction_ok(Commands::GET_SERIAL_NUMBER, nullptr, 0, data, sizeof(data)));
+
+  auto on_failure = [&data]() {
+    for (uint8_t i = 0; i < sizeof(data); ++i) {
+      Serial.print(" 0x"); Serial.print(data[i], HEX);
+    }
+  };
+
+  // This is not required by the protocol, but the current
+  // implementation returns the unique data from the attiny chip, which
+  // is a lot number, wafer number and x/y position. The lot number
+  // seems to be an ASCII uppercase letter followed by 5 ASCII numbers,
+  // so check that.
+  assertMoreOrEqual(data[0], 'A');
+  assertLessOrEqual(data[0], 'Z');
+  for (uint8_t i = 1; i < 6; ++i) {
+    assertMoreOrEqual(data[i], '0');
+    assertLessOrEqual(data[i], '9');
+  }
+}
+
 test(080_crc_error) {
   uint8_t status;
   uint8_t crc_xor = random(1, 256);

BootloaderTest/Constants.h

@@ -48,10 +48,11 @@ struct Commands {
     SET_I2C_ADDRESS       = 0x01,
     POWER_UP_DISPLAY      = 0x02,
     GET_HARDWARE_INFO     = 0x03,
-    START_APPLICATION     = 0x04,
-    WRITE_FLASH           = 0x05,
-    FINALIZE_FLASH        = 0x06,
-    READ_FLASH            = 0x07,
+    GET_SERIAL_NUMBER     = 0x04,
+    START_APPLICATION     = 0x05,
+    WRITE_FLASH           = 0x06,
+    FINALIZE_FLASH        = 0x07,
+    READ_FLASH            = 0x08,
     END_OF_COMMANDS
   };
 };