-
-
Notifications
You must be signed in to change notification settings - Fork 245
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
SecAuditLogParts: add tests for the expected behavior #1304
base: main
Are you sure you want to change the base?
Conversation
@tty2 could you please enable actions in your fork? |
@jcchavezs Unfortunately actions for my account are disabled. I made a request for github support, but they answer not quick (probably during a month as I can see from threads) |
@jcchavezs I saw codecov complained about less coverage than before. |
This PR fixes the behavior for
SecAuditLogParts
that was not correct according to research. Look hereAdditionally it separates the auditlog and errorlog logic, before it was tightly coupled and this test failed on
nolog,auditlog
couple together, which is not supposed according to the doc.