PHPUnit Error when using runtests.php

[remib33]

When a unit-tested class contains the following Jelix instruction: jclasses::inc, then there is an error when launching Jelix runtests.php

     [exec] PHP Notice:  Trying to get property of non-object in /home/user/projet/workspace_php/lib/jelix/core/selector/jSelectorLoc.class.php on line 41
     [exec] PHP Notice:  Trying to get property of non-object in /home/user/projet/workspace_php/lib/jelix/core/selector/jSelectorLoc.class.php on line 44
     [exec] PHP Fatal error:  Uncaught exception 'jExceptionSelector' with message 'jelix module is not enabled !!' in /home/user/projet/workspace_php/lib/jelix/core/selector/jSelectorClass.class.php:62
     [exec] Stack trace:
     [exec] #0 /home/user/projet/workspace_php/lib/jelix/core/selector/jSelectorClass.class.php(52): jSelectorClass->_createPath()
     [exec] #1 /home/user/projet/workspace_php/lib/jelix/utils/jClasses.class.php(129): jSelectorClass->__construct('m2m~ws/account/...')
     [exec] #2 /home/user/projet/workspace_php/m2m/modules/m2m/tests/AccountSummary.pu.php(3): jClasses::inc('m2m~ws/account/...')
     [exec] #3 /usr/share/php/PHPUnit/Util/Fileloader.php(95): include_once('/home/user/pr...')
     [exec] #4 /usr/share/php/PHPUnit/Util/Fileloader.php(79): PHPUnit_Util_Fileloader::load('/home/user/pr...')
     [exec] #5 /usr/share/php/PHPUnit/Framework/TestSuite.php(358): PHPUnit_Util_Fileloader::checkAndLoad('/home/user/pr...')
     [exec] #6 /usr/share/php/PHPUnit/Framewo in /home/user/projet/workspace_php/lib/jelix/core/selector/jSelectorClass.class.php on line 62

The direct cause of this is that $gJconfig global var is not properly initialized.

Actually, during PHPUnit addTestFile method call (performed by Jelix), jelix is not properly initialized and jclasses::inc framework instruction is not recognized in AccountSummary? unit tested class.

I propose to perform the following modification in command.php Jelix script: Instantiate JCoordinator class before the existing following code (instead of after in current code)

		// it will initialize global variables $gJCoord $gJConfig. it could be needed by tests
        $coord = new jCoordinator($this->epInfo->configFile, false); // NOW BEFORE THE FOLLOWING CODE:

        if ($modulesTests == 0) {
            // we add all modules in the test list
            $suite = $this->getAllModulesTestSuites();
            if (count($suite)) {
                $this->arguments['test'] = $suite;
                unset ($this->arguments['testFile']);
            }
            else {
                $this->showMessage("Error: no tests in modules\n");
                exit(PHPUnit_TextUI_TestRunner::FAILURE_EXIT);
            }
        }
       

[laurentj]

Fixed, thanks

​https://github.com/jelix/jelix/commit/5decee29538ab9bee05b67c1e701b73caedd1dc6