Skip to content
This repository has been archived by the owner on Feb 6, 2024. It is now read-only.

ci: run UT in ci #81

Merged
merged 2 commits into from
Oct 21, 2022
Merged

ci: run UT in ci #81

merged 2 commits into from
Oct 21, 2022

Conversation

ShiKaiWi
Copy link
Member

Which issue does this PR close?

Closes #

Rationale for this change

Currently, the unit test is not run in the ci, and that is unacceptable.

What changes are included in this PR?

  • Make ci run the unit tests;
  • Fix some broken unit tests;
  • Ignore some complex unit tests;

Are there any user-facing changes?

None

How does this change test

By the existing UT.

Copy link
Contributor

@chunshao90 chunshao90 left a comment

Choose a reason for hiding this comment

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

LGTM

@ShiKaiWi ShiKaiWi merged commit 4a78bbf into apache:main Oct 21, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants