Enable SQLite for Unit Testing & Improve ODBC Compatibility in PHP 8.4

[jonkerw85]

Description:
This PR introduces the ability to run unit tests using SQLite, which can be helpful when an IBM i system is not available. It also enables running the unit tests on pull requests across multiple PHP versions using GitHub Actions. Additionally, it addresses some compatibility issues with ODBC connections in PHP 8.4.

Key Changes:

• Improved ODBC connection handling to support \Odbc\Connection and fix compatibility issues in PHP 8.4.
• Updated unit tests to dynamically determine the DSN based on configuration and added SQLite support through the mockDb2UsingSqlite option for improved flexibility and maintainability.

I understand that these changes might not have been requested, so if they're not of interest or not aligned with the project's goals, I completely understand. Thank you for considering these updates! 😊

[NattyNarwhal]

I like the idea of this, though I'm not sure how much you can actually test with a mocked database, since you can't call XMLSERVICE. I think the param building stuff can be tested without a DB connection?

[jonkerw85]

You're right that XMLSERVICE can't be tested this way. However, using sqlite3 allowed me to:

• Run all current unit tests.
• Compare the differences between the resources returned by ODBC in older versions and the new \Odbc\Connection class since PHP 8.4.
• Find an issue in ToolkitApi/Toolkit.php that I had missed in Update ODBC connection validation for PHP 8.4 compatibility #212.

A more idiomatic approach would probably involve using Mockery, though I'm not sure how complex that would be. I tried this solution but didn’t fully grasp it, which is why I fell back to SQLite.

I believe there’s room for improvement in code and maintainability, but I’m not sure what aligns with the project's goals and scope. For example, when I opened #212 to resolve PHP 8.4 issues, my specific problems were solved. However, there is still code with implicitly nullable types. Also, fix #212 introduced a regression that you later resolved in f21f7c7.

Tools like PHP-CS-Fixer and PHPStan could help with these issues. I have experience with them, and I think one or both could improve code quality. Adding unit tests is also an option, but in my experience, it is often more involved.

[NattyNarwhal]

Could you split the ODBC changes from the SQLite mocking into a separate PR?

I have a suspicion a different approach for the mocking part could be a dummy backend (as we have for i.e. the DB/SSH/HTTP ones); this might be a way to avoid putting mocking code in the DB backend.

[NattyNarwhal]

FWIW, if you do need access to an IBM i system with PHP (to test things that might be harder to run on a local system), let us know. We also need to figure out a CI story for the Toolkit in general as you've found out.

[jonkerw85]

@NattyNarwhal It might be useful to differentiate between integration tests and unit tests and to skip integration tests on systems that are not i5 or do not have an configuration file.

I think I can invest some time in building unit tests for important parts of the library that are currently not covered. I am mainly considering some methods of the XMLWrapper class and possibly the Toolkit class as well. Would this be helpful and what areas would be important to cover?