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: tests #1061
Add: tests #1061
Conversation
Codecov Report
@@ Coverage Diff @@
## dev #1061 +/- ##
==========================================
+ Coverage 78.59% 79.09% +0.49%
==========================================
Files 79 80 +1
Lines 1640 1674 +34
==========================================
+ Hits 1289 1324 +35
+ Misses 351 350 -1
Continue to review full report at Codecov.
|
@chfw @sunhailin-Leo 可以往这个分支上继续补充测试用例。 此分支目前还未准备好 merge |
pyecharts/options/series_options.py
Outdated
from ..commons.types import JSFunc, List, Numeric, Optional, Sequence, Tuple, Union | ||
|
||
|
||
class BasicOpts: | ||
pass | ||
opts: Union[dict, Sequence, None] = None |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
这个是 class 属性。
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
我知道 不过没什么影响
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
用 None 其实也可以
@@ -11,3 +14,61 @@ def test_bar_base(): | |||
assert c.theme == "white" | |||
assert c.renderer == "canvas" | |||
c.render("render.html") | |||
|
|||
|
|||
@patch("pyecharts.render.engine.write_utf8_html_file") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
💯
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
and will protect my SSD hard disk when running pyecharts unit tests.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good to go!
本次 PR 内容,