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

Setup coverage report #36

Merged
merged 1 commit into from
May 8, 2017
Merged

Setup coverage report #36

merged 1 commit into from
May 8, 2017

Conversation

gaocegege
Copy link
Member

No description provided.

Signed-off-by: Ce Gao <ce.gao@outlook.com>
@gaocegege gaocegege merged commit 3b71fd0 into master May 8, 2017
@gaocegege gaocegege deleted the test/add branch May 8, 2017 08:52
@gaocegege
Copy link
Member Author

ref #10

@gaocegege gaocegege changed the title Add Unit Test Cases Setup coverage report May 9, 2017
@jeremydouglass
Copy link
Member

Codacy looks good -- I hope it is helpful.

Have you worked using TDD in the past, and are you planning on writing unit tests as part of your development workflow?

Keep in mind that, for a project like this, a very practical model of testing is a large collection of simple reference sketches that either compile and run or fail. Many of the simple reference examples for Processing functions/methods -- once translated into R -- can also be sketches that serve as test cases (although you need to rig them up to count as runable tests for coverage reports).

@gaocegege
Copy link
Member Author

Yep. I have tried BDD and TDD in the past. But there are some difficulties when it comes to Processing.R.

Actually I have no idea how to test Processing.R. Unit test is not enough, I don't know much about UI test. Is there any documentation about the test in Processing?

@jeremydouglass
Copy link
Member

Is there any documentation about the tests in Processing?

I'm not too familiar with unit tests or other testing in Processing core, although I know where some testing code is. This strikes me as a good open discussion for the forum. I'll start a thread there.

@gaocegege
Copy link
Member Author

OK, I have one English writing exam this week, so maybe I don't have much time on Processing.R. But I will sync the thread as I can.

@jeremydouglass
Copy link
Member

Good, and don't forget: you don't have to spend any time on it -- GSOC doesn't start until after May 30! Good luck with your exam.

@gaocegege
Copy link
Member Author

gaocegege commented May 15, 2017

😄 Got it. Thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants