Note:

If you want to create a new page for developers, you should create it on the Moodle Developer Resource site.

Unit test API: Difference between revisions

From MoodleDocs
(MDL-35628)
Line 120: Line 120:


<code php>
<code php>
require_once(dirname(__FILE__) . '/../../config.php'); // Do this.
require_once(__DIR__ . '/../../config.php'); // Do this.
</code>
</code>



Revision as of 16:48, 14 June 2016

Location: Administration > Reports > Unit tests in 1.9 or Site administration > Development > Unit tests in Moodle 2.0-2.2

See PHPUnit page for new unit testing support in Moodle 2.3


The purpose of Unit tests is to evaluate the individual parts of a program (functions, and methods of classes) to make sure that each element individually does the right thing. Unit Tests can be one of the first steps in a quality control process for developing or tweaking Moodle code. The next steps will involve other forms of testing to ensure that these different parts work together properly.

The unit testing framework is based on the SimpleTest framework.


Running the unit tests in Moodle

Running the basic tests

  1. Log in with an admin account.
  2. Access Administration > Reports > Unit tests in 1.9 or Site administration > Development > Unit tests in Moodle 2.0 onwards
  3. Click on the Reports link near the bottom of the page.
  4. Click the Run tests button and wait.

This finds all the tests in Moodle and runs them. You can run a subset of the tests by entering a path (for example question/type) in the 'Only run tests in' box. Similarly, if a test fails, you get some links in the failure message to make it easy to re-run just those tests.

Writing new tests

As an example, suppose we wanted to write some tests for the string_manager class in mod/quiz/editlib.php.

Where to put the tests

The unit test report finds tests by looking for files called 'test....php' inside folders called 'simpletest'.

So, for our example, we want to create called something like mod/quiz/simpletest/testeditlib.php. The skeleton of this file should look like:

<?php /**

* Unit tests for (some of) mod/quiz/editlib.php.
*
* @license http://www.gnu.org/copyleft/gpl.html GNU Public License
* @package question
*/

if (!defined('MOODLE_INTERNAL')) {

   die('Direct access to this script is forbidden.'); //  It must be included from a Moodle page

}

// Make sure the code being tested is accessible. require_once($CFG->dirroot . '/mod/quiz/editlib.php'); // Include the code to test

/** This class contains the test cases for the functions in editlib.php. */ class quiz_editlib_test extends UnitTestCase {

   function test_something() {
       // Do the test here.
   }
   // ... more test methods.

} ?>

That is, you have a class called something_test, and in that class you have lots of methods called test_something. Normally, you have one test method for each particular thing you want to test, and you should try to name the function to describe what is being tested - without making the name too ridiculously long!

A test function

The a test function typically looks like

function test_move_question_up() {

   // Setup fixture
   // Exercise SUT
   $newlayout = quiz_move_question_up('1,2,0', 2);
   // Validate outcome
   $this->assertEqual($newlayout, '2,1,0');
   // Teardown fixture

}

This is the four phase test pattern. Those comments use a lot of testing jargon. The fixture is the background situation that needs to be set up before the test runs. SUT is short for 'situation under test'. This is where you call the function or method that you want to test. Then you check to see if the function did the right thing. Finally, you have to clean up the fixture you created. With luck there is nothing to do here

In this simple example, there is no setup or teardown to do. We just call the function we are testing with some sample input, and check that the return value is what we expect.

Shared setUp and tearDown methods

If all your test cases relate to the same area of code, then they may all need to same bit of fixture set up. For example, all the tests in lib/simpletest/teststringmanager.php need an instance of the string_manager class to test.

To avoid duplicating code, you can override a method called setUp() that sets up the test data. If present, this method will be called before each test method. You can write a matching tearDown() method if there is any clean-up that needs to be done after each test case has run. For example, in lib/simpletest/teststringmanager.php there are setUp and tearDown methods that do something like: public function setUp() {

   // ...
   $this->stringmanager = new string_manager(...);

}

