Skip to content
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

add unit test. #323

Merged
merged 14 commits into from
Jul 31, 2022
Merged

add unit test. #323

merged 14 commits into from
Jul 31, 2022

Conversation

dongzl
Copy link
Contributor

@dongzl dongzl commented Jul 28, 2022

What this PR does:

Which issue(s) this PR fixes:

ref #72

Special notes for your reviewer:

Does this PR introduce a user-facing change?:


@dongzl dongzl added the test Unit test. label Jul 28, 2022
@codecov-commenter
Copy link

codecov-commenter commented Jul 28, 2022

Codecov Report

Merging #323 (3f6f526) into master (b04366f) will increase coverage by 0.04%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #323      +/-   ##
==========================================
+ Coverage   39.75%   39.79%   +0.04%     
==========================================
  Files          99      101       +2     
  Lines       13702    13945     +243     
==========================================
+ Hits         5447     5550     +103     
- Misses       7588     7721     +133     
- Partials      667      674       +7     
Impacted Files Coverage Δ
pkg/config/file/file.go 89.47% <100.00%> (ø)
pkg/proto/rule/rule.go 69.14% <0.00%> (-12.11%) ⬇️
pkg/runtime/namespace/namespace.go 53.44% <0.00%> (-1.34%) ⬇️
pkg/boot/discovery.go 64.39% <0.00%> (-0.91%) ⬇️
pkg/runtime/namespace/command.go 26.92% <0.00%> (-0.11%) ⬇️
pkg/config/model.go 28.97% <0.00%> (ø)
pkg/sequence/snowflake/snowflake.go 12.76% <0.00%> (ø)
pkg/boot/boot.go 56.75% <0.00%> (+0.59%) ⬆️
pkg/dataset/priority_queue.go 67.39% <0.00%> (+1.08%) ⬆️
... and 1 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b04366f...3f6f526. Read the comment docs.

@dongzl dongzl changed the title [WIP] add unit test. add unit test. Jul 29, 2022
@jjeffcaii jjeffcaii merged commit 5754645 into arana-db:master Jul 31, 2022
binbin0325 pushed a commit to binbin0325/arana that referenced this pull request Jul 31, 2022
* Add: add test case for iterator.

* fix: format import.

* fix: fix test case error.

* Add: add test case for shard_expr.

* Add: add license header.

* Add: add new test case.

* fix: fix import format.

* refactor schemanager cache to lru cache (arana-db#246)

Co-authored-by: Dong Jianhui <dongjianhui03@meituan.com>

* optimize maxAllowedPacket config fix: arana-db#292 arana-db#281 (arana-db#297)

* insert multi value at once request, throw error msg when any row error fix: arana-db#287 (arana-db#313)

* feat: support show status (arana-db#309)

* feat: max_allowed_packet is set by the user (arana-db#312)

Co-authored-by: Zonglei Dong <dongzonglei@apache.org>
Co-authored-by: Mulavar <978007503@qq.com>
Co-authored-by: Dong Jianhui <dongjianhui03@meituan.com>
Co-authored-by: adair peng <luqiangpeng5945@gmail.com>

* style: fix code style

* ci: fix ci bug

* ci: fix ci bug

* ci: fix ci error

* fix: fix ci bug

* fix: fix runtime.Runtime use

Co-authored-by: Jeffsky <jjeffcaii@outlook.com>
Co-authored-by: Zonglei Dong <dongzonglei@apache.org>
Co-authored-by: Mulavar <978007503@qq.com>
Co-authored-by: Dong Jianhui <dongjianhui03@meituan.com>
Co-authored-by: adair peng <luqiangpeng5945@gmail.com>

debug: listen capabilities fix:arana-db#317 (arana-db#328)

feat: support "show collation" SQL statement. (arana-db#307)

chore: add unit test. (arana-db#323)

* Add: add unit test for api.go.

* Add: add license header.

* fix: fix import format.

* fix: refactor mock file directory.

* fix: fix import format.

* fix: fix path name.

* fix: fix test config file content.

* add: add test case for file.go .

* fix: fix import format.

* fix: fix file format.

* fix: fix code review problem.

* fix: fix error test case.

* fix: fix bootstrap test config.

* fix: fix test config data.

commit
@dongzl dongzl deleted the unit-test branch August 27, 2022 08:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
test Unit test.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants