<div dir="ltr"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">It'd be nice if you could develop consistent style for $variable vs ${variable}, but for the most part, things look okay now.</blockquote>

<div class="gmail_extra"><br></div><div class="gmail_extra">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?</div>

<div class="gmail_extra"><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">One thing I'd like to see in the harness is generating new gpg keys on the fly, but perhaps this is unnecessary.</blockquote>

<div class="gmail_extra"><br></div><div class="gmail_extra">I don't see the value off-hand, but don't see any reason why it couldn't be done.</div><div class="gmail_extra"><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">

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.</blockquote><div><br></div>

<div>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.</div><div><br></div><div>

Von</div><div><br></div><div class="gmail_extra"><div class="gmail_quote">On Tue, Apr 22, 2014 at 12:36 PM, Jason A. Donenfeld <span dir="ltr"><<a href="mailto:Jason@zx2c4.com" target="_blank">Jason@zx2c4.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra">I've imported your tests here:</div>

<div class="gmail_extra"><a href="http://git.zx2c4.com/password-store/commit/?id=760c644a1d9f55c91b74521b112958966e00ea31" target="_blank">http://git.zx2c4.com/password-store/commit/?id=760c644a1d9f55c91b74521b112958966e00ea31</a><br>


</div><div class="gmail_extra"><br></div><div class="gmail_extra">It'd be nice if you could develop consistent style for $variable vs ${variable}, but for the most part, things look okay now.</div><div class="gmail_extra">


<br></div><div class="gmail_extra">One thing I'd like to see in the harness is generating new gpg keys on the fly, but perhaps this is unnecessary.</div><div class="gmail_extra"><br></div><div class="gmail_extra">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.</div>


<div class="gmail_extra"><br></div><div class="gmail_extra">Just a matter now of writing a full test suite. : )</div><div class="gmail_extra"><br></div><div class="gmail_extra"><br></div><div class="gmail_extra">@LUKAS: feel free to go to town on the harness, playing with things, changing things, cleaning things up, etc...</div>


</div>
</blockquote></div><br></div></div>