<div dir="ltr">I guess the conclusion is the test directory was the old approach. Now we should focus in the -spec suffix. +1 to the spec. I think the epic story should be updated to sync our unit testing efforts during these sprints</div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Feb 9, 2017 at 9:49 PM, Peter Darrow <span dir="ltr"><<a href="mailto:pdarrow@redhat.com" target="_blank">pdarrow@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">So what's the conclusion here? Test files alongside code with -spec suffix, test files in a `test` directory, or test files in a `spec` directory? Since we're essentially rolling our own framework for RainCatcher modules, we should probably build a spec that defines exactly how modules should be laid out and then create a few JIRA issues to ensure we adhere to that spec everywhere.</div><div class="gmail_extra"><br><div class="gmail_quote"><div><div class="h5">On Thu, Feb 9, 2017 at 11:36 AM, Paolo Haji <span dir="ltr"><<a href="mailto:phaji@redhat.com" target="_blank">phaji@redhat.com</a>></span> wrote:<br></div></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div class="h5"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><span><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div class="m_-3414325717287123892m_-6905641602461563278gmail-m_3554561487792783638HOEnZb"><div class="m_-3414325717287123892m_-6905641602461563278gmail-m_3554561487792783638h5"><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div class="m_-3414325717287123892m_-6905641602461563278gmail-m_3554561487792783638m_6511481305295272475h5"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div class="m_-3414325717287123892m_-6905641602461563278gmail-m_3554561487792783638m_6511481305295272475m_-6884301129224893663HOEnZb"><div class="m_-3414325717287123892m_-6905641602461563278gmail-m_3554561487792783638m_6511481305295272475m_-6884301129224893663h5"><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div><div class="m_-3414325717287123892m_-6905641602461563278gmail-m_3554561487792783638m_6511481305295272475m_-6884301129224893663m_112356575843089851h5"><div dir="ltr"><div>I'm wondering why these files tend to be put together in the same directory? This has been a point of confusion when trying to understand the modules.</div><div>For example, the user-client.js & user-router.js files in <a href="https://github.com/feedhenry-raincatcher/raincatcher-user/tree/master/lib/user" target="_blank">https://github.com/feedhenr<wbr>y-raincatcher/raincatcher-user<wbr>/tree/master/lib/user</a> are intended for the browser and node.js respectively.</div></div></div></div></blockquote></div></div></blockquote></div></div></div></div></blockquote></div></div></div></div></blockquote></div></div></div></div></blockquote><div><br></div></span><div>On this point raincatcher-user is probably the module in worst shape in terms of clear separation.</div><div>For instance router/cloud.js is the cloud app part and router/mbaas.js is the auth service part... This is mostly because the auth service is handling all CRUD operations for the user data, as opposed to just handling auth and leaving the profile data in the cloud app, so there's a lot of calling through fh-mbaas-api from the cloud app to forward the http calls. So for this module specifically there's browser, node-cloud-app and node-auth-service code mixed in.</div></div></div></div>
<br></div></div><span class="">______________________________<wbr>_________________<br>
Feedhenry-raincatcher mailing list<br>
<a href="mailto:Feedhenry-raincatcher@redhat.com" target="_blank">Feedhenry-raincatcher@redhat.c<wbr>om</a><br>
<a href="https://www.redhat.com/mailman/listinfo/feedhenry-raincatcher" rel="noreferrer" target="_blank">https://www.redhat.com/mailman<wbr>/listinfo/feedhenry-raincatche<wbr>r</a><br>
<br></span></blockquote></div><br></div>
<br>______________________________<wbr>_________________<br>
Feedhenry-raincatcher mailing list<br>
<a href="mailto:Feedhenry-raincatcher@redhat.com">Feedhenry-raincatcher@redhat.<wbr>com</a><br>
<a href="https://www.redhat.com/mailman/listinfo/feedhenry-raincatcher" rel="noreferrer" target="_blank">https://www.redhat.com/<wbr>mailman/listinfo/feedhenry-<wbr>raincatcher</a><br>
<br></blockquote></div><br></div>