<html><head><meta http-equiv="Content-Type" content="text/html charset=us-ascii"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class="">Hi Kostas,<div class=""><br class=""></div><div class="">From my perspective, unit tests are not optional in our system. They are a standard practice for any developer today, regardless of whether they work in industry or research. The requirement to write unit tests for all functionality shouldn't be unusual or difficult for most developers. </div><div class=""><br class=""></div><div class="">The reason is that if you don't write unit tests, we will have no way to verify that the PMT is actually functioning as expected and that it continues to work when things change as we prepare of the review and beyond. A practical example of this is the recent breakage of the PMT with the "fireball branch." Had there been an automated suite of unit tests, we would have caught this regression weeks ago.</div><div class=""><br class=""></div><div class="">Equally, if there are cases where you are expecting Flow Manager-side support for the PMT, we need unit tests to ensure that anything we implement is correctly meeting the contract that you expect. We can't be responsible for a working integration in the absence of unit tests.</div><div class=""><br class=""></div><div class="">When it comes to the pressure of deadlines related to the review (which I totally empathize with) and your desire to reduce scope by not writing unit tests, I will need to defer to Manuel and Gregg. My advice is that not writing unit tests is a technical risk, and you will likely need to be responsible for a more rigorous manual testing procedure (i.e. testing by hand daily against the latest maser branch of the whole system) in order to ensure that everything continues to work.</div><div class=""><br class=""></div><div class="">Colin</div><div class=""><br class=""><div><blockquote type="cite" class=""><div class="">On Dec 4, 2014, at 9:54 AM, Kostas Kalogirou <<a href="mailto:kalogir@certh.gr" class="">kalogir@certh.gr</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><div class="WordSection1" style="page: WordSection1; font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;"><div style="margin: 0cm 0cm 0.0001pt; font-size: 11pt; font-family: Calibri, sans-serif;" class=""><span lang="EN-US" class="">Dear UXers and architecturers,<o:p class=""></o:p></span></div><div style="margin: 0cm 0cm 0.0001pt; font-size: 11pt; font-family: Calibri, sans-serif;" class=""><span lang="EN-US" class=""> </span></div><div style="margin: 0cm 0cm 0.0001pt; font-size: 11pt; font-family: Calibri, sans-serif;" class=""><span lang="EN-US" class="">I would like to ask you whether it is possible to<span class="Apple-converted-space"> </span><b class="">not</b><span class="Apple-converted-space"> </span>include unit tests for signup(<a href="http://issues.gpii.net/browse/GPII-947" style="color: purple; text-decoration: underline;" class="">http://issues.gpii.net/browse/GPII-947</a>) , auto-adjust adjuster(<a href="http://issues.gpii.net/browse/GPII-1023" style="color: purple; text-decoration: underline;" class="">http://issues.gpii.net/browse/GPII-1023</a><span class="Apple-converted-space"> </span>), mobile adjusters(<a href="http://issues.gpii.net/browse/GPII-803" style="color: purple; text-decoration: underline;" class="">http://issues.gpii.net/browse/GPII-803</a>)<span class="Apple-converted-space"> </span><o:p class=""></o:p></span></div><div style="margin: 0cm 0cm 0.0001pt; font-size: 11pt; font-family: Calibri, sans-serif;" class=""><span lang="EN-US" class="">and context wireframes(<a href="http://issues.gpii.net/browse/GPII-1018" style="color: purple; text-decoration: underline;" class="">http://issues.gpii.net/browse/GPII-1018</a>, GPII-1019, GPII-1020) for PMT  development.<o:p class=""></o:p></span></div><div style="margin: 0cm 0cm 0.0001pt; font-size: 11pt; font-family: Calibri, sans-serif;" class=""><span lang="EN-US" class=""> </span></div><div style="margin: 0cm 0cm 0.0001pt; font-size: 11pt; font-family: Calibri, sans-serif;" class=""><span lang="EN-US" class="">It seems that we are running out of time and we have to finish the development of all of the above JIRAs, including the<span class="Apple-converted-space"> </span><a href="http://issues.gpii.net/browse/GPII-1029" style="color: purple; text-decoration: underline;" class="">http://issues.gpii.net/browse/GPII-1029</a><span class="Apple-converted-space"> </span>(the new API provided by GPII server) and get ready for the E.C. review.<o:p class=""></o:p></span></div><div style="margin: 0cm 0cm 0.0001pt; font-size: 11pt; font-family: Calibri, sans-serif;" class=""><span lang="EN-US" class="">Besides, I think that no one from the reviewers will examine the code to see whether we have included unit tests.<o:p class=""></o:p></span></div><div style="margin: 0cm 0cm 0.0001pt; font-size: 11pt; font-family: Calibri, sans-serif;" class=""><span lang="EN-US" class=""> </span></div><div style="margin: 0cm 0cm 0.0001pt; font-size: 11pt; font-family: Calibri, sans-serif;" class=""><span lang="EN-US" class="">Regards,<o:p class=""></o:p></span></div><div style="margin: 0cm 0cm 0.0001pt; font-size: 11pt; font-family: Calibri, sans-serif;" class=""><span lang="EN-US" class="">Kostas<o:p class=""></o:p></span></div><div style="margin: 0cm 0cm 0.0001pt; font-size: 11pt; font-family: Calibri, sans-serif;" class=""><span lang="EN-US" class=""> </span></div><div style="margin: 0cm 0cm 0.0001pt; font-size: 11pt; font-family: Calibri, sans-serif;" class=""><span lang="EN-US" style="font-size: 10pt; font-family: Arial, sans-serif; color: navy;" class="">------------------------------------------------------------<o:p class=""></o:p></span></div><div style="margin: 0cm 0cm 0.0001pt; font-size: 11pt; font-family: Calibri, sans-serif;" class=""><span lang="EN-US" style="font-size: 10pt; font-family: Arial, sans-serif; color: navy;" class="">Konstantinos Kalogirou<o:p class=""></o:p></span></div><div style="margin: 0cm 0cm 0.0001pt; font-size: 11pt; font-family: Calibri, sans-serif;" class=""><span lang="EN-US" style="font-size: 10pt; font-family: Arial, sans-serif; color: navy;" class=""> </span></div><div style="margin: 0cm 0cm 0.0001pt; font-size: 11pt; font-family: Calibri, sans-serif;" class=""><span lang="EN-US" style="font-size: 10pt; font-family: Arial, sans-serif; color: navy;" class="">Software Engineering/MSc at Digital Signal Processing</span><span lang="EN-US" style="font-size: 12pt; font-family: 'Times New Roman', serif; color: rgb(31, 73, 125);" class=""><o:p class=""></o:p></span></div><div style="margin: 0cm 0cm 0.0001pt; font-size: 11pt; font-family: Calibri, sans-serif;" class=""><span lang="EN-US" style="font-size: 10pt; font-family: Arial, sans-serif; color: navy;" class="">Centre for Research & Technology Hellas(CE.R.T.H.)/<o:p class=""></o:p></span></div><div style="margin: 0cm 0cm 0.0001pt; font-size: 11pt; font-family: Calibri, sans-serif;" class=""><span lang="EN-US" style="font-size: 10pt; font-family: Arial, sans-serif; color: navy;" class="">Hellenic Institute of Transport (H.I.T.)</span><span lang="EN-US" style="color: rgb(31, 73, 125);" class=""><o:p class=""></o:p></span></div><div style="margin: 0cm 0cm 0.0001pt; font-size: 11pt; font-family: Calibri, sans-serif;" class=""><span lang="EN-US" style="font-size: 10pt; font-family: Arial, sans-serif; color: navy;" class="">6th km Harilaou-Thermi Road,</span><span lang="EN-US" style="color: rgb(31, 73, 125);" class=""><o:p class=""></o:p></span></div><div style="margin: 0cm 0cm 0.0001pt; font-size: 11pt; font-family: Calibri, sans-serif;" class=""><span lang="EN-US" style="font-size: 10pt; font-family: Arial, sans-serif; color: navy;" class="">P.O. Box 60361, 57001 Thermi, Macedonia,</span><span lang="EN-US" style="color: rgb(31, 73, 125);" class=""><o:p class=""></o:p></span></div><div style="margin: 0cm 0cm 0.0001pt; font-size: 11pt; font-family: Calibri, sans-serif;" class=""><span lang="EN-US" style="font-size: 10pt; font-family: Arial, sans-serif; color: navy;" class="">GREECE<o:p class=""></o:p></span></div><div style="margin: 0cm 0cm 0.0001pt; font-size: 11pt; font-family: Calibri, sans-serif;" class=""><span lang="EN-US" style="font-size: 10pt; font-family: Arial, sans-serif; color: navy;" class=""> </span></div><div style="margin: 0cm 0cm 0.0001pt; font-size: 11pt; font-family: Calibri, sans-serif;" class=""><span lang="EN-US" style="font-size: 10pt; font-family: Arial, sans-serif; color: navy;" class="">Tel: +30 2310 498461</span><span lang="EN-US" style="color: rgb(31, 73, 125);" class=""><o:p class=""></o:p></span></div><div style="margin: 0cm 0cm 0.0001pt; font-size: 11pt; font-family: Calibri, sans-serif;" class=""><span lang="EN-US" style="font-size: 10pt; font-family: Arial, sans-serif; color: navy;" class="">Fax:+30 2310 498269</span><span lang="EN-US" style="color: rgb(31, 73, 125);" class=""><o:p class=""></o:p></span></div><div style="margin: 0cm 0cm 0.0001pt; font-size: 11pt; font-family: Calibri, sans-serif;" class=""><span style="font-size: 10pt; font-family: Arial, sans-serif; color: navy;" class=""><a href="http://www.certh.gr/" style="color: purple; text-decoration: underline;" class=""><span lang="EN-US" style="color: blue;" class="">www.imet.gr</span></a></span><span lang="EN-US" style="font-size: 10pt; font-family: Arial, sans-serif; color: navy;" class=""><o:p class=""></o:p></span></div><div style="margin: 0cm 0cm 0.0001pt; font-size: 11pt; font-family: Calibri, sans-serif;" class=""><span style="font-size: 10pt; font-family: Arial, sans-serif; color: navy;" class=""><a href="http://www.certh.gr/" style="color: purple; text-decoration: underline;" class=""><span lang="EN-US" style="color: blue;" class="">www.certh.gr</span></a></span><span lang="EN-US" style="color: rgb(31, 73, 125);" class=""><o:p class=""></o:p></span></div><div style="margin: 0cm 0cm 0.0001pt; font-size: 11pt; font-family: Calibri, sans-serif;" class=""><o:p class=""> </o:p></div></div><span style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px; float: none; display: inline !important;" class="">_______________________________________________</span><br style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class=""><span style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px; float: none; display: inline !important;" class="">Ux mailing list</span><br style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class=""><a href="mailto:Ux@lists.gpii.net" style="color: purple; text-decoration: underline; font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class="">Ux@lists.gpii.net</a><br style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class=""><a href="http://lists.gpii.net/cgi-bin/mailman/listinfo/ux" style="color: purple; text-decoration: underline; font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class="">http://lists.gpii.net/cgi-bin/mailman/listinfo/ux</a></div></blockquote></div><br class=""></div></body></html>