simple-store icon indicating copy to clipboard operation
simple-store copied to clipboard

Added a testcase that an IllegalArgumentException for invalid keys

Open PaulWoitaschek opened this issue 5 years ago • 2 comments

Currently simple-store provides no checking on the naming of it's keys. Instead, when sth. like ./ is used as a key, the store does unexpected things (moving wrong files etc.). This PR adds a testcase that simply expects an IllegalArgumentException to be thrown because that's what I would expect from the library to do in these cases.

PaulWoitaschek avatar Jun 15 '20 05:06 PaulWoitaschek

CLA assistant check
All committers have signed the CLA.

CLAassistant avatar Jun 15 '20 05:06 CLAassistant

I approve, can you add the check to the library?

kurtisnelson avatar Jul 07 '20 16:07 kurtisnelson