#777 test pam_interactive (including multistep capability) automatically

[d-w-moore]

No description provided.

[d-w-moore]

Test of multistep doesn't yet pass, even when iRODS4j is run, so something is probably wrong with the test setup. Awaiting a branch of irods/irods4j in which ports are exposed to test PRC on a valid setup

[korydraughn]

Is the PAM Interactive test run expected to fail in GitHub Actions? See the following:

• https://github.com/irods/python-irodsclient/actions/runs/31617526699/job/94183959927?pr=824

[d-w-moore]

Is the PAM Interactive test run expected to fail in GitHub Actions? See the following:

* https://github.com/irods/python-irodsclient/actions/runs/31617526699/job/94183959927?pr=824

No, it should pass ... looking

[d-w-moore]

Is the PAM Interactive test run expected to fail in GitHub Actions? See the following:

* https://github.com/irods/python-irodsclient/actions/runs/31617526699/job/94183959927?pr=824

No, it should pass ... looking

I can only get it to pass by reverting our change (702f723) to the native handoff from pam_interactive to the native auth phase. If I do that, all tests related to pam_interactive (including my new tests in test012...) pass once again.

Not sure how this should look, for the moment. Will need to study.

[korydraughn]

That's fine. Those changes do not need to be part of this PR. Keep them in a separate branch so that we can investigate them later.

[d-w-moore]

Waiting for tests to pass. As soon as they do, let's go with a final review.

[d-w-moore]

TODOs have been moved to another dev branch, issues will be created for those. Other troubleshooting related changes that resulted in dead ends have also been removed.

[korydraughn]

Looks like this is ready for squashing?

[d-w-moore]

Looks like this is ready for squashing?

I'd say so. Will squash soon.

[d-w-moore]

Squashed but then adding a commit to ensure conditions are right for both test methods in test012_pam_interactive_multistep.bats , in other words .irodsA is not present as an initial condition for each test.

[korydraughn]

Squash it.

[d-w-moore]

Final squash done.

[korydraughn]

Pound it.

Don't forget to remove the [SQUASH] text from the commit message.

[d-w-moore]

Pounded

[korydraughn]

Will merge after github actions reports success.