diff --git a/tests/gy521_driver/test_gy521_driver.c b/tests/gy521_driver/test_gy521_driver.c index 1f0e6c5..47435cb 100644 --- a/tests/gy521_driver/test_gy521_driver.c +++ b/tests/gy521_driver/test_gy521_driver.c @@ -39,12 +39,12 @@ void fake_twi_rx(uint8_t slave_addr, uint8_t *data, uint8_t size) reg_addr_arr[0].dev_addr = slave_addr; /*Read the registers requested*/ - for(; idx < size; idx++) { + for(uint8_t i = 0; i < size; i++) { /*Wridxte the register read address*/ - reg_addr_arr[idx].addr = *(data + idx); + reg_addr_arr[i + idx].addr = *(data + i); - /*Read the response value idxnto the pased data ptr*/ - *(data + idx) = reg_addr_arr[idx].value; + /*Read the response value into the pased data ptr*/ + *(data + i) = reg_addr_arr[i + idx].value; } /*Post increment the global index*/ idx++; @@ -100,16 +100,16 @@ static void test_gy521_update_accel(void **sate) for(uint8_t i = 0; i < 6; i++) { reg_addr_arr[i].value = i; } - + + /*Zero the global index for the TWI*/ idx = 0; gy521_update_accel(m); - /*Ensure the correct registers are read*/ _Bool is_correct = 1; - for(int i = 0; i < 6; i++){ - /*The starting address of the registers is 59 and goes to 64*/ - print_message("expected: %d, actual: %d\n", (59+i), reg_addr_arr[i].addr); + assert_true(is_correct); + for(uint8_t i = 0; i < 6; i++){ + /*The starting address of the registers is 59 and goes to 64*/ if((59 + i) != reg_addr_arr[i].addr) { is_correct = 0; } @@ -117,7 +117,9 @@ static void test_gy521_update_accel(void **sate) assert_true(is_correct); /*Check that the values are assembled correctly*/ - + + + gy521_free(m); }