-
Notifications
You must be signed in to change notification settings - Fork 48
Add Unit Testing to SplitImage #191
Comments
@smcgregor Hi! I was travelling so could not put in another pull request. I remember your comments on the last PR and I would like to clarify a bit about writing the unit test. var home_dom = document.createElement('a');
home_dom.addClass("home_domain");
document.body.appendChild(home_dom); and initialize them using proper initialization functions (such as |
Both processes you are describing are creating fixtures. Fixtures are an expression of the pre-conditions, not a specific filetype. We may eventually start defining fixtures in a collection like on Rails (and others), but for now your manual method of creating elements is sufficient.
|
Issue #191: Unit tests for SplitImage.
SplitImage has useful code for those looking to implement an image encryption app on top of Privly, but the architecture has some major changes introduced in the gsoc-development branch. It would be better to start over from that branch's Message application. |
The existing proof-of-concept for SplitImage does not have unit tests. We should add them.
See also #160 #189 #190
The text was updated successfully, but these errors were encountered: