search

Mardak / profile

2 stars 7 forks source link

Make test-Crypto environment bootstrap so it actually runs #41

Closed oyiptong closed 10 years ago

oyiptong commented 10 years ago

The issue is that test-Crypto::test_WriteMappings will not test anything unless a survey.txt file is dropped in a directory.

The automated test run should run independent of human intervention, i.e. without the need for someone to drop a file in a directory before it actually runs.

It is also incorrect to assert.ok(true) regardless of if the test has run

oyiptong commented 10 years ago

For reference: https://github.com/Mardak/profile/blob/master/test/test-Crypto.js#L54

mzhilyaev commented 10 years ago

This is done by design. This code generates uuid mapping IF survey.txt is in the root directory. Other options for encrypted uuid mapping generation were too involved, so me and Ed decided that using test infrastructure for generation is the least evil.

mzhilyaev commented 10 years ago

closing as not worth fixing - oyiptong feel free to object