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

Test case should be added to enhance the test coverage #195

Closed
slievrly opened this issue Jan 21, 2019 · 3 comments · Fixed by #822, #499, #504, #505 or #506
Closed

Test case should be added to enhance the test coverage #195

slievrly opened this issue Jan 21, 2019 · 3 comments · Fixed by #822, #499, #504, #505 or #506
Labels
good first issue Good for newcomers task: help-wanted Extra attention is needed

Comments

@slievrly
Copy link
Member

slievrly commented Jan 21, 2019

Requirement:

Test case should be added to enhance the test coverage . Junit and mockito can be used as test framework.
Welcome everyone to join in,you can comment on the issue that you're interest with.

Project address

https://github.com/alibaba/fescar/projects/1

Issues

status assignee issues
Add unit test to fescar-common module
Add unit test to fescar-config module
@xiajunhust Add unit test to fescar-core module
Add unit test to fescar-dubbo module
Add unit test to fescar-rm-datasource module
Add unit test to fescar-server module
@tswstarplanet Add unit test to fescar-spring module
Add unit test to fescar-tm module
@slievrly slievrly added the task: help-wanted Extra attention is needed label Jan 21, 2019
@gzlicanyi
Copy link
Contributor

I didn't find the standard create table SQL statement in the documentation or the code, including undo_log, account_tbl, storage_tbl, and order_tbl, so please provide it, and not others who want to help must guess it from the code.

@slievrly
Copy link
Member Author

It is included in the quickstart and will be added to the demo script later.

@slievrly slievrly pinned this issue Jan 22, 2019
@slievrly
Copy link
Member Author

UnitTest Must contain copyright

@lovepoem lovepoem changed the title [discuss] Test case should be added to enhance the test coverage Test case should be added to enhance the test coverage Jan 31, 2019
@lovepoem lovepoem added the good first issue Good for newcomers label Jan 31, 2019
github-ygy added a commit to github-ygy/fescar that referenced this issue Feb 28, 2019
github-ygy added a commit to github-ygy/fescar that referenced this issue Mar 1, 2019
github-ygy added a commit to github-ygy/fescar that referenced this issue Mar 1, 2019
github-ygy added a commit to github-ygy/fescar that referenced this issue Mar 2, 2019
github-ygy added a commit to github-ygy/fescar that referenced this issue Mar 2, 2019
github-ygy added a commit to github-ygy/fescar that referenced this issue Mar 3, 2019
github-ygy added a commit to github-ygy/fescar that referenced this issue Mar 3, 2019
slievrly pushed a commit that referenced this issue Mar 3, 2019
* fix #195 modified ut with AssertJ

* fix #195 add latch until the num add
github-ygy added a commit to github-ygy/fescar that referenced this issue Mar 4, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment