{"id":682,"date":"2011-11-18T13:29:00","date_gmt":"2011-11-18T13:29:00","guid":{"rendered":"http:\/\/www.javacodegeeks.com\/2012\/10\/the-misuse-of-end-to-end-tests-testing-techniques-2.html"},"modified":"2012-10-21T20:38:37","modified_gmt":"2012-10-21T20:38:37","slug":"misuse-of-end-to-end-tests-testing","status":"publish","type":"post","link":"https:\/\/www.javacodegeeks.com\/2011\/11\/misuse-of-end-to-end-tests-testing.html","title":{"rendered":"The Misuse of End To End Tests – Testing Techniques 2"},"content":{"rendered":"
My last blog<\/a> was the first in a series of blogs on approaches to testing code, outlining a simple scenario of retrieving an address from a database using a very common pattern:<\/p>\n
<\/a><\/div>\n

…and describing a very common testing technique: not writing tests<\/i> and doing everything manually.<\/p>\n

Today\u2019s blog covers another practise which I also feel is sub-optimal. In this scenario, the developers use JUnit to write tests, but write them after they have completed writing the code and without any class isolation. This is really an \u2018End to End\u2019 (aka Integration) test posing as a Unit Test.<\/p>\n

Although yesterday I said that I\u2019m only testing the AddressService class, a test using this technique starts by loading the database with some test data and then grabbing hold of the AddressController<\/strong> to call the method under test. The AddressController<\/strong> calls the AddressService<\/strong> which then calls the AddressDao<\/strong> to obtain and return the requested data.<\/p>\n

@RunWith(UnitilsJUnit4TestClassRunner.class)\r\n@SpringApplicationContext(\"servlet-context.xml\")\r\n@Transactional(TransactionMode.DISABLED)\r\npublic class EndToEndAddressServiceTest {\r\n\r\n  @SpringBeanByType\r\n  private AddressController instance;\r\n\r\n  \/**\r\n   * Test method for\r\n   * {@link com.captaindebug.address.AddressService#findAddress(int)}.\r\n   *\/\r\n  @Test\r\n  public void testFindAddressWithNoAddress() {\r\n\r\n    final int id = 10;\r\n    BindingAwareModelMap model = new BindingAwareModelMap();\r\n\r\n    String result = instance.findAddress(id, model);\r\n    assertEquals(\"address-display\", result);\r\n\r\n    Address resultAddress = (Address) model.get(\"address\");\r\n    assertEquals(Address.INVALID_ADDRESS, resultAddress);\r\n  }\r\n\r\n  \/**\r\n   * Test method for\r\n   * {@link com.captaindebug.address.AddressService#findAddress(int)}.\r\n   *\/\r\n  @Test\r\n  @DataSet(\"FindAddress.xml\")\r\n  public void testFindAddress() {\r\n\r\n    final int id = 1;\r\n    Address expected = new Address(id, \"15 My Street\", \"My Town\",\r\n        \"POSTCODE\", \"My Country\");\r\n\r\n    BindingAwareModelMap model = new BindingAwareModelMap();\r\n\r\n    String result = instance.findAddress(id, model);\r\n    assertEquals(\"address-display\", result);\r\n\r\n    Address resultAddress = (Address) model.get(\"address\");\r\n    assertEquals(expected.getId(), resultAddress.getId());\r\n    assertEquals(expected.getStreet(), resultAddress.getStreet());\r\n    assertEquals(expected.getTown(), resultAddress.getTown());\r\n    assertEquals(expected.getPostCode(), resultAddress.getPostCode());\r\n    assertEquals(expected.getCountry(), resultAddress.getCountry());\r\n  }\r\n}\r\n<\/pre>\n
The code above uses Unitils<\/a> to both load the test data into a database and to load the classes in our Spring context. I find Untils a useful tool that takes the hard work out of writing tests like this and having to setup such a large scale test is hard work.
 
<\/div> <\/div><\/p>\n
This kind of test has to be written after the code has been completed; it\u2019s NOT Test Driven Development (which from previous blogs, you\u2019ll gather I\u2019m a big fan), and it\u2019s not a unit test. One of the problems with writing a test after<\/i> the code is that developers who have to do it see it as a chore rather than as part of development, which means that it\u2019s often rushed and not done in the neatest of coding styles.<\/p>\n
You\u2019ll also need a certain amount of infrastructure to code using this technique, as a database needs setting up, which may or may not be on your local machine and consequently you may have to be connected to a network to run the test. Test data is either held in test files (as in this case), which are loaded into the database when the test runs, or held permanently in the database. If a requirement change forces a change in the test, then the database files will usually need updating together with the test code, which forces you to update the test in at least two places. <\/p>\n
Another big problem with this kind of test, apart from the lack of test subject isolation, is that fact that they can be very slow, sometimes taking seconds to execute. Shane Warden in his book \u2018The Art of Agile Development\u2019 states that unit tests should run at a rate of \u201chundreds per second\u201d. Warden also goes on to cite Michael Feather\u2019s book Working Effectively with Legacy Code<\/i> for a good definition of what a unit test, or is not:<\/p>\n
A test is not a unit test if:<\/p>\n
    \n
  1. It talks to a database.<\/li>\n
  2. It communicates across a network.<\/li>\n
  3. It touches the file system.<\/li>\n
  4. You have to do special things to your environment (such as editing configuration files) to run it.<\/li>\n<\/ol>\n
    …now I like that.<\/p>\n
    …although I don\u2019t necessarily agree with point three. One of the main tenants of good unit test code is Readability. Method arguments that are passed to objects under test are sometimes large in size, especially when they\u2019re XML. In this case I think that it\u2019s more pragmatic to favour test readability and store data of this size in a data file rather than having it as a private static final String, so I only adhere to point three where ever practical.<\/span><\/p>\n
    Unit tests can be summed up using the FIRST<\/a> acronym: Fast, Independent, Repeatable, Self Validating and Timely, whilst Roy Osherove in his book The Art Of Unit Testing<\/a> sums up a good unit test as: “an automated piece of code that invokes the method or class being tested and then checks some assumptions about the logical behaviour of that method or class. A unit test is almost always written using a unit testing framework. It can be written easily and runs quickly. It’s full automated, trustworthy, readable and maintainable”.<\/p>\n
    The benefit of an End to End test is that they do test your test subject in collaboration with other objects and surroundings, something that you really must<\/strong> before shipping your code. This means that when complete, you code should comprise of hundreds<\/i> of unit tests, but only tens of \u2018End to End\u2019 tests.<\/p>\n
    Given this, then my introductory premise, when I said that is technique is \u2018sub-optimal\u2019, is not strictly true; there\u2019s nothing wrong with \u2018End to End\u2019 tests, every project should have some together with some ordinary integration tests, but these kinds of tests shouldn\u2019t replace, or be called unit tests, which is often the case.<\/p>\n
    Having defined what a unit test is, my next blog investigates what you should test and why…<\/p>\n
    Reference: <\/i><\/strong>The Misuse of End To End Tests – Testing Techniques 2<\/a> from our JCG partner<\/a> Roger Hughes <\/span><\/span>at the Captain Debug blog<\/a><\/p>\n
    Related Articles :<\/i><\/strong><\/p>\n