[pass] Importing tests

Jason A. Donenfeld Jason at zx2c4.com
Wed Apr 23 04:03:31 CEST 2014


On Wed, Apr 23, 2014 at 4:01 AM, Von Welch <von at vwelch.com> wrote:

> It'd be nice if you could develop consistent style for $variable vs
>> ${variable}, but for the most part, things look okay now.
>
>
> I tend to use the brackets if I think it makes for easier-to-read code,
> but it's fairly subjective and I don't feel strongly. Looks like you prefer
> the first form unless you are using a modifier?
>

More or less, or when next to something odd in a quoted string.


>
> One thing I'd like to see in the harness is generating new gpg keys on the
>> fly, but perhaps this is unnecessary.
>
>
> I don't see the value off-hand
>

Okay.


>
> Another thing is I'd like to keep setup.sh fairly lean, and move any of
>> those functions that aren't needed globally into the specific tests
>> themselves. But for now, again, things seem fine.
>
>
> Every function is used in multiple tests, or at least I envision it will
> be once we have a full suite. In any case, I suggest waiting to do that
> optimization until you have a complete suite.
>

Agreed.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.zx2c4.com/pipermail/password-store/attachments/20140423/29bb7840/attachment.html>


More information about the Password-Store mailing list