Skip to content

feat!: 调整商店测试输出内容#169

Merged
he0119 merged 7 commits intomainfrom
refactor/validation
Aug 23, 2023
Merged

feat!: 调整商店测试输出内容#169
he0119 merged 7 commits intomainfrom
refactor/validation

Conversation

@he0119
Copy link
Copy Markdown
Member

@he0119 he0119 commented Aug 23, 2023

validation.output 会直接返回验证通过的数据与错误。如果所有数据均验证通过,则返回 None。

同时将插件加载测试相关验证移至 validate_info 函数内,这样商店测试那边也能获得完整的验证信息。

插件信息增加测试时插件的版本号。

@he0119 he0119 enabled auto-merge (squash) August 23, 2023 00:36
@he0119 he0119 disabled auto-merge August 23, 2023 00:36
@codecov
Copy link
Copy Markdown

codecov bot commented Aug 23, 2023

Codecov Report

Merging #169 (5332cc2) into main (63aea02) will increase coverage by 15.27%.
The diff coverage is 100.00%.

❗ Current head 5332cc2 differs from pull request most recent head 047a557. Consider uploading reports for the commit 047a557 to get more accurate results

@@             Coverage Diff             @@
##             main     #169       +/-   ##
===========================================
+ Coverage   80.93%   96.20%   +15.27%     
===========================================
  Files          13       14        +1     
  Lines         855      791       -64     
===========================================
+ Hits          692      761       +69     
+ Misses        163       30      -133     
Files Changed Coverage Δ
src/plugins/publish/utils.py 91.59% <ø> (ø)
src/plugins/publish/render.py 100.00% <100.00%> (ø)
src/utils/store_test/models.py 100.00% <100.00%> (ø)
src/utils/store_test/validation.py 100.00% <100.00%> (ø)
src/utils/validation/__init__.py 95.65% <100.00%> (+1.71%) ⬆️

... and 1 file with indirect coverage changes

@he0119 he0119 changed the title refactor: 将插件加载测试相关验证移至 validate_info 函数内 feat!: 调整商店测试输出内容 Aug 23, 2023
@he0119 he0119 enabled auto-merge (squash) August 23, 2023 01:51
@he0119 he0119 merged commit a0d2c51 into main Aug 23, 2023
@he0119 he0119 deleted the refactor/validation branch August 23, 2023 01:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant