tests: drivers: gpio: basic_api: Improve test

Add return code checking for gpio_pin_interrupt_configure functions.
Add disabling pin interrupt at the end of the test to restore pin
state.

Signed-off-by: Krzysztof Chruściński <[email protected]>

Author: nordic-krch

tests/drivers/gpio/gpio_basic_api/src/test_callback_manage.c

@@ -66,16 +66,19 @@ static int init_callback(const struct device *dev_in, const struct device *dev_o
 static void trigger_callback(const struct device *dev_in, const struct device *dev_out,
 			     int enable_cb)
 {
+	int rc;
+
 	gpio_pin_set(dev_out, PIN_OUT, 0);
 	k_sleep(K_MSEC(100));
 
 	cb_cnt[0] = 0;
 	cb_cnt[1] = 0;
 	if (enable_cb == 1) {
-		gpio_pin_interrupt_configure(dev_in, PIN_IN, GPIO_INT_EDGE_RISING);
+		rc = gpio_pin_interrupt_configure(dev_in, PIN_IN, GPIO_INT_EDGE_RISING);
 	} else {
-		gpio_pin_interrupt_configure(dev_in, PIN_IN, GPIO_INT_DISABLE);
+		rc = gpio_pin_interrupt_configure(dev_in, PIN_IN, GPIO_INT_DISABLE);
 	}
+	zassert_equal(rc, 0);
 	k_sleep(K_MSEC(100));
 	gpio_pin_set(dev_out, PIN_OUT, 1);
 	k_sleep(K_MSEC(1000));

tests/drivers/gpio/gpio_basic_api/src/test_config_trigger.c

@@ -73,6 +73,9 @@ ZTEST(after_flash_gpio_config_trigger, test_gpio_config_twice_trigger)
 	k_sleep(K_MSEC(10));
 	zassert_between_inclusive(cb_cnt, 0, 1, "Got %d interrupts", cb_cnt);
 
+	ret = gpio_pin_interrupt_configure(dev_in, PIN_IN, GPIO_INT_DISABLE);
+	zassert_ok(ret, "interrupt disabling failed");
+
 	gpio_remove_callback(dev_in, &drv_data->gpio_cb);
 }
 
@@ -125,5 +128,8 @@ ZTEST(after_flash_gpio_config_trigger, test_gpio_config_trigger)
 	k_sleep(K_MSEC(10));
 	zassert_between_inclusive(cb_cnt, 0, 1, "Got %d interrupts", cb_cnt);
 
+	ret = gpio_pin_interrupt_configure(dev_in, PIN_IN, GPIO_INT_DISABLE);
+	zassert_ok(ret, "interrupt disabling failed");
+
 	gpio_remove_callback(dev_in, &drv_data->gpio_cb);
 }