John  Mertic   Rate  this  talk  at  https://joind.in/6293   

¡  John  Mertic   §  Contact  Info   ▪  http:// jmertic.wordpress.com   ▪  Twitter:  @jmertic   §  Community  Manager  for   SugarCRM   ▪  http://www.sugarcrm.com   ▪  http://bit.ly/SugarDevBlog   ▪  Twitter:  @sugarcrmdev   ▪  [email protected]     

No content

No content

No content

No content

THE  GOOD  PART   ¡  Code  is  established,  known   working   ¡  Functionality  is  well  vetted   by  the  stakeholders   ¡  End  users  are  well  trained   and  versed  it  using  it   AND  THE  BAD   ¡  Code  can  be  very   “patchwork”  like,  many   different  hands  over  time   ¡  Functionality  may  not   match  today’s  needs   ¡  End  users  work  with  the   system  versus  the  system   working  with  them   

No content

No content

No content

No content

No content

No content

No content

No content

No content

FUNCTIONAL   ¡  Tests  the  code  indirectly     from  the  user  perspective   ¡  Tests  how  several   components  work  together   ¡  Can  be  easier  to  get  going   on  a  legacy  codebase   UNIT   ¡  Tests  the  code  directly   from  the  developer   perspective   ¡  Tests  individual   components  specifically   ¡  Tends  to  be  more   challenging  to  get  going  on   a  legacy  codebase   

¡  Procedural  code   ¡  Poorly  designed  objects   §  Static  dependencies   §  Busy  constructors   §  Really  looooooooooong  methods   §  A  bizarre  affection  for  the  global  scope   

No content

No content

No content

No content

¡  Procedural  code;  weigh  the  viability  of  a  unit   versus  functional  test   ¡  Build  helper  functions/classes/tools  to  do   repetitive  work  for  you.   ¡  Make  a  practice  of  resetting  the  global  scope   §  PHPUnit  can  do  this  for  you   ¡  Keep  focused  on  the  primary  goal  of  your   tests.   

id; return $user_ids; } } 

retrieve('1'); } function tearDown() { } ---CLIPPED--- } 

should_process) { return ''; } $str = ''; if ($this->multiSelect == true && $this->show_mass_update_form) { $str = $this->mass->getDisplayMassUpdateForm(true, $this->multi_select_popup). $this->mass->getMassUpdateFormHeader($this->multi_select_popup); } return $str; } --clipped code section— } 

_lvd = new ListViewDisplay; } public function testDisplayIfShowMassUpdateFormTrueAndMultiSelectTrue() { $this->_lvd->should_process = true; $this->_lvd->show_mass_update_form = true; $this->_lvd->multiSelect = true; $this->_lvd->multi_select_popup = true; $this->_lvd->mass = $this->getMock('MassUpdate'); $this->_lvd->mass->expects($this->any()) ->method('getDisplayMassUpdateForm') ->will($this->returnValue('foo')); $this->_lvd->mass->expects($this->any()) ->method('getMassUpdateFormHeader') ->will($this->returnValue('bar')); $this->assertEquals('foobar',$this->_lvd->display()); } } 

No content

¡  Write  a  unit  test  as  part  of  each  bug  you  fix.   ¡  Learn  lessons  from  each  test  you  write   §  How  difficult  was  it  to  isolate  the  issue?   §  How  much  scaffolding  had  to  be  done  to  do  the   tests?   §  What  tools/techniques  would  make  testing  easier?   

¡  Find  ways  to  make  writing  the  tests  easier   §  Add  helpers  or  toolkits  to  use   §  Build  testing  templates   §  Build  example  tests  people  can  copy  from.   ¡  Expand  testing  to  full  components   §  Idea:  create  a  policy  that  if  you  add  a  unit  test  for   a  function/method,  you  should  make  sure  you  add   full  coverage  for  the  function/method  and  not  just   for  the  bug  itself   

No content

¡  It  takes  too  much  time   ¡  It’s  too  hard  to  test  a   certain  piece  of  code   ¡  I’ll  never  get  100%   code  coverage,  so  why   bother   ¡  I  can’t  justify  the  cost/ value  of  it   ¡  So  does  fixing  the   same  bug  over  and   over   ¡  Develop  a  better   strategy  for  testing   ¡  Nobody  has  100%  code   coverage   ¡  Adding  a  test  now   helps  prevent  the   problem  from  re-­‐ occurring   

No content

assertEqual($this- >_mod_strings[$label_name], $this- >_mod_strings['LBL_CURRENCY']); } } 

¡  Are  the  right  type  (  unit  vs  functional  )   ¡  Isolate  and  cover  the  issue   ¡  Fail  if  the  fix  isn’t  in  place;  pass  if  it  is   ¡  Sensible  to  debug   

¡  Elegant   ¡  High  performance   ¡  Hold  to  some  ideology   

No content

No content

No content

No content

¡  Things  change   ¡  New  shiny  things  make  things  easier   ¡  Pressure   ¡  Cluelessness   

¡  Learn  from  mistakes   ¡  Embrace  pragmatism   ¡  Don’t  be  ignorant   ¡  Ask  for  help!   

Rate  this  talk  at  https://joind.in/6293