JasonThon / lightflus

A Lightweight, Cloud-Native Stateful Distributed Dataflow Engine
Apache License 2.0
99 stars 9 forks source link

[feat/LIG-58] move unittests #24

Closed JasonThon closed 1 year ago

JasonThon commented 1 year ago

Description

  1. move proto unittest from common -> proto
  2. modify workflow: should test proto package

Checklist

Refer to associated PR or issue links (optional)

Document (optional)

The changes that users must know

Release Note

codecov-commenter commented 1 year ago

Codecov Report

Merging #24 (bfca25c) into master (6f796e4) will decrease coverage by 2.56%. The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@            Coverage Diff             @@
##           master      #24      +/-   ##
==========================================
- Coverage   55.56%   53.00%   -2.56%     
==========================================
  Files          30       30              
  Lines        2849     2843       -6     
==========================================
- Hits         1583     1507      -76     
- Misses       1266     1336      +70     
Impacted Files Coverage Δ
src/proto/src/common_impl.rs 29.90% <ø> (-23.41%) :arrow_down:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more