servalproject / serval-dna

The Serval Project's core daemon that implements Distributed Numbering Architecture (DNA), MDP, VoMP, Rhizome, MeshMS, etc.
http://servalproject.org
Other
171 stars 80 forks source link

keyring create does not destroy existing keys. #89

Closed b0661 closed 9 years ago

b0661 commented 9 years ago

Doing my own tests, keyring create does not destroy existing keys.

Looking at the keyring testcase it seems the output of keyring create is ckecked for keys. But doing a keyring list afterwards still shows the two keys.

doc_KeyringCreate="Create keyring destroys existing keys" test_KeyringCreate() { executeOk_servald keyring add '' executeOk_servald keyring add '' executeOk_servald keyring list assert_keyring_list 2 executeOk_servald keyring create executeOk_servald keyring list <== added assert_keyring_list 0 }

lakeman commented 9 years ago

True, that doesn't seem to be what keyring create has ever done.

I've broken the test in 032502f80 as you suggested, since it wasn't doing what it said it was.

quixotique commented 9 years ago

Fixed by d031747

karlgilbert commented 9 years ago

Xhacktly.