public function tearDown() {

   $this->stringmanager = null;

} Then, each test can use $this->stringmanager without having to worry about the details of how it is set up.

Further information

The SimpleTest documentation is at: http://www.simpletest.org/.

Changes to your existing code to make it work with unit testing

The whole point of unit testing is to test each piece of functionality separately. You can only do this is by isolating that function and call it individually, perhaps after setting up a few other things.

Therefore, it is good if you can write your code to depend on as few other things as possible.

Include paths

Includes like

require_once('../../config.php'); // Won't work.

won't work. Instead, the more robust option is

require_once(__DIR__ . '/../../config.php'); // Do this.

Access to global variables

Because your code was included from within a function, you can't access global variables until you have done a global statement.

require_once(dirname(__FILE__) . '/../../config.php'); require_once($CFG->libdir . '/moodlelib.php'); // Won't work.

require_once(dirname(__FILE__) . '/../../config.php');

global $CFG; // You need this. require_once($CFG->libdir . '/moodlelib.php'); // Will work now.

Calls to global functions

Testing a class method that calls global functions can be problematic. At least, it's always complex, because we can't control what goes on in the global functions. We can't override the global functions or mock them in our unit tests. If the global functions themselves are well tested, this may not be a big problem, but most global functions are not well tested.

Bridge Pattern

If your code needs to rely extensively on some public API, you could use the bridge pattern to decouple your code from that API. This way, when you write unit tests, you can override the bridging class or mock it, and control its outputs while you focus exclusively on testing your code.

An basic example follows: Imagine that I do not trust the get_string() global function, but my code needs to use it. Initially my code has strong coupling with get_string():

class myclass {

   public function print_stuff($stuff) {
       echo get_string($stuff);
   }

}

Now let's write a bridging class to solve this coupling issue and use it instead of get_string():

class languageBridge {

   public function get_string($stuff,$module='moodle') {
       return get_string($stuff, $module);
   }

}

class myclass {

   public $lang_bridge;
   public function __construct() {
       $this->lang_bridge = new languageBridge();
   }
   public function print_stuff($stuff) {
       echo $this->lang_bridge->get_string($stuff);
   }

}

The following is yet another example using a bridging method to decouple from the Moodle core API.

class workshop_api {

   /**
    * This is a method we want to unittest
    */
   public function get_peer_reviewers($context) {
       static $users=null;
       if (is_null($users)) {
           $users = $this->get_users_by_capability($context, 'mod/workshop:peerassess', 
                       'u.id, u.lastname, u.firstname', 'u.lastname,u.firstname', , , , , false, false, true);
       }
       return $users;
   }
   /**  
    * Bridging method to decouple from Moodle core API
    */
   protected function get_users_by_capability() {
       $args = func_get_args();
       return call_user_func_array('get_users_by_capability', $args);
   }

}

Warning: Here are some comments on the examples above expressing that the bridge pattern should be used very carefully.

  • "I think that is a case of unit tests leading to worse software design, in that you are not using the standard API for something. But if you really want to unit test, I can't think of a better solution."
  • "I think this can be OK if used very selectively. Unfortunately I don't think it's the solution if you want to decouple the very complex and deeply nested Moodle functions from each other"
  • "...your code is designed to be part of Moodle, so decoupling from a standard Moodle API is perverse."

Unit testing in 2.0

Moodle 2.0


With the Objectification of the Database libraries in Moodle 2.0, new and better approaches to Unit testing can be used. Here is a sample of a simple test case: (in course/simpletest)

require_once($CFG->dirroot . '/course/lib.php');

global $DB;
Mock::generate(get_class($DB), 'mockDB');

class courselib_test extends UnitTestCase {
    var $realDB;

    function setUp() {
        global $DB;
        $this->realDB = $DB;
        $DB           = new mockDB();
    }

    function tearDown() {
        global $DB;
        $DB = $this->realDB;
    }

