Skip to content

test: switches listauditevents into a unit test#1823

Merged
ale8k merged 6 commits intocanonical:v3from
ale8k:unit-test-listauditevents
Jan 22, 2026
Merged

test: switches listauditevents into a unit test#1823
ale8k merged 6 commits intocanonical:v3from
ale8k:unit-test-listauditevents

Conversation

@ale8k
Copy link
Contributor

@ale8k ale8k commented Jan 21, 2026

Description

Switches listauditevents into unit tests (verifying failures now). I also added an additional test to verify tabular output works as expected.

Engineering checklist

  • Documentation updated
  • Covered by unit tests
  • Covered by integration tests

Test instructions

@ale8k ale8k requested a review from a team as a code owner January 21, 2026 14:41
@ale8k ale8k force-pushed the unit-test-listauditevents branch from cfb6b74 to f2b0c93 Compare January 21, 2026 18:29
Copy link
Contributor

@kian99 kian99 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just change c.Assert() to c.Check() inside of the DoAndReturn() functions.

Copy link
Collaborator

@alesstimec alesstimec left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM with a comment

@ale8k ale8k force-pushed the unit-test-listauditevents branch from a0f3ef8 to 28ed9df Compare January 22, 2026 16:20
@ale8k ale8k merged commit 1907fbe into canonical:v3 Jan 22, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants