PHPUnit test for this module

Seemed to cause failures

Did come across any error by my local test.

I upload the patch here to test it via Gitlab.

Problem 1
- Root composer.json requires drupal/event_log_track-event_log_track, it could not be found in any version, there may be a typo in the package name.

I think it is the project name issue.

It might be the typo in those sub modules' info.yml file.

So I think we will need a settings to log CLI changes.

Also some weirdness because the original maintainer names the project events_log_track but names evertyhing event_log_track

@Mingsong got this working now with 1 test.

Could start a META for tracking additional ones but lets get one to land for next release.

Nice work @Stephen.

I just saw another issue where the $LogOnCLI parameter comes from.

#3205398: Allow log entries also on CLI operations →

If we remove it, it will affect their custom module/drush command.

The reason I suggest removing it is that, basically there are two variables to decide if this module will log cli actions.

This might cause confusion where the log_cli configuration is set to false in the setting form of this module, but cli logs might still are logged as $LogOnCLI parameter passed into that function is true.

We can deprecate it and remove in 4.0.x whenever that happens

Then it looks good me. So I mark it as RTBC.

Thanks again.

@Stephen, I add a new test for CLI action log to ensure by default CLI action won't be logged.

Here is the new patch for 3.0.x branch.

And the interdiff file

For you review. We are almost there.

Automatically closed - issue fixed for 2 weeks with no activity.