<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">El jue., 12 ene. 2017 a las 0:54, David Cook (<<a href="mailto:dcook@prosentient.com.au">dcook@prosentient.com.au</a>>) escribió:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div lang="EN-AU" link="blue" vlink="purple" class="gmail_msg"><div class="m_-5833620569918746622WordSection1 gmail_msg"><p class="MsoNormal gmail_msg"><span style="font-size:11.0pt;font-family:"Calibri",sans-serif" class="gmail_msg">I suppose in this case I rather be told what’s enough than doing my best and being told it’s not enough. I suppose we’re all time poor, but I don’t have 100% time on Koha anymore.</span></p></div></div></blockquote><div><br></div><div>I guess there's no straight answer. It will depend on signoffers and QA people's opinion. People tend to focus on the stuff they care, lately, so that's why I said...</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div lang="EN-AU" link="blue" vlink="purple" class="gmail_msg"><div class="m_-5833620569918746622WordSection1 gmail_msg"><p class="MsoNormal gmail_msg"><span style="font-size:11.0pt;font-family:"Calibri",sans-serif" class="gmail_msg"><u class="gmail_msg"></u><u class="gmail_msg"></u></span></p><p class="MsoNormal gmail_msg"><span style="font-family:Calibri,sans-serif;font-size:11pt">Test::MockModule might work. Long term, it probably makes more sense to refactor get_matches, but I think it could work for now to show that the code I’m using works. Although in that case, should I put my sorting code as a function or copy and paste from the module into the test? I suppose any time a person says copy and paste, the answer is make a function…</span><br></p><p class="MsoNormal gmail_msg"><span style="font-size:11.0pt;font-family:"Calibri",sans-serif" class="gmail_msg"><u class="gmail_msg"></u> <u class="gmail_msg"></u></span></p><p class="MsoNormal gmail_msg"><span style="font-size:11.0pt;font-family:"Calibri",sans-serif" class="gmail_msg">What do you mean by keep in touch with QA members?</span></p></div></div></blockquote><div><br></div><div>... you should keep in touch with people that might have interest in helping you get your patches in. Sometimes people just drop some patches and wait for others to care about them. And it doesn't work like that most of the time. Keeping in touch means contacting them, explain things, ask for feedback on specific edges.</div><div><br></div><div>When you write patches for complicated stuff that doesn't have prior tests, your best bet is to mock most of the context so you can create the desired scenario. That's what I did with Auth_with_ldap.t at some point. It was even fun to write.</div></div></div><div dir="ltr">-- <br></div><div data-smartmail="gmail_signature"><div dir="ltr"><div style="color:rgb(117,117,117);font-family:"helvetica neue",helvetica,arial,sans-serif;font-size:12.8px">Tomás Cohen Arazi</div><div style="color:rgb(117,117,117);font-family:"helvetica neue",helvetica,arial,sans-serif;font-size:12.8px">Theke Solutions (<a href="http://theke.io/">https://theke.io</a>)<br>✆ +54 9351 3513384<br>GPG: B2F3C15F</div></div></div>