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

Create tests for JsonUtil.scala #66

Closed
ruebot opened this issue Oct 2, 2017 · 2 comments
Closed

Create tests for JsonUtil.scala #66

ruebot opened this issue Oct 2, 2017 · 2 comments
Labels

Comments

@ruebot
Copy link
Member

ruebot commented Oct 2, 2017

JsonUtil.scala has no test coverage. We need to create tests for it.

@ruebot ruebot added the tests label Oct 2, 2017
ruebot pushed a commit that referenced this issue Oct 6, 2017
…#64 & #66 (#87)

* Add tests for remove Tuples, CreateGDF and RemoveHTML

* Add RemoveHttpHeader test.

* Add license header to tests.

* Issues #56, #57, #58, #59, #61, #62

* resolves Issue #85 (JsonUtil)

* Fix issue # 64 (ExtractDate coverage)

* Resolve Issue #63 (Extract Domain coverage)

* Fix error caused by repeated test names.
@greebie
Copy link
Contributor

greebie commented Oct 11, 2017

This 100% now. Closing.

@greebie greebie closed this as completed Oct 11, 2017
@ruebot
Copy link
Member Author

ruebot commented Oct 11, 2017

Resolved with: 41cd498

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

No branches or pull requests

2 participants