Join GitHub today
GitHub is home to over 28 million developers working together to host and review code, manage projects, and build software together.
Sign upCreate tests for ExtractGraph.scala #49
Comments
ruebot
added
the
tests
label
Oct 2, 2017
ruebot
added this to To Do
in Test coverage
Oct 2, 2017
greebie
self-assigned this
Oct 11, 2017
greebie
referenced this issue
Oct 13, 2017
Merged
Add ExtractGraphTest; lint fixes on RemoveHttpHeaderTest. #92
This comment has been minimized.
This comment has been minimized.
|
Partially resolved with: f28c6e1 |
ruebot
moved this from To Do
to In Progress
in Test coverage
Oct 13, 2017
This comment has been minimized.
This comment has been minimized.
|
This is up to 94%. I am lost as to why we do not have 100%, but it relates to the Edges & Nodes case classes. |
ruebot
added
the
URA-Task
label
Feb 5, 2018
ruebot
moved this from In Progress
to To Do
in Test coverage
May 21, 2018
This comment has been minimized.
This comment has been minimized.
|
Looks like we need to account for both ExtractGraph and ExtractGraphX here? |
This comment has been minimized.
This comment has been minimized.
|
As part of #203 ExtractGraph has been deprecated. I can't remember why, but the tests have been commented out since it started. I think we can leave this. Except that I like the name ExtractGraph over ExtractGraphX. |
ruebot
moved this from To Do
to In Progress
in Test coverage
Aug 13, 2018
greebie
referenced this issue
Nov 22, 2018
Closed
Remove Deprecated ExtractGraph.scala file for next release. #291
This comment has been minimized.
This comment has been minimized.
|
With #291 merged,
|
This comment has been minimized.
This comment has been minimized.
ruebot
closed this
Nov 29, 2018
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
ruebot commentedOct 2, 2017
ExtractGraph.scalahas no test coverage. We need to create tests for it.