Upgrade to Pro — share decks privately, control downloads, hide ads and more …

Magic Tricks of Testing (AncientCityRuby)

Magic Tricks of Testing (AncientCityRuby)

Tests are supposed to save us money. How is it, then, that many times they become millstones around our necks, gradually morphing into fragile, breakable things that raise the cost of change?

We write too many tests and we test the wrong kinds of things. This talk strips away the veil and offers simple, practical guidelines for choosing what to test and how to test it. Finding the right testing balance isn’t magic, it’s a magic trick; come and learn the secret of writing stable tests that protect your application at the lowest possible cost.

Sandi Metz

April 05, 2013
Tweet

More Decks by Sandi Metz

Other Decks in Programming

Transcript

  1. @sandimetz Apr 2013 Query: Returns something but changes nothing. Command:

    Returns nothing but changes something. Type Friday, April 5, 13
  2. @sandimetz Apr 2013 Query: Returns something but changes nothing. Command:

    Returns nothing but changes something. Type Friday, April 5, 13
  3. @sandimetz Apr 2013 Query: Returns something but changes nothing. Command:

    Returns nothing but changes something. Type Friday, April 5, 13
  4. Query Command Message Origin x Type Outgoing Incoming Type POV

    @sandimetz Apr 2013 Message Sent to Self Friday, April 5, 13
  5. @sandimetz GoGaRuCo 2012 @sandimetz Apr 2013 to Self Outgoing Incoming

    Incoming Query Messages Incoming Friday, April 5, 13
  6. @sandimetz GoGaRuCo 2012 @sandimetz Apr 2013 to Self Outgoing Incoming

    class  Wheel    attr_reader  :rim,  :tire    def  initialize(rim,  tire)    #  ...    end    def  diameter        rim  +  (tire  *  2)    end    #  ... Friday, April 5, 13
  7. @sandimetz GoGaRuCo 2012 @sandimetz Apr 2013 to Self Outgoing Incoming

    class  Wheel    attr_reader  :rim,  :tire    def  initialize(rim,  tire)    #  ...    end    def  diameter        rim  +  (tire  *  2)    end    #  ... Friday, April 5, 13
  8. @sandimetz GoGaRuCo 2012 @sandimetz Apr 2013 to Self Outgoing Incoming

    class  Wheel    attr_reader  :rim,  :tire    def  initialize(rim,  tire)    #  ...    end    def  diameter        rim  +  (tire  *  2)    end    #  ... Friday, April 5, 13
  9. @sandimetz GoGaRuCo 2012 @sandimetz Apr 2013 to Self Outgoing Incoming

    class  WheelTest  <  MiniTest::Unit::TestCase    def  test_calculates_diameter        wheel  =  Wheel.new(26,  1.5)        assert_in_delta(29,                                        wheel.diameter,                                        0.01)    end end Friday, April 5, 13
  10. @sandimetz GoGaRuCo 2012 @sandimetz Apr 2013 to Self Outgoing Incoming

    class  WheelTest  <  MiniTest::Unit::TestCase    def  test_calculates_diameter        wheel  =  Wheel.new(26,  1.5)        assert_in_delta(29,                                        wheel.diameter,                                        0.01)    end end Friday, April 5, 13
  11. @sandimetz GoGaRuCo 2012 @sandimetz Apr 2013 to Self Outgoing Incoming

    class  WheelTest  <  MiniTest::Unit::TestCase    def  test_calculates_diameter        wheel  =  Wheel.new(26,  1.5)        assert_in_delta(29,                                        wheel.diameter,                                        0.01)    end end Friday, April 5, 13
  12. @sandimetz Apr 2013 Test incoming query messages by making assertions

    about what they send back. Rule Friday, April 5, 13
  13. Query Command The Unit Testing Minimalist Incoming Type POV @sandimetz

    Apr 2013 Message Sent to Self Outgoing Friday, April 5, 13
  14. Query Command Assert result The Unit Testing Minimalist Incoming Type

    POV @sandimetz Apr 2013 Message Sent to Self Outgoing Friday, April 5, 13
  15. @sandimetz GoGaRuCo 2012 @sandimetz Apr 2013 to Self Outgoing Incoming

    Incoming Another Incoming Query Message Friday, April 5, 13
  16. @sandimetz GoGaRuCo 2012 @sandimetz Apr 2013 to Self Outgoing Incoming

    Gear gear_inches Incoming Friday, April 5, 13
  17. @sandimetz GoGaRuCo 2012 @sandimetz Apr 2013 to Self Outgoing Incoming

    class  Gear    attr_reader  :chainring,  :cog,  :wheel    def  initialize(args)        #  ...    end      #  ...    def  gear_inches        ratio  *  wheel.diameter    end      private    def  ratio        chainring  /  cog.to_f    end      #  ... end Friday, April 5, 13
  18. @sandimetz GoGaRuCo 2012 @sandimetz Apr 2013 to Self Outgoing Incoming

    class  Gear    attr_reader  :chainring,  :cog,  :wheel    def  initialize(args)        #  ...    end      #  ...    def  gear_inches        ratio  *  wheel.diameter    end      private    def  ratio        chainring  /  cog.to_f    end      #  ... end Friday, April 5, 13
  19. @sandimetz GoGaRuCo 2012 @sandimetz Apr 2013 to Self Outgoing Incoming

    class  Gear    attr_reader  :chainring,  :cog,  :wheel    def  initialize(args)        #  ...    end      #  ...    def  gear_inches        ratio  *  wheel.diameter    end      private    def  ratio        chainring  /  cog.to_f    end      #  ... end Friday, April 5, 13
  20. @sandimetz GoGaRuCo 2012 @sandimetz Apr 2013 to Self Outgoing Incoming

    class  Gear    attr_reader  :chainring,  :cog,  :wheel    def  initialize(args)        #  ...    end      #  ...    def  gear_inches        ratio  *  wheel.diameter    end      private    def  ratio        chainring  /  cog.to_f    end      #  ... end Friday, April 5, 13
  21. @sandimetz GoGaRuCo 2012 @sandimetz Apr 2013 to Self Outgoing Incoming

    class  Gear    attr_reader  :chainring,  :cog,  :wheel    def  initialize(args)        #  ...    end      #  ...    def  gear_inches        ratio  *  wheel.diameter    end      private    def  ratio        chainring  /  cog.to_f    end      #  ... end Wheel Friday, April 5, 13
  22. @sandimetz GoGaRuCo 2012 @sandimetz Apr 2013 to Self Outgoing Incoming

    class  GearTest  <  MiniTest::Unit::TestCase    def  test_calculates_gear_inches        gear  =    Gear.new(                            chainring:  52,                            cog:              11,                            wheel:          Wheel.new(26,  1.5))        assert_in_delta(137.1,                                        gear.gear_inches,                                        0.01)    end end Friday, April 5, 13
  23. @sandimetz GoGaRuCo 2012 @sandimetz Apr 2013 to Self Outgoing Incoming

    class  GearTest  <  MiniTest::Unit::TestCase    def  test_calculates_gear_inches        gear  =    Gear.new(                            chainring:  52,                            cog:              11,                            wheel:          Wheel.new(26,  1.5))        assert_in_delta(137.1,                                        gear.gear_inches,                                        0.01)    end end Friday, April 5, 13
  24. @sandimetz GoGaRuCo 2012 @sandimetz Apr 2013 to Self Outgoing Incoming

    class  GearTest  <  MiniTest::Unit::TestCase    def  test_calculates_gear_inches        gear  =    Gear.new(                            chainring:  52,                            cog:              11,                            wheel:          Wheel.new(26,  1.5))        assert_in_delta(137.1,                                        gear.gear_inches,                                        0.01)    end end Friday, April 5, 13
  25. @sandimetz GoGaRuCo 2012 @sandimetz Apr 2013 to Self Outgoing Incoming

    Incoming Command Messages Incoming Friday, April 5, 13
  26. @sandimetz GoGaRuCo 2012 @sandimetz Apr 2013 to Self Outgoing Incoming

    class  Gear  attr_reader  :chainring,  :cog,  :wheel    def  initialize(args)        #  ...    end    def  set_cog(new_cog)        @cog  =  new_cog    end end Friday, April 5, 13
  27. @sandimetz GoGaRuCo 2012 @sandimetz Apr 2013 to Self Outgoing Incoming

    class  Gear  attr_reader  :chainring,  :cog,  :wheel    def  initialize(args)        #  ...    end    def  set_cog(new_cog)        @cog  =  new_cog    end end Friday, April 5, 13
  28. @sandimetz GoGaRuCo 2012 @sandimetz Apr 2013 to Self Outgoing Incoming

    class  Gear  attr_reader  :chainring,  :cog,  :wheel    def  initialize(args)        #  ...    end    def  set_cog(new_cog)        @cog  =  new_cog    end end Friday, April 5, 13
  29. @sandimetz GoGaRuCo 2012 @sandimetz Apr 2013 to Self Outgoing Incoming

    class  GearTest  <  MiniTest::Unit::TestCase    def  test_set_cog        gear  =  Gear.new        gear.set_cog(27)        assert(27,  gear.cog)    end end Friday, April 5, 13
  30. @sandimetz GoGaRuCo 2012 @sandimetz Apr 2013 to Self Outgoing Incoming

    class  GearTest  <  MiniTest::Unit::TestCase    def  test_set_cog        gear  =  Gear.new        gear.set_cog(27)        assert(27,  gear.cog)    end end Friday, April 5, 13
  31. @sandimetz GoGaRuCo 2012 @sandimetz Apr 2013 to Self Outgoing Incoming

    class  GearTest  <  MiniTest::Unit::TestCase    def  test_set_cog        gear  =  Gear.new        gear.set_cog(27)        assert(27,  gear.cog)    end end Send the message Friday, April 5, 13
  32. @sandimetz GoGaRuCo 2012 @sandimetz Apr 2013 to Self Outgoing Incoming

    class  GearTest  <  MiniTest::Unit::TestCase    def  test_set_cog        gear  =  Gear.new        gear.set_cog(27)        assert(27,  gear.cog)    end end Assert the side effect Friday, April 5, 13
  33. @sandimetz Apr 2013 Test incoming command messages by making assertions

    about direct public side effects. Rule Friday, April 5, 13
  34. Query Command Assert result The Unit Testing Minimalist Incoming Type

    POV @sandimetz Apr 2013 Message Sent to Self Outgoing Friday, April 5, 13
  35. Query Command Assert result The Unit Testing Minimalist Incoming Type

    POV @sandimetz Apr 2013 Message Assert direct public side effects Sent to Self Outgoing Friday, April 5, 13
  36. @sandimetz Apr 2013 Receiver of incoming message has sole responsibility

    for asserting the result direct public side effects. DRY It Out Friday, April 5, 13
  37. @sandimetz GoGaRuCo 2012 @sandimetz Apr 2013 to Self Outgoing Incoming

    to Self Incoming Messages Sent to Self Friday, April 5, 13
  38. @sandimetz GoGaRuCo 2012 @sandimetz Apr 2013 to Self Outgoing Incoming

    class  Gear      #  ...    def  gear_inches        ratio  *  wheel.diameter    end      private    def  ratio        chainring  /  cog.to_f    end      #  ... end Friday, April 5, 13
  39. @sandimetz GoGaRuCo 2012 @sandimetz Apr 2013 to Self Outgoing Incoming

    Anti-Pattern to Self Incoming Friday, April 5, 13
  40. @sandimetz GoGaRuCo 2012 @sandimetz Apr 2013 to Self Outgoing Incoming

    class  GearTest  <  MiniTest::Unit::TestCase    def  test_calculates_ratio        gear  =    Gear.new(                            chainring:  52,                            cog:              11,                            wheel:          Wheel.new(26,  1.5))        assert_in_delta(4.7,                                        gear.ratio,                                        0.01)    end Friday, April 5, 13
  41. @sandimetz GoGaRuCo 2012 @sandimetz Apr 2013 to Self Outgoing Incoming

    class  GearTest  <  MiniTest::Unit::TestCase    def  test_calculates_ratio        gear  =    Gear.new(                            chainring:  52,                            cog:              11,                            wheel:          Wheel.new(26,  1.5))        assert_in_delta(4.7,                                        gear.ratio,                                        0.01)    end Test the private method? Friday, April 5, 13
  42. @sandimetz GoGaRuCo 2012 @sandimetz Apr 2013 to Self Outgoing Incoming

    class  GearTest  <  MiniTest::Unit::TestCase    def  test_calculates_ratio        gear  =    Gear.new(                            chainring:  52,                            cog:              11,                            wheel:          Wheel.new(26,  1.5))        assert_in_delta(4.7,                                        gear.ratio,                                        0.01)    end Friday, April 5, 13
  43. @sandimetz GoGaRuCo 2012 @sandimetz Apr 2013 to Self Outgoing Incoming

    class  GearTest  <  MiniTest::Unit::TestCase    def  test_calculates_ratio        gear  =    Gear.new(                            chainring:  52,                            cog:              11,                            wheel:          Wheel.new(26,  1.5))        assert_in_delta(4.7,                                        gear.ratio,                                        0.01)    end Friday, April 5, 13
  44. @sandimetz GoGaRuCo 2012 @sandimetz Apr 2013 to Self Outgoing Incoming

    class  GearTest  <  MiniTest::Unit::TestCase    def  test_calculates_ratio        gear  =    Gear.new(                            chainring:  52,                            cog:              11,                            wheel:          Wheel.new(26,  1.5))        assert_in_delta(4.7,                                        gear.ratio,                                        0.01)    end Redundant: Other tests already prove Gear works. Friday, April 5, 13
  45. @sandimetz GoGaRuCo 2012 @sandimetz Apr 2013 to Self Outgoing Incoming

    Anti-Pattern to Self Incoming Friday, April 5, 13
  46. @sandimetz GoGaRuCo 2012 @sandimetz Apr 2013 to Self Outgoing Incoming

    class  Gear      #  ...    def  gear_inches        ratio  *  wheel.diameter    end      private    def  ratio        chainring  /  cog.to_f    end      #  ... end Friday, April 5, 13
  47. @sandimetz GoGaRuCo 2012 @sandimetz Apr 2013 to Self Outgoing Incoming

    class  Gear      #  ...    def  gear_inches        ratio  *  wheel.diameter    end      private    def  ratio        chainring  /  cog.to_f    end      #  ... end Friday, April 5, 13
  48. @sandimetz GoGaRuCo 2012 @sandimetz Apr 2013 to Self Outgoing Incoming

    class  Gear      #  ...    def  gear_inches        ratio  *  wheel.diameter    end      private    def  ratio        chainring  /  cog.to_f    end      #  ... end Friday, April 5, 13
  49. @sandimetz GoGaRuCo 2012 @sandimetz Apr 2013 to Self Outgoing Incoming

    class  GearTest  <  MiniTest::Unit::TestCase    def  test_calculates_gear_inches        gear  =    Gear.new(                            chainring:  52,                            cog:              11,                            wheel:          Wheel.new(26,  1.5))        assert_in_delta(137.1,                                        gear.gear_inches,                                        0.01)    end end Friday, April 5, 13
  50. @sandimetz GoGaRuCo 2012 @sandimetz Apr 2013 to Self Outgoing Incoming

    class  GearTest  <  MiniTest::Unit::TestCase    def  test_calculates_gear_inches        gear  =    Gear.new(                            chainring:  52,                            cog:              11,                            wheel:          Wheel.new(26,  1.5))        assert_in_delta(137.1,                                        gear.gear_inches,                                        0.01)        gear.expect(:ratio)        gear.verify    end Friday, April 5, 13
  51. @sandimetz GoGaRuCo 2012 @sandimetz Apr 2013 to Self Outgoing Incoming

    class  GearTest  <  MiniTest::Unit::TestCase    def  test_calculates_gear_inches        gear  =    Gear.new(                            chainring:  52,                            cog:              11,                            wheel:          Wheel.new(26,  1.5))        assert_in_delta(137.1,                                        gear.gear_inches,                                        0.01)        gear.expect(:ratio)        gear.verify    end Expect to send the private method? Friday, April 5, 13
  52. @sandimetz GoGaRuCo 2012 @sandimetz Apr 2013 to Self Outgoing Incoming

    class  GearTest  <  MiniTest::Unit::TestCase    def  test_calculates_gear_inches        gear  =    Gear.new(                            chainring:  52,                            cog:              11,                            wheel:          Wheel.new(26,  1.5))        assert_in_delta(137.1,                                        gear.gear_inches,                                        0.01)        gear.expect(:ratio)        gear.verify    end Over Speci ed: Adds no safety yet breaks every time the implementation changes. Friday, April 5, 13
  53. @sandimetz Apr 2013 Do not test private methods. Do not

    make assertions about their result. Rule Friday, April 5, 13
  54. @sandimetz Apr 2013 Do not test private methods. Do not

    make assertions about their result. Do not expect to send them. Rule Friday, April 5, 13
  55. @sandimetz Apr 2013 Temporarily breaking this rule can save $$$

    during development. Caveat Friday, April 5, 13
  56. Query Command Assert result Assert direct public side-effects The Unit

    Testing Minimalist Incoming Type POV @sandimetz Apr 2013 Message Sent to Self Outgoing Friday, April 5, 13
  57. Query Command Assert result Assert direct public side-effects The Unit

    Testing Minimalist Incoming Type POV @sandimetz Apr 2013 Ignore Message Sent to Self Outgoing Friday, April 5, 13
  58. @sandimetz GoGaRuCo 2012 @sandimetz Apr 2013 to Self Outgoing Incoming

    Part 3 Outgoing to Self Incoming Friday, April 5, 13
  59. @sandimetz GoGaRuCo 2012 @sandimetz Apr 2013 to Self Outgoing Incoming

    Outgoing to Self Incoming Outgoing Query Messages Friday, April 5, 13
  60. @sandimetz GoGaRuCo 2012 @sandimetz Apr 2013 to Self Outgoing Incoming

    Outgoing to Self Incoming Spoiler Alert Same rules as ‘Sent to Self’ Friday, April 5, 13
  61. @sandimetz GoGaRuCo 2012 @sandimetz Apr 2013 to Self Outgoing Incoming

    diameter Gear gear_inches Wheel Outgoing to Self Incoming Friday, April 5, 13
  62. @sandimetz GoGaRuCo 2012 @sandimetz Apr 2013 to Self Outgoing Incoming

    class  Gear    attr_reader  :chainring,  :cog,  :wheel    def  initialize(args)          #  ...    end    def  gear_inches        ratio  *  wheel.diameter    end      private    def  ratio        chainring  /  cog.to_f    end      #  ... end Outgoing to Self Incoming Friday, April 5, 13
  63. @sandimetz GoGaRuCo 2012 @sandimetz Apr 2013 to Self Outgoing Incoming

    class  Gear    attr_reader  :chainring,  :cog,  :wheel    def  initialize(args)          #  ...    end    def  gear_inches        ratio  *  wheel.diameter    end      private    def  ratio        chainring  /  cog.to_f    end      #  ... end Outgoing to Self Incoming Wheel Friday, April 5, 13
  64. @sandimetz GoGaRuCo 2012 @sandimetz Apr 2013 to Self Outgoing Incoming

    Gear Gear Outgoing Wheel diameter Incoming Query Outgoing to Self Incoming gear_inches Friday, April 5, 13
  65. @sandimetz GoGaRuCo 2012 @sandimetz Apr 2013 to Self Outgoing Incoming

    Anti-Pattern to Self Incoming Outgoing to Self Incoming Friday, April 5, 13
  66. @sandimetz GoGaRuCo 2012 @sandimetz Apr 2013 to Self Incoming Outgoing

    class  GearTest  <  MiniTest::Unit::TestCase    def  test_calculates_gear_inches        gear  =    Gear.new(                            chainring:  52,                            cog:              11,                            wheel:          Wheel.new(26,  1.5))        assert_in_delta(137.1,                                        gear.gear_inches,                                        0.01)    end end Friday, April 5, 13
  67. @sandimetz GoGaRuCo 2012 @sandimetz Apr 2013 to Self Incoming Outgoing

    class  GearTest  <  MiniTest::Unit::TestCase    def  test_calculates_gear_inches        gear  =    Gear.new(                            chainring:  52,                            cog:              11,                            wheel:          Wheel.new(26,  1.5))        assert_in_delta(137.1,                                        gear.gear_inches,                                        0.01)          assert_in_delta(29,                                        gear.wheel.diameter,                                        0.01)    end end Friday, April 5, 13
  68. @sandimetz GoGaRuCo 2012 @sandimetz Apr 2013 to Self Incoming Outgoing

    class  GearTest  <  MiniTest::Unit::TestCase    def  test_calculates_gear_inches        gear  =    Gear.new(                            chainring:  52,                            cog:              11,                            wheel:          Wheel.new(26,  1.5))        assert_in_delta(137.1,                                        gear.gear_inches,                                        0.01)          assert_in_delta(29,                                        gear.wheel.diameter,                                        0.01)    end end Assert result of outgoing query? Friday, April 5, 13
  69. @sandimetz GoGaRuCo 2012 @sandimetz Apr 2013 to Self Incoming Outgoing

    class  GearTest  <  MiniTest::Unit::TestCase    def  test_calculates_gear_inches        gear  =    Gear.new(                            chainring:  52,                            cog:              11,                            wheel:          Wheel.new(26,  1.5))        assert_in_delta(137.1,                                        gear.gear_inches,                                        0.01)          assert_in_delta(29,                                        gear.wheel.diameter,                                        0.01)    end end Redundant: This duplicates Wheel’s tests. Friday, April 5, 13
  70. @sandimetz GoGaRuCo 2012 @sandimetz Apr 2013 to Self Outgoing Incoming

    Anti-Pattern to Self Incoming Outgoing to Self Incoming Friday, April 5, 13
  71. @sandimetz GoGaRuCo 2012 @sandimetz Apr 2013 to Self Incoming Outgoing

    class  GearTest  <  MiniTest::Unit::TestCase    def  test_calculates_gear_inches        gear  =    Gear.new(                            chainring:  52,                            cog:              11,                            wheel:          Wheel.new(26,  1.5))        assert_in_delta(137.1,                                        gear.gear_inches,                                        0.01)          gear.wheel.expect(:diameter)        gear.verify    end end Friday, April 5, 13
  72. @sandimetz GoGaRuCo 2012 @sandimetz Apr 2013 to Self Incoming Outgoing

    class  GearTest  <  MiniTest::Unit::TestCase    def  test_calculates_gear_inches        gear  =    Gear.new(                            chainring:  52,                            cog:              11,                            wheel:          Wheel.new(26,  1.5))        assert_in_delta(137.1,                                        gear.gear_inches,                                        0.01)          gear.wheel.expect(:diameter)        gear.verify    end end Expect to send outgoing query? Friday, April 5, 13
  73. @sandimetz GoGaRuCo 2012 @sandimetz Apr 2013 to Self Incoming Outgoing

    class  GearTest  <  MiniTest::Unit::TestCase    def  test_calculates_gear_inches        gear  =    Gear.new(                            chainring:  52,                            cog:              11,                            wheel:          Wheel.new(26,  1.5))        assert_in_delta(137.1,                                        gear.gear_inches,                                        0.01)          gear.wheel.expect(:diameter)        gear.verify    end end Over Speci ed: Adds no safety yet breaks every time the implementation changes. Friday, April 5, 13
  74. @sandimetz Apr 2013 Rule Do not test outgoing query messages.

    Do not make assertions about their result. Friday, April 5, 13
  75. @sandimetz Apr 2013 Rule Do not test outgoing query messages.

    Do not make assertions about their result. Do not expect to send them. Friday, April 5, 13
  76. @sandimetz Apr 2013 If the message has no visible side-effects

    the sender should not test it. Friday, April 5, 13
  77. Query Command Assert result Assert direct public side effects The

    Unit Testing Minimalist Incoming Type POV @sandimetz Apr 2013 Message Ignore Sent to Self Outgoing Friday, April 5, 13
  78. Query Command Assert result Assert direct public side effects The

    Unit Testing Minimalist Incoming Type POV @sandimetz Apr 2013 Message Ignore Sent to Self Outgoing Friday, April 5, 13
  79. @sandimetz GoGaRuCo 2012 @sandimetz Apr 2013 to Self Outgoing Incoming

    Outgoing Command Messages to Self Incoming Outgoing Friday, April 5, 13
  80. @sandimetz GoGaRuCo 2012 @sandimetz Apr 2013 to Self Outgoing Incoming

    diameter gear_inches set_cog changed to Self Incoming Outgoing Wheel Gear Obs Friday, April 5, 13
  81. @sandimetz GoGaRuCo 2012 @sandimetz Apr 2013 to Self Incoming Outgoing

    class  Gear    attr_reader  :chainring,  :cog,  :wheel    def  initialize(args)              #  ...      end    def  set_cog(new_cog)        @cog  =  new_cog    end end Change #set_cog to add observer. Friday, April 5, 13
  82. @sandimetz GoGaRuCo 2012 @sandimetz Apr 2013 to Self Incoming Outgoing

    class  Gear    attr_reader  :chainring,  :cog,  :wheel    def  initialize(args)              #  ...      end    def  set_cog(new_cog)        @cog  =  new_cog    end end Friday, April 5, 13
  83. @sandimetz GoGaRuCo 2012 @sandimetz Apr 2013 to Self Incoming Outgoing

    class  Gear    attr_reader  :chainring,  :cog,  :wheel,  :observer    def  initialize(args)              #  ...        @observer    =  args[:observer]    end    def  set_cog(new_cog)        @cog  =  new_cog    end end Friday, April 5, 13
  84. @sandimetz GoGaRuCo 2012 @sandimetz Apr 2013 to Self Incoming Outgoing

    class  Gear      #  ...    def  set_cog(new_cog)        @cog  =  new_cog    end end Friday, April 5, 13
  85. @sandimetz GoGaRuCo 2012 @sandimetz Apr 2013 to Self Incoming Outgoing

    class  Gear      #  ...    def  set_cog(new_cog)        @cog  =  new_cog    end end Friday, April 5, 13
  86. @sandimetz GoGaRuCo 2012 @sandimetz Apr 2013 to Self Incoming Outgoing

    class  Gear      #  ...    def  set_cog(new_cog)        @cog  =  new_cog        changed        @cog    end    def  changed        observer.changed(chainring,  cog)    end end Friday, April 5, 13
  87. @sandimetz GoGaRuCo 2012 @sandimetz Apr 2013 to Self Incoming Outgoing

    class  Gear      #  ...    def  set_cog(new_cog)        @cog  =  new_cog        changed        @cog    end    def  changed        observer.changed(chainring,  cog)    end end Friday, April 5, 13
  88. @sandimetz GoGaRuCo 2012 @sandimetz Apr 2013 to Self Outgoing Incoming

    Gear set_cog changed Obs to Self Incoming Outgoing Friday, April 5, 13
  89. @sandimetz GoGaRuCo 2012 @sandimetz Apr 2013 to Self Outgoing Incoming

    Gear set_cog changed Obs to Self Incoming Outgoing Side Effects Friday, April 5, 13
  90. @sandimetz GoGaRuCo 2012 @sandimetz Apr 2013 to Self Outgoing Incoming

    changed Gear set_cog Obs Incoming Command to Self Incoming Outgoing Side Effects Friday, April 5, 13
  91. @sandimetz GoGaRuCo 2012 @sandimetz Apr 2013 to Self Outgoing Incoming

    Gear Outgoing set_cog Obs changed Incoming Command to Self Incoming Outgoing Side Effects Friday, April 5, 13
  92. @sandimetz GoGaRuCo 2012 @sandimetz Apr 2013 to Self Incoming Outgoing

    class  GearTest  <  MiniTest::Unit::TestCase    def  test_set_cog        gear  =  Gear.new        gear.set_cog(27)        assert(27,  gear.cog)    end end Make existing #set_cog test run. Friday, April 5, 13
  93. @sandimetz GoGaRuCo 2012 @sandimetz Apr 2013 to Self Incoming Outgoing

    class  GearTest  <  MiniTest::Unit::TestCase    def  test_set_cog        gear  =  Gear.new                                                                                                                    @gear.set_cog(27)        assert(27,  @gear.cog)    end end Friday, April 5, 13
  94. @sandimetz GoGaRuCo 2012 @sandimetz Apr 2013 to Self Incoming Outgoing

    class  GearTest  <  MiniTest::Unit::TestCase    def  test_set_cog        @observer  =  Obs.new        gear  =  Gear.new                                                                                                                    @gear.set_cog(27)        assert(27,  @gear.cog)    end end Friday, April 5, 13
  95. @sandimetz GoGaRuCo 2012 @sandimetz Apr 2013 to Self Incoming Outgoing

    class  GearTest  <  MiniTest::Unit::TestCase    def  test_set_cog        @observer  =  Obs.new        @gear          =  Gear.new(                                    chainring:  52,                                    cog:              11,                                    observer:    @observer)        @gear.set_cog(27)        assert(27,  @gear.cog)    end end Friday, April 5, 13
  96. @sandimetz GoGaRuCo 2012 @sandimetz Apr 2013 to Self Incoming Outgoing

    class  GearTest  <  MiniTest::Unit::TestCase    def  test_set_cog        @observer  =  Obs.new        @gear          =  Gear.new(                                    chainring:  52,                                    cog:              11,                                    observer:    @observer)        @gear.set_cog(27)        assert(27,  @gear.cog)    end end Runs real @observer#changed 1st version Friday, April 5, 13
  97. @sandimetz GoGaRuCo 2012 @sandimetz Apr 2013 to Self Incoming Outgoing

    class  GearTest  <  MiniTest::Unit::TestCase    def  test_set_cog        @observer  =  Obs.new        @gear          =  Gear.new(                                    chainring:  52,                                    cog:              11,                                    observer:    @observer)        @observer.stub(:changed,  @gear.set_cog(27))  do            assert(27,  @gear.cog)        end    end end Variant Runs fake @observer#changed Friday, April 5, 13
  98. @sandimetz GoGaRuCo 2012 @sandimetz Apr 2013 to Self Incoming Outgoing

    class  Gear        #  ...    def  set_cog(new_cog)        @cog  =  new_cog        changed        @cog    end    def  changed        observer.changed(chainring,  cog)    end end Friday, April 5, 13
  99. @sandimetz GoGaRuCo 2012 @sandimetz Apr 2013 to Self Incoming Outgoing

    class  Gear        #  ...    def  set_cog(new_cog)        @cog  =  new_cog        changed        @cog    end    def  changed        observer.changed(chainring,  cog)    end end The app will be correct only if this message gets sent. Friday, April 5, 13
  100. @sandimetz GoGaRuCo 2012 @sandimetz Apr 2013 Wheel Incoming Query changed

    Side Effects Gear Incoming Command Outgoing Obs Outgoing diameter gear_inches set_cog Friday, April 5, 13
  101. @sandimetz GoGaRuCo 2012 @sandimetz Apr 2013 to Self Outgoing Incoming

    Anti-Pattern to Self Incoming Outgoing Friday, April 5, 13
  102. @sandimetz GoGaRuCo 2012 @sandimetz Apr 2013 to Self Incoming Outgoing

    class  GearTest  <  MiniTest::Unit::TestCase    def  test_saves_changed_cog_in_db        @observer  =  Obs.new        @gear          =  Gear.new(                                    chainring:  52,                                    cog:              11,                                    observer:    @observer)          @gear.set_cog(27)          #  assert  something  about  the  state  of  the  db    end end Friday, April 5, 13
  103. @sandimetz GoGaRuCo 2012 @sandimetz Apr 2013 to Self Incoming Outgoing

    class  GearTest  <  MiniTest::Unit::TestCase    def  test_saves_changed_cog_in_db        @observer  =  Obs.new        @gear          =  Gear.new(                                    chainring:  52,                                    cog:              11,                                    observer:    @observer)          @gear.set_cog(27)          #  assert  something  about  the  state  of  the  db    end end Friday, April 5, 13
  104. @sandimetz GoGaRuCo 2012 @sandimetz Apr 2013 to Self Incoming Outgoing

    class  GearTest  <  MiniTest::Unit::TestCase    def  test_saves_changed_cog_in_db        @observer  =  Obs.new        @gear          =  Gear.new(                                    chainring:  52,                                    cog:              11,                                    observer:    @observer)          @gear.set_cog(27)          #  assert  something  about  the  state  of  the  db    end end Friday, April 5, 13
  105. @sandimetz GoGaRuCo 2012 @sandimetz Apr 2013 to Self Incoming Outgoing

    class  GearTest  <  MiniTest::Unit::TestCase    def  test_saves_changed_cog_in_db        @observer  =  Obs.new        @gear          =  Gear.new(                                    chainring:  52,                                    cog:              11,                                    observer:    @observer)          @gear.set_cog(27)          #  assert  something  about  the  state  of  the  db    end end Depends on the distant side effect. Friday, April 5, 13
  106. @sandimetz GoGaRuCo 2012 @sandimetz Apr 2013 to Self Incoming Outgoing

    class  GearTest  <  MiniTest::Unit::TestCase    def  test_saves_changed_cog_in_db        @observer  =  Obs.new        @gear          =  Gear.new(                                    chainring:  52,                                    cog:              11,                                    observer:    @observer)          @gear.set_cog(27)          #  assert  something  about  the  state  of  the  db    end end Is this Gear’s responsibility? Friday, April 5, 13
  107. @sandimetz GoGaRuCo 2012 @sandimetz Apr 2013 to Self Incoming Outgoing

    class  GearTest  <  MiniTest::Unit::TestCase    def  test_saves_changed_cog_in_db        @observer  =  Obs.new        @gear          =  Gear.new(                                    chainring:  52,                                    cog:              11,                                    observer:    @observer)          @gear.set_cog(27)          #  assert  something  about  the  state  of  the  db    end end Is this Gear’s responsibility? Friday, April 5, 13
  108. @sandimetz GoGaRuCo 2012 @sandimetz Apr 2013 to Self Incoming Outgoing

    class  GearTest  <  MiniTest::Unit::TestCase    def  test_saves_changed_cog_in_db        @observer  =  Obs.new        @gear          =  Gear.new(                                    chainring:  52,                                    cog:              11,                                    observer:    @observer)          @gear.set_cog(27)          #  assert  something  about  the  state  of  the  db    end end This is an integration test. Friday, April 5, 13
  109. @sandimetz GoGaRuCo 2012 @sandimetz Apr 2013 to Self Incoming Outgoing

    class  GearTest  <  MiniTest::Unit::TestCase    def  test_notifies_observers_when_cogs_change        @observer  =  MiniTest::Mock.new        @gear          =  Gear.new(                                    chainring:  52,                                    cog:              11,                                    observer:    @observer)        @observer.expect(:changed,  true,  [52,  27])        @gear.set_cog(27)        @observer.verify    end Friday, April 5, 13
  110. @sandimetz GoGaRuCo 2012 @sandimetz Apr 2013 to Self Incoming Outgoing

    class  GearTest  <  MiniTest::Unit::TestCase    def  test_notifies_observers_when_cogs_change        @observer  =  MiniTest::Mock.new        @gear          =  Gear.new(                                    chainring:  52,                                    cog:              11,                                    observer:    @observer)        @observer.expect(:changed,  true,  [52,  27])        @gear.set_cog(27)        @observer.verify    end Friday, April 5, 13
  111. @sandimetz GoGaRuCo 2012 @sandimetz Apr 2013 to Self Incoming Outgoing

    class  GearTest  <  MiniTest::Unit::TestCase    def  test_notifies_observers_when_cogs_change        @observer  =  MiniTest::Mock.new        @gear          =  Gear.new(                                    chainring:  52,                                    cog:              11,                                    observer:    @observer)        @observer.expect(:changed,  true,  [52,  27])        @gear.set_cog(27)        @observer.verify    end Friday, April 5, 13
  112. @sandimetz GoGaRuCo 2012 @sandimetz Apr 2013 to Self Incoming Outgoing

    class  GearTest  <  MiniTest::Unit::TestCase    def  test_notifies_observers_when_cogs_change        @observer  =  MiniTest::Mock.new        @gear          =  Gear.new(                                    chainring:  52,                                    cog:              11,                                    observer:    @observer)        @observer.expect(:changed,  true,  [52,  27])        @gear.set_cog(27)        @observer.verify    end Friday, April 5, 13
  113. @sandimetz GoGaRuCo 2012 @sandimetz Apr 2013 to Self Incoming Outgoing

    class  GearTest  <  MiniTest::Unit::TestCase    def  test_notifies_observers_when_cogs_change        @observer  =  MiniTest::Mock.new        @gear          =  Gear.new(                                    chainring:  52,                                    cog:              11,                                    observer:    @observer)        @observer.expect(:changed,  true,  [52,  27])        @gear.set_cog(27)        @observer.verify    end Friday, April 5, 13
  114. @sandimetz GoGaRuCo 2012 @sandimetz Apr 2013 to Self Incoming Outgoing

    class  GearTest  <  MiniTest::Unit::TestCase    def  test_notifies_observers_when_cogs_change        @observer  =  MiniTest::Mock.new        @gear          =  Gear.new(                                    chainring:  52,                                    cog:              11,                                    observer:    @observer)        @observer.expect(:changed,  true,  [52,  27])        @gear.set_cog(27)        @observer.verify    end Friday, April 5, 13
  115. @sandimetz GoGaRuCo 2012 @sandimetz Apr 2013 to Self Incoming Outgoing

    class  GearTest  <  MiniTest::Unit::TestCase    def  test_notifies_observers_when_cogs_change        @observer  =  MiniTest::Mock.new        @gear          =  Gear.new(                                    chainring:  52,                                    cog:              11,                                    observer:    @observer)        @observer.expect(:changed,  true,  [52,  27])        @gear.set_cog(27)        @observer.verify    end Depends on the interface. Friday, April 5, 13
  116. @sandimetz GoGaRuCo 2012 @sandimetz Apr 2013 to Self Incoming Outgoing

    class  GearTest  <  MiniTest::Unit::TestCase    def  test_notifies_observers_when_cogs_change        @observer  =  MiniTest::Mock.new        @gear          =  Gear.new(                                    chainring:  52,                                    cog:              11,                                    observer:    @observer)        @observer.expect(:changed,  true,  [52,  27])        @gear.set_cog(27)        @observer.verify    end Gear is responsible for sending #changed to @observer. Friday, April 5, 13
  117. @sandimetz Apr 2013 Breaking this rule can save $$$ if

    side effects are both stable and cheap. Caveat Friday, April 5, 13
  118. Query Command Assert result Assert direct public side effects Ignore

    The Unit Testing Minimalist Incoming Type POV @sandimetz Apr 2013 Message Ignore Sent to Self Outgoing Friday, April 5, 13
  119. Query Command Assert result Assert direct public side effects Ignore

    Expect to send The Unit Testing Minimalist Incoming Type POV @sandimetz Apr 2013 Message Ignore Sent to Self Outgoing Friday, April 5, 13
  120. @sandimetz GoGaRuCo 2012 @sandimetz Apr 2013 to Self Incoming Outgoing

    class  GearTest  <  MiniTest::Unit::TestCase    def  test_notifies_observers_when_cogs_change        @observer  =  MiniTest::Mock.new        @gear          =  Gear.new(                                    chainring:  52,                                    cog:              11,                                    observer:    @observer)        @observer.expect(:changed,  true,  [52,  27])        @gear.set_cog(27)        @observer.verify    end What happens if the Observer class stops implementing #changed? Friday, April 5, 13
  121. @sandimetz Apr 2013 Stubs Stubs de ne context. Stub as

    needed (but zillions is a smell). Friday, April 5, 13
  122. @sandimetz Apr 2013 Stubs Stubs de ne context. Stub as

    needed (but zillions is a smell). Avoid stubbing in the object under test. Friday, April 5, 13
  123. @sandimetz Apr 2013 Stubs Stubs de ne context. Stub as

    needed (but zillions is a smell). Avoid stubbing in the object under test. Do not make assertions about stubs! Friday, April 5, 13
  124. @sandimetz Apr 2013 Mocks Mocks test behavior. Set expectations on

    mocks to test outgoing command messages. Friday, April 5, 13
  125. @sandimetz Apr 2013 Mocks Mocks test behavior. Set expectations on

    mocks to test outgoing command messages. One expectation per test (do your best). Friday, April 5, 13
  126. @sandimetz Apr 2013 Mocks Mocks test behavior. Set expectations on

    mocks to test outgoing command messages. One expectation per test (do your best). Do not stub using a mock! Friday, April 5, 13
  127. @sandimetz GoGaRuCo 2012 @sandimetz Apr 2013 to Self Incoming Outgoing

    class  GearTest  <  MiniTest::Unit::TestCase    def  test_notifies_observers_when_cogs_change        @observer  =  MiniTest::Mock.new        @gear          =  Gear.new(                                    chainring:  52,                                    cog:              11,                                    observer:    @observer)        @observer.expect(:changed,  true,  [52,  27])        @gear.set_cog(27)        @observer.verify    end So really, what happens if Observer stops implementing #changed? Friday, April 5, 13
  128. @sandimetz Apr 2013 Honor the contract. Ensure test doubles stay

    in sync with the API. Rule Friday, April 5, 13
  129. @sandimetz Apr 2013 Automagically  minitest  requires  that  stubbed  methods  exist

     https://github.com/benmoss/quacky  https://github.com/xaviershay/rspec-­‐fire  https://github.com/cfcosta/minitest-­‐firemock Friday, April 5, 13
  130. Query Command Assert result Assert direct public side effects Ignore

    Expect to send Be a Minimalist Incoming Type POV @sandimetz Apr 2013 Message Ignore Sent to Self Outgoing Friday, April 5, 13
  131. http://www. ickr.com/photos/phil_shirley/5452562957/ thicket http://www. ickr.com/photos/zebble/6080622/ one padlock http://www. ickr.com/photos/chrisinplymouth/5543281168/ two

    padlocks http://www. ickr.com/photos/donjohnson395/2915304926/ twins at beach http://www.med.navy.mil/sites/nmcp/Patients/pediatrics/PublishingImages/binocular_boy.gif http://upload.wikimedia.org/wikipedia/commons/e/e1/Ordinary_bicycle02.jpg ordinary bike Magic Hat (82805980) Copyright Mmaxer - Shutterstock.com Dogs in Hats (39263399) Copyright MLoiselle - Fotolia.com Garden (35521385) Copyright Beboy - Fotolia.com Space Walk - Nasa http://grin.hq.nasa.gov/copyright.html Photo Credits Friday, April 5, 13