tests: drivers: spi: extend error_cases test.

[nordic-babu]

Extend with no reconfiguration on every tranceive case.

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[decsny]

I do not think it says anywhere in the SPI API that this would be the behavior. Even if it is a de facto of many drivers. If you want to make this official, it needs to be documented.

[nordic-babu]

I do not think it says anywhere in the SPI API that this would be the behavior. Even if it is a de facto of many drivers. If you want to make this official, it needs to be documented.

Please let me know if it's not enough https://github.com/zephyrproject-rtos/zephyr/blob/main/include/zephyr/drivers/spi.h#L434,

If not I will move this to nordic-specific area for now, this scenario just matched exisitng test suite pretty well for me.

On the other hand frequency of 124999Hz is afaik not documented as invalid anywhere, but such case is present in already existing tests.

[decsny]

This is a weird case of the API , where the API is not saying how things should be but has a warning about how things may be, so I don't know if it's right to be testing other behavior as an "error". Let's see what @tbursztyka and @teburd think about it

[nordic-babu]

on the plus side: at least we'll notice if this behavior changes during some rework and make sure that such a change is intentional (as it requires change in tests).