qa:whattotest
Differences
This shows you the differences between two versions of the page.
Both sides previous revisionPrevious revisionNext revision | Previous revision | ||
qa:whattotest [2008/05/06 10:55] – zoe | qa:whattotest [2017/09/22 13:28] (current) – external edit 127.0.0.1 | ||
---|---|---|---|
Line 1: | Line 1: | ||
- | ==== More information on what to test ==== | + | ====== More information on what to test ====== |
- | You can tell a certain amount from line coverage - but it doesn' | + | There are currently independent efforts underway to improve the testing |
- | I've put the source code for two scripts | + | ===== Some scripts |
- | 1) This script runs through all of the phpt tests in the source tree and then compares the function calls in the tests with a complete list of PHP's internal functions. At the end it will tell you if there is a function in PHP that is not mentioned in any test case (I get 415). Thanks to Damien Seguy for this. | + | You can tell a certain amount from line coverage - but it doesn't really show you exactly what tests there are already |
- | [[http:// | + | I've put the source code for three scripts below - they are a little unfinished, for example they all produce warning messages (which I think can safely be ignored). |
- | 2) A slight modification to Damien' | + | The first two scripts |
- | [[http:// | ||
+ | ==== Script 1 ==== | ||
+ | This script runs through all of the phpt tests in the source tree and then compares the function calls in the tests with a complete list of PHP's internal functions. At the end it will tell you if there is a function in PHP that is not mentioned in any test case (I get 415). Thanks to Damien Seguy for this. | ||
+ | <code php> | ||
+ | <?php | ||
+ | // define here your PHP source directory | ||
+ | define(' | ||
+ | |||
+ | //Get a list of all of the test (phpt) files in the source tree. | ||
+ | $files = array(); | ||
+ | get_phpt_files(PHP_SRC, | ||
+ | |||
+ | //Find all the functions that are called in the test case code | ||
+ | $tested = array(); | ||
+ | foreach($files as $file) { | ||
+ | $tested = array_merge(extract_tests($file), | ||
+ | } | ||
+ | $tested = array_unique($tested); | ||
+ | // | ||
+ | |||
+ | //Get a list of all of the internal PHP functions | ||
+ | $internal = get_defined_functions(); | ||
+ | $internal = $internal[' | ||
+ | |||
+ | //Compare the list of all function with those that are called from a phpt test case. | ||
+ | $non_tested = array_diff($internal, | ||
+ | print_r($non_tested); | ||
+ | print count($non_tested). " untested functions\n"; | ||
+ | |||
+ | |||
+ | function | ||
+ | $code = file_get_contents($file); | ||
+ | |||
+ | if (!preg_match('/ | ||
+ | // print " | ||
+ | return array(); | ||
+ | } | ||
+ | |||
+ | $tokens = token_get_all($r[1]); | ||
+ | $functions = array_filter($tokens, | ||
+ | $functions = array_map( ' | ||
+ | $functions = array_unique($functions); | ||
+ | |||
+ | return $functions; | ||
+ | } | ||
+ | |||
+ | function filter_functions($x) { | ||
+ | return $x[0] == 307; | ||
+ | } | ||
+ | |||
+ | function map_token_value($x) { | ||
+ | return $x[1]; | ||
+ | } | ||
+ | function get_phpt_files($dir, | ||
+ | { | ||
+ | $thisdir = dir($dir.'/' | ||
+ | while(($file = $thisdir-> | ||
+ | if ($file != ' | ||
+ | $path = $thisdir-> | ||
+ | if(is_dir($path) == true) { | ||
+ | get_phpt_files($path , $phpt_file_count , $all_phpt); | ||
+ | } else { | ||
+ | if (preg_match("/ | ||
+ | $all_phpt[$phpt_file_count] = $path; | ||
+ | $phpt_file_count++; | ||
+ | } | ||
+ | } | ||
+ | } | ||
+ | } | ||
+ | return; | ||
+ | } | ||
+ | |||
+ | |||
+ | ?> | ||
+ | </ | ||
+ | |||
+ | |||
+ | ==== Script 2 ==== | ||
+ | A slight modification to Damien' | ||
+ | <code php> | ||
+ | <?php | ||
+ | // define here your PHP source directory | ||
+ | define(' | ||
+ | |||
+ | //Get a list of all of the test (phpt) files in the source tree. | ||
+ | $files = array(); | ||
+ | get_phpt_files(PHP_SRC, | ||
+ | |||
+ | |||
+ | $tests_by_function=array(); | ||
+ | |||
+ | //Get a list of all of the internal PHP functions | ||
+ | $internal = get_defined_functions(); | ||
+ | $internal = $internal[' | ||
+ | foreach($internal as $function) { | ||
+ | $tests_by_function[$function] = array(); | ||
+ | } | ||
+ | |||
+ | //Find all the functions that are called in the test case code | ||
+ | $tested = array(); | ||
+ | $temp_array = array(); | ||
+ | |||
+ | foreach($files as $file) { | ||
+ | $temp_array = extract_tests($file); | ||
+ | $tested = array_merge($temp_array, | ||
+ | foreach($temp_array as $function) { | ||
+ | if(array_key_exists($function, | ||
+ | array_push($tests_by_function[$function], | ||
+ | } | ||
+ | } | ||
+ | } | ||
+ | $tested = array_unique($tested); | ||
+ | print_r($tests_by_function); | ||
+ | |||
+ | |||
+ | function | ||
+ | $code = file_get_contents($file); | ||
+ | |||
+ | if (!preg_match('/ | ||
+ | // print " | ||
+ | return array(); | ||
+ | } | ||
+ | |||
+ | $tokens = token_get_all($r[1]); | ||
+ | $functions = array_filter($tokens, | ||
+ | $functions = array_map( ' | ||
+ | $functions = array_unique($functions); | ||
+ | |||
+ | return $functions; | ||
+ | } | ||
+ | |||
+ | function filter_functions($x) { | ||
+ | return $x[0] == 307; | ||
+ | } | ||
+ | |||
+ | function map_token_value($x) { | ||
+ | return $x[1]; | ||
+ | } | ||
+ | function get_phpt_files($dir, | ||
+ | { | ||
+ | $thisdir = dir($dir.'/' | ||
+ | while(($file = $thisdir-> | ||
+ | if ($file != ' | ||
+ | $path = $thisdir-> | ||
+ | if(is_dir($path) == true) { | ||
+ | get_phpt_files($path , $phpt_file_count , $all_phpt); | ||
+ | } else { | ||
+ | if (preg_match("/ | ||
+ | $all_phpt[$phpt_file_count] = $path; | ||
+ | $phpt_file_count++; | ||
+ | } | ||
+ | } | ||
+ | } | ||
+ | } | ||
+ | return; | ||
+ | } | ||
+ | |||
+ | |||
+ | ?> | ||
+ | </ | ||
+ | |||
+ | ==== Script 3 ==== | ||
+ | This script lists all the classes in PHP and then finds their methods using a ReflectionClass. Using essentially the same logic as the other two scripts teh code searches for tests that refer to these methods. The script is flawed in that many classes have methods of the same name (for example %%__toString%% occurs in many classes), so looking for all tests that refer to %%__toString%% will give a list of mainly irrelevant tests. Because of this I've excluded any method beginning " | ||
+ | |||
+ | <code php> | ||
+ | <?php | ||
+ | // define here your PHP source directory | ||
+ | define(' | ||
+ | |||
+ | //Get a list of all of the test (phpt) files in the source tree. | ||
+ | $files = array(); | ||
+ | get_phpt_files(PHP_SRC, | ||
+ | |||
+ | |||
+ | $tests_by_method=array(); | ||
+ | $class_by_method=array(); | ||
+ | |||
+ | //Get a list of all of the PHP classes | ||
+ | $classes = get_declared_classes(); | ||
+ | |||
+ | $methodArray = array(); | ||
+ | $methodNameArray = array(); | ||
+ | $method_names = array(); | ||
+ | |||
+ | //Get the method names from each class | ||
+ | foreach($classes as $class) { | ||
+ | $classInfo = new ReflectionClass($class); | ||
+ | $methodArray = $classInfo-> | ||
+ | $method_names = get_method_names($methodArray); | ||
+ | $class_by_method[$class] = array(); | ||
+ | array_push($class_by_method[$class], | ||
+ | $methodNameArray = array_merge($method_names, | ||
+ | foreach($method_names as $method) { | ||
+ | $tests_by_method[$method] = array(); | ||
+ | } | ||
+ | } | ||
+ | |||
+ | //Find all the functions that are called in the test case code | ||
+ | $tested = array(); | ||
+ | $temp_array = array(); | ||
+ | |||
+ | foreach($files as $file) { | ||
+ | $temp_array = extract_tests($file); | ||
+ | $tested = array_merge($temp_array, | ||
+ | foreach($temp_array as $function) { | ||
+ | if(array_key_exists($function, | ||
+ | array_push($tests_by_method[$function], | ||
+ | } | ||
+ | } | ||
+ | } | ||
+ | //Print out the tests in which methods are used | ||
+ | //print_r ($class_by_method); | ||
+ | foreach($classes as $class) { | ||
+ | echo " | ||
+ | foreach($class_by_method[$class] as $methods) { | ||
+ | foreach($methods as $method) { | ||
+ | if(!preg_match('/ | ||
+ | echo " | ||
+ | foreach($tests_by_method[$method] as $test) { | ||
+ | echo "Test: " | ||
+ | } | ||
+ | } | ||
+ | } | ||
+ | } | ||
+ | } | ||
+ | |||
+ | |||
+ | function | ||
+ | $code = file_get_contents($file); | ||
+ | |||
+ | if (!preg_match('/ | ||
+ | // print " | ||
+ | return array(); | ||
+ | } | ||
+ | |||
+ | $tokens = token_get_all($r[1]); | ||
+ | $functions = array_filter($tokens, | ||
+ | $functions = array_map( ' | ||
+ | $functions = array_unique($functions); | ||
+ | return $functions; | ||
+ | } | ||
+ | |||
+ | function filter_functions($x) { | ||
+ | return $x[0] == 307; | ||
+ | } | ||
+ | |||
+ | function map_token_value($x) { | ||
+ | return $x[1]; | ||
+ | } | ||
+ | function get_phpt_files($dir, | ||
+ | { | ||
+ | $thisdir = dir($dir.'/' | ||
+ | while(($file = $thisdir-> | ||
+ | if ($file != ' | ||
+ | $path = $thisdir-> | ||
+ | if(is_dir($path) == true) { | ||
+ | get_phpt_files($path , $phpt_file_count , $all_phpt); | ||
+ | } else { | ||
+ | if (preg_match("/ | ||
+ | $all_phpt[$phpt_file_count] = $path; | ||
+ | $phpt_file_count++; | ||
+ | } | ||
+ | } | ||
+ | } | ||
+ | } | ||
+ | return; | ||
+ | } | ||
+ | function get_method_names($methodArray) { | ||
+ | $names = array(); | ||
+ | foreach ($methodArray as $method) { | ||
+ | array_push($names, | ||
+ | } | ||
+ | return $names; | ||
+ | } | ||
+ | |||
+ | ?> | ||
+ | </ |
qa/whattotest.1210071345.txt.gz · Last modified: 2017/09/22 13:28 (external edit)