[saket0187]

This PR adds feature requested in #7504

The test file I added contains very small tests. Should we keep it?

[petrasovaa]

The test file I added contains very small tests. Should we keep it?

Create a pytest in a separate PR.

[saket0187]

The test file I added contains very small tests. Should we keep it?

Create a pytest in a separate PR.

I'm happy to move the tests to a separate PR! Just to double-check: since these tests specifically validate the new C logic added here, wouldn't it make more sense to just convert this file to pytest in this PR only, or should I strictly keep the C code and tests separated?

@petrasovaa @ninsbl

[petrasovaa]

The test file I added contains very small tests. Should we keep it?

Create a pytest in a separate PR.

I'm happy to move the tests to a separate PR! Just to double-check: since these tests specifically validate the new C logic added here, wouldn't it make more sense to just convert this file to pytest in this PR only, or should I strictly keep the C code and tests separated?

@petrasovaa @ninsbl

Sorry what I meant to say is to create a pytest testing the current functionality in a separate PR and (after that is merged) then in this PR expand on that test - that way we can check there is no regression.

[saket0187]

Hello @ninsbl , I believe I have addressed issue #7538 as well. It seems the exit() statement was simply missing. Could you please take a look and cross-check it on your end?