    function testMoveSection() {
        global $DB;
        $course = new stdClass();
        $course->id = 1;

        $sections = array();
        for ($i = 1; $i < 11; $i++) {
            $sections[$i]          = new stdClass();
            $sections[$i]->id      = $i;
            $sections[$i]->section = $i - 1;
        }

        $DB->expectOnce('get_records', array('course_sections', array('course' => $course->id)));
        $DB->setReturnValue('get_records', $sections);
        $this->assertFalse(move_section($course, 2, 3));
    }
}

See also UnitTestCaseUsingDatabase in lib/simpletestlib.php.

Testing HTML output

(work in progress - to be documented properly once the API stabilizes)

  • ContainsTagWithAttribute($tag, $attribute, $value)
  • ContainsTagWithAttributes($tag, $attributes)
  • ContainsTagWithContents($tag, $content)
  • ContainsEmptyTag($tag)

The syntax is $this->assert(new ContainsTagWithAttribute($tag, $attribute, $value), $html);

Code coverage analysis

Note: This section is a work in progress. Please use the page comments or an appropriate moodle.org forum for any recommendations/suggestions for improvement.

Moodle 2.0


Code coverage is a technique, strongly tied with software testing, that allows to check and improve the quality of the tests by measuring the degree of source code that is being covered by them. With Moodle supporting more and more tests each day (moving slowly towards a Test Driven Development model) we need to integrate some tool into our development process to help analyse the completeness of our test regime.

Right now (in Moodle 2.0) we are using SimpleTest, a simple and useful tool that supports unit testing. Unfortunately it doesn't support code coverage analysis at all. On the other hand, other PHP unit testing products like PHPUnit, more complex and powerful, have built-in support for code coverage analysis. However switching to a new product is out of scope of our current Roadmap.

Further reading and investigation suggest that Moodle will implement its own extensions to SimpleTest in order to fulfill the main goal of having statement/line code coverage analysis working under Moodle 2.0 onwards. To achieve this Spike PHPCoverage, a basic code-coverage tool, will be used and extended. You can find the details of the implementation of this tool at MDL-19579.

Further update: according to SimpleTest conversion Moodle 2.3 switched to PHPUnit as the recommended unit testing framework and SimpleTest support will be completely removed from Moodle 2.4. See cited link for how to convert test scripts from SimpleTest to PHPUnit

Changes

To enable code coverage in your tests, only a few modifications need to be made to your current code:

  1. Change your test classes by adding two (public static) attributes: $includecoverage and $excludecoverage, both arrays. These are used to inform the code coverage tool (via reflection) about which source code files and directories must be covered/skipped by the analysis.
  2. Use simpletestcoveragelib.php instead of simpletestlib.php in your caller scripts.
  3. Use the autogroup_test_coverage class instead of the AutoGroupTest class (see below for details) in your caller scripts.

Note that only item 1 above is needed for new unit tests being created because both 2 and 3 (changes in caller scripts) are already implemented in Moodle and awaiting your cool unit tests.

That's all! With these 3 basic changes, you will end with a complete code coverage report available for further analysis.

API

When using code coverage within Moodle there are two alternative APIs available, both providing the same code coverage reports at the end, but each doing that in a different way. Here they are:

  • Internal (hidden) coverage API: This API is completely hidden behind the Unit testing API and you won't need to know the details of it. Just make the three changes described above and, after running the tests, the coverage report will be available for use; without needing any other changes to your code. The major drawback: it only can perform one "code coverage session" (a.k.a. instrumentation), so it's only suitable for testing scripts using only one unit test execution. One example of this type of unit testing is admin/report/unittest/index.php where only one (big) test-group is executed.
  • External (explicit) coverage API: This API needs extra coding as long as coverage instantiating, configuration and report generation happens in the main script. It's a bit more complex but, on the other hand, it supports multiple instrumentations to be performed, and gives you more control about the code coverage process. One example of this type of unit testing is admin/report/unittest/dbtest.php where multiple (one for each DB being tested) test-groups are executed.

So, first of all (point 1 in prev section - usage), we need to define, for each unit test, which files / directories (relative to dirroot) we want to analyse with the tool. Here is one example, for the dml_test unit test (/lib/dml/simpletest/testdml.php), all we need to do is add the following lines to the class declaration :

