[6.2][AI] Adding AI-generated unittests for Log classes by Hackwar · Pull Request #48653 · joomla/joomla-cms · GitHub
Skip to content

[6.2][AI] Adding AI-generated unittests for Log classes - #48653

Open
Hackwar wants to merge 27 commits into
joomla:6.2-devfrom
Hackwar:6.2-unittests-3
Open

Hackwar wants to merge 27 commits into
joomla:6.2-devfrom
Hackwar:6.2-unittests-3

Conversation

@Hackwar

@Hackwar Hackwar commented Oct 18, 2026

Copy link
Copy Markdown
Member
  • I read the Generative AI policy and my contribution is either not created with the help of AI or is compatible with the policy and GNU/GPL 2 or later.

Summary of Changes

This PR implements unittests for the Log classes. These tests are AI-generated. The goal is to quickly achieve a bigger coverage for our codebase.

Testing Instructions

The tests to get this out the door are basically to see if they pass in Github Actions and for a second person to review the test code if they actually do what we want them to do.

Link to documentations

Please select:

  • Documentation link for guide.joomla.org:

  • No documentation changes for guide.joomla.org needed

  • Pull Request link for manual.joomla.org:

  • No documentation changes for manual.joomla.org needed

heelc29 and others added 26 commits July 26, 2026 22:37
PHPUnit 10.0: Using a non-static method as a data provider is now deprecated
PHPUnit 10.0: Using a non-public method as a data provider is now deprecated
works with PHPUnit 9 because option convertWarningsToExceptions=true
PHPUnit 12.2: A warning is now emitted when a data provider provides data sets that have more values than the test method consumes using arguments
sebastianbergmann/phpunit#6213
PHPUnit 12.5: A PHPUnit notice is now emitted for test methods that create a mock object but do not configure an expectation for it
prefer same (typesafe) over equals

pgsql should match mysqli
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants