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.

Sign up for GitHub

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

Jump to bottom

move *Spec tests in src/test (one by one) from 'specs' to scalatest #64

Closed
pankajgupta opened this issue Apr 28, 2014 · 3 comments
Closed

move *Spec tests in src/test (one by one) from 'specs' to scalatest #64

pankajgupta opened this issue Apr 28, 2014 · 3 comments
Labels
enhancement newbie

Comments

@pankajgupta
Copy link
Contributor

'org.specs' has been deprecated and the tests should move to using org.scalatest. An example of the latter is in com.twitter.cassovary.graph.TestGraphSpec.
@szymonm
Copy link
Contributor

szymonm commented Dec 30, 2014

Please fill the Migrator field in the sheet before working on a particular test to avoid duplication.

@pankajgupta pankajgupta mentioned this issue Jan 3, 2015
Closed
@pankajgupta
Copy link
Contributor Author

Since someone asked, here is the online documentation for ScalaTest -- we are using WordSpec and ShouldMatchers here.
http://doc.scalatest.org/2.2.1/#org.scalatest.WordSpec
http://www.scalatest.org/user_guide/using_matchers

Can also check out recent pull requests #94 or #96 to see the difference between Specs and Scalatest

Also, please fill the sheet mentioned by @szymonm above as multiple people are working on this issue.

pankajgupta added a commit that referenced this issue Jan 10, 2015
@pankajgupta
Merge pull request #101 from AnishShah/SynchronizedDynamicGraphSpec
e5b90d7 
[issue #64] SynchronizedDynamicGraphSpec migration to scalatest
pankajgupta added a commit that referenced this issue Jan 10, 2015
@pankajgupta
Merge pull request #102 from AnishShah/DirectedGraphUtilsSpec
f0d69fc 
[issue #64] DirectedGraphUtilsSpec migration to scalatest
@pankajgupta pankajgupta mentioned this issue Jan 16, 2015
Merged
@pankajgupta
Copy link
Contributor Author

Fixed and removed specs in PR #117

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement newbie
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@pankajgupta @szymonm