public static $includecoverage = array('lib/dml'); public static $excludecoverage = array('lib/dml/somedir');

By doing so, the code coverage tool will know which are the target files on which to perform coverage analysis/reporting and will do that for all the files (recursively) in the lib/dml' directory but excluding the lib/dml/somedir directory (recursively too). Note that both attributes are arrays so multiple paths can be specified in any of them. Also note that the directory where the UnitTest is stored is automatically excluded (usually the simpletest directories).

And, as said, that's all you need to do in order to get current/new unit tests analysed by the code coverage tool in current Moodle scripts. The documentation below is only interesting for developers wanting to create new scripts able to support unit testing with code coverage (points 2 and 3 in prev section - usage).

Internal coverage API

$test = new autogroup_test_coverage($showsearch, $test_name, $performcoverage, $coveragename, $coveragedir); Create one new autogroup test object with code coverage support, there you can specify if you want to perform coverage (true/false), the name of the report (title) and the directory where the final report will be created (under moodledata/codecoverage). Optionally you can add more files and directories (relative to dirroot) to the list of files to be covered / ignored by using these functions (in case those defined in point 1 aren't enough). $test->add_coverage_include_path($path); $test->add_coverage_exclude_path($path); And then, after adding a bunch of unit tests to the group, you simply invoke the test execution with code coverage support to end with a nice code coverage report under dataroot/codecoverage/$coveragedir: $test->run($unit_test_reporter); (don't forget that this API supports only one instrumentation to be performed)

And that's all!

External coverage API

$covreporter = new moodle_coverage_reporter($coveragename, $coveragedir); $covrecorder = new moodle_coverage_recorder($covreporter); Create one coverage reporter, by passing its title and output directory as parameters. $test = new autogroup_test_coverage($showsearch, $test_name, $performcoverage); Create one new autogroup test object with code coverage support, you don't need to specify the title and dir here (as already have been defined by the moodle_coverage_reporter object). Optionally you can add more files and directories (relative to dirroot) to the list of files to be covered / ignored by using these functions (in case the defined in point 1 aren't enough). $test->add_coverage_include_path($path); $test->add_coverage_exclude_path($path); Then, after adding a bunch of unit tests to the group, you simply invoke the test execution with code coverage support with: $test->run_with_external_coverage($unit_test_reporter, $covrecorder); (don't forget that this API supports multiple instrumentations to be performed)

And finally, you generate the code coverage report (under dataroot/codecoverage/$coveragedir) using: $covrecorder->generate_report(); Once more, that's all!

Final notes

  • Note that there are some more methods available in the moodle_coverage_recorder class. They will allow to control starting/stopping instrumentations by hand and other minor things but they shouldn't really be used. The run() and run_with_external_coverage() methods should be enough in 99% of cases.
  • Not being part of the API, but used by it, there is one coveragefile.php script under admin/report/unittest responsible for serving the coverage report files from within Moodle. See current scripts in that directory to see how it can be used.
  • All the test execution / reporting / coverage utilities must be protected with the 'moodle/site:config' permission.

A warning

In 'xUnit Test Patterns' there is a scale of testing difficulty that goes from 1. to 6. Moodle is definitely at number 6. on that scale 'Non-object-oriented legacy software'. It then goes recommend that you don't start to learn about unit testing with that sort of software :-(

Further reading about unit testing

PHP in Action has an excellent chapter explaining unit testing in PHP with simpletest. (Although the rest of that book advocates a style of programming that is very different from the style used in Moodle.)

Pragmatic Unit Testing in Java with JUnit is also a very good introduction, despite being in the wrong programming language. JUnit and Simpletest are very similar.

xUnit Test Patterns is the ultimate unit test book. I think it teaches you everything you could learn about unit testing by reading a book. The only way to learn more would be years of experience. It has really great advice for dealing with the kind of messy problems you get in a big, real project like Moodle.