alxarafe / alixar

dolibarr/test/phpunit/UserGroupTest.php

Indentation +3 added lines, -3 removed lines

@@ -59,9 +59,9 @@ 
      */
     function __construct()
     {
-    	parent::__construct();
+        parent::__construct();
 
-    	//$this->sharedFixture
+        //$this->sharedFixture
         global $conf,$user,$langs,$db;
         $this->savconf=$conf;
         $this->savuser=$user;
@@ -95,7 +95,7 @@ 
      * Init phpunit tests
      *
      * @return  void
-    */
+     */
     protected function setUp()
     {
         global $conf,$user,$langs,$db;

dolibarr/test/phpunit/functional/InstallTest.php

Indentation +153 added lines, -153 removed lines

@@ -23,263 +23,263 @@
  */
 class InstallTest extends PHPUnit_Extensions_Selenium2TestCase
 {
-	protected static $url = 'https://dev.dolibarr.org';
-	protected static $db_name = 'dolibarr_test';
-	protected static $db_host = 'localhost';
-	protected static $db_admin_user = 'root';
-	protected static $db_admin_pass = '';
-	protected static $db_user = 'dolibarr';
-	protected static $db_pass = 'dolibarr';
-	protected static $dol_admin_user = 'admin';
-	protected static $dol_admin_pass = 'admin';
-
-	public static $browsers = array(
-		array(
-			'browser' => 'chrome',
-			'browserName' => 'chrome',
-			'sessionStrategy' => 'shared',
-			'desiredCapabilities' => array()
-		)
-	);
+    protected static $url = 'https://dev.dolibarr.org';
+    protected static $db_name = 'dolibarr_test';
+    protected static $db_host = 'localhost';
+    protected static $db_admin_user = 'root';
+    protected static $db_admin_pass = '';
+    protected static $db_user = 'dolibarr';
+    protected static $db_pass = 'dolibarr';
+    protected static $dol_admin_user = 'admin';
+    protected static $dol_admin_pass = 'admin';
+
+    public static $browsers = array(
+        array(
+            'browser' => 'chrome',
+            'browserName' => 'chrome',
+            'sessionStrategy' => 'shared',
+            'desiredCapabilities' => array()
+        )
+    );
 
     /**
      * setUpBeforeClass
      *
      * @return	void
      */
-	public static function setUpBeforeClass()
-	{
-		// Make sure we backup and remove the configuration file to force new install.
-		@rename('htdocs/conf/conf.php', sys_get_temp_dir() . '/conf.php');
+    public static function setUpBeforeClass()
+    {
+        // Make sure we backup and remove the configuration file to force new install.
+        @rename('htdocs/conf/conf.php', sys_get_temp_dir() . '/conf.php');
 
-		// Start without a database
-		self::dropTestDatabase();
+        // Start without a database
+        self::dropTestDatabase();
 
-		// Run the tests in the same window
-		self::shareSession(true);
-	}
+        // Run the tests in the same window
+        self::shareSession(true);
+    }
 
     /**
      * dropTestDatabase
      *
      * @return	void
      */
-	protected static function dropTestDatabase()
-	{
-		$mysqli = new mysqli(self::$db_host, self::$db_admin_user, self::$db_admin_pass);
-		$mysqli->query("DROP DATABASE " . self::$db_name);
-	}
+    protected static function dropTestDatabase()
+    {
+        $mysqli = new mysqli(self::$db_host, self::$db_admin_user, self::$db_admin_pass);
+        $mysqli->query("DROP DATABASE " . self::$db_name);
+    }
 
     /**
      * tearDownAfterClass
      *
      * @return	void
      */
-	public static function tearDownAfterClass()
-	{
-		// Remove the generated configuration and restore the backed up file.
-		@unlink('htdocs/conf/conf.php');
-		@rename(sys_get_temp_dir() . '/conf.php', 'htdocs/conf/conf.php');
+    public static function tearDownAfterClass()
+    {
+        // Remove the generated configuration and restore the backed up file.
+        @unlink('htdocs/conf/conf.php');
+        @rename(sys_get_temp_dir() . '/conf.php', 'htdocs/conf/conf.php');
 
-		// Cleanup test database
-		self::dropTestDatabase();
-	}
+        // Cleanup test database
+        self::dropTestDatabase();
+    }
 
     /**
      * setUp
      *
      * @return  void
      */
-	public function setUp()
-	{
-		// Populating the database can take quite long.
-		$this->setSeleniumServerRequestsTimeout(120000);
-		$this->setBrowserUrl(self::$url);
-	}
+    public function setUp()
+    {
+        // Populating the database can take quite long.
+        $this->setSeleniumServerRequestsTimeout(120000);
+        $this->setBrowserUrl(self::$url);
+    }
 
     /**
      * testInstallRedirect
      *
      * @return  void
      */
-	public function testInstallRedirect()
-	{
-		$this->url('/');
-		$this->assertContains('/install/index.php', $this->url());
-	}
+    public function testInstallRedirect()
+    {
+        $this->url('/');
+        $this->assertContains('/install/index.php', $this->url());
+    }
 
     /**
      * testInstallPageTitle
      *
      * @return  void
      */
-	public function testInstallPageTitle()
-	{
-		$this->assertContains('Dolibarr', $this->title());
-	}
+    public function testInstallPageTitle()
+    {
+        $this->assertContains('Dolibarr', $this->title());
+    }
 
     /**
      * testInstallProcess
      *
      * @return  void
      */
-	public function testInstallProcess()
-	{
-		// FIXME: the button itself should have an ID
-		$this->byId('nextbutton')->byTag('input')->click();
-		$this->assertContains('/install/check.php', $this->url());
-	}
+    public function testInstallProcess()
+    {
+        // FIXME: the button itself should have an ID
+        $this->byId('nextbutton')->byTag('input')->click();
+        $this->assertContains('/install/check.php', $this->url());
+    }
 
     /**
      * testCheckPage
      *
      * @return  void
      */
-	public function testCheckPage()
-	{
-		$unavailable_choices = $this->byId('navail_choices');
-		$show_hide_choices = $this->byId('AShowChoices')->byTag('a');
-		$this->assertFalse($unavailable_choices->displayed());
-		// FIXME: the link itself should have an ID
-		$show_hide_choices->click();
-		$this->assertTrue($unavailable_choices->displayed());
-		$show_hide_choices->click();
-		$this->assertFalse($unavailable_choices->displayed());
-		$this->byClassName('button')->click();
-		$this->assertContains('/install/fileconf.php', $this->url());
-	}
+    public function testCheckPage()
+    {
+        $unavailable_choices = $this->byId('navail_choices');
+        $show_hide_choices = $this->byId('AShowChoices')->byTag('a');
+        $this->assertFalse($unavailable_choices->displayed());
+        // FIXME: the link itself should have an ID
+        $show_hide_choices->click();
+        $this->assertTrue($unavailable_choices->displayed());
+        $show_hide_choices->click();
+        $this->assertFalse($unavailable_choices->displayed());
+        $this->byClassName('button')->click();
+        $this->assertContains('/install/fileconf.php', $this->url());
+    }
 
     /**
      * testForm
      *
      * @return  void
      */
-	public function testForm()
-	{
-		$this->assertFalse($this->byClassName('hideroot')->displayed());
-		$this->assertTrue($this->byClassName('hidesqlite')->displayed());
+    public function testForm()
+    {
+        $this->assertFalse($this->byClassName('hideroot')->displayed());
+        $this->assertTrue($this->byClassName('hidesqlite')->displayed());
 
-		// FIXME: This element should have an ID
-		$this->assertFalse($this->byName('main_force_https')->selected());
-		$this->byName('main_force_https')->click();
+        // FIXME: This element should have an ID
+        $this->assertFalse($this->byName('main_force_https')->selected());
+        $this->byName('main_force_https')->click();
 
-		$this->assertEquals('dolibarr', $this->byId('db_name')->value());
-		$this->byId('db_name')->clear();
-		$this->byId('db_name')->value(self::$db_name);
+        $this->assertEquals('dolibarr', $this->byId('db_name')->value());
+        $this->byId('db_name')->clear();
+        $this->byId('db_name')->value(self::$db_name);
 
-		$this->assertEquals('mysqli', $this->byId('db_type')->value());
+        $this->assertEquals('mysqli', $this->byId('db_type')->value());
 
-		// FIXME: This element should have an ID
-		$this->assertEquals('localhost', $this->byName('db_host')->value());
+        // FIXME: This element should have an ID
+        $this->assertEquals('localhost', $this->byName('db_host')->value());
 
-		$this->assertEquals(3306, $this->byId('db_port')->value());
+        $this->assertEquals(3306, $this->byId('db_port')->value());
 
-		$this->assertEquals('llx_', $this->byId('db_prefix')->value());
+        $this->assertEquals('llx_', $this->byId('db_prefix')->value());
 
-		$this->byId('db_create_database')->click();
-		$this->assertTrue($this->byClassName('hideroot')->displayed());
-		$this->byId('db_create_database')->click();
-		$this->assertFalse($this->byClassName('hideroot')->displayed());
+        $this->byId('db_create_database')->click();
+        $this->assertTrue($this->byClassName('hideroot')->displayed());
+        $this->byId('db_create_database')->click();
+        $this->assertFalse($this->byClassName('hideroot')->displayed());
 
-		$this->byId('db_user')->value(self::$db_user);
+        $this->byId('db_user')->value(self::$db_user);
 
-		$this->byId('db_pass')->value(self::$db_pass);
+        $this->byId('db_pass')->value(self::$db_pass);
 
-		$this->byId('db_create_user')->click();
-		$this->assertTrue($this->byClassName('hideroot')->displayed());
-		$this->byId('db_create_user')->click();
-		$this->assertFalse($this->byClassName('hideroot')->displayed());
+        $this->byId('db_create_user')->click();
+        $this->assertTrue($this->byClassName('hideroot')->displayed());
+        $this->byId('db_create_user')->click();
+        $this->assertFalse($this->byClassName('hideroot')->displayed());
 
-		$this->byId('db_create_database')->click();
-		$this->byId('db_create_user')->click();
-		$this->assertTrue($this->byClassName('hideroot')->displayed());
+        $this->byId('db_create_database')->click();
+        $this->byId('db_create_user')->click();
+        $this->assertTrue($this->byClassName('hideroot')->displayed());
 
-		$this->byId('db_user_root')->value('root');
-		$this->byId('db_pass_root')->value('');
-	}
+        $this->byId('db_user_root')->value('root');
+        $this->byId('db_pass_root')->value('');
+    }
 
     /**
      * testFormSubmit
      *
      * @return  void
      */
-	public function testFormSubmit()
-	{
-		$this->byName('forminstall')->submit();
-		$this->assertContains('/install/step1.php', $this->url());
-	}
+    public function testFormSubmit()
+    {
+        $this->byName('forminstall')->submit();
+        $this->assertContains('/install/step1.php', $this->url());
+    }
 
     /**
      * testStep1
      *
      * @return  void
      */
-	public function testStep1()
-	{
-		$this->assertFalse($this->byId('pleasewait')->displayed());
-		$start = new DateTimeImmutable();
-		// FIXME: the button itself should have an ID
-		$this->byId('nextbutton')->byTag('input')->click();
-		$time = $start->diff(new DateTimeImmutable());
-		echo "\nPopulating the database took " . $time->format("%s seconds.\n");
-		$this->assertContains('/install/step2.php', $this->url());
-	}
+    public function testStep1()
+    {
+        $this->assertFalse($this->byId('pleasewait')->displayed());
+        $start = new DateTimeImmutable();
+        // FIXME: the button itself should have an ID
+        $this->byId('nextbutton')->byTag('input')->click();
+        $time = $start->diff(new DateTimeImmutable());
+        echo "\nPopulating the database took " . $time->format("%s seconds.\n");
+        $this->assertContains('/install/step2.php', $this->url());
+    }
 
     /**
      * testStep2
      *
      * @return  void
      */
-	public function testStep2()
-	{
-		$this->byName('forminstall')->submit();
-		$this->assertContains('/install/step4.php', $this->url());
-	}
+    public function testStep2()
+    {
+        $this->byName('forminstall')->submit();
+        $this->assertContains('/install/step4.php', $this->url());
+    }
 
-	// There is no step 3
+    // There is no step 3
 
     /**
      * testStep4
      *
      * @return  void
      */
-	public function testStep4()
-	{
-		// FIXME: should have an ID
-		$this->byName('login')->value(self::$dol_admin_user);
-		// FIXME: should have an ID
-		$this->byName('pass')->value('admin');
-		// FIXME: should have an ID
-		$this->byName('pass_verif')->value(self::$dol_admin_pass);
-		// FIXME: the button itself should have an ID
-		$this->byId('nextbutton')->byTag('input')->click();
-		$this->assertContains('/install/step5.php', $this->url());
-	}
+    public function testStep4()
+    {
+        // FIXME: should have an ID
+        $this->byName('login')->value(self::$dol_admin_user);
+        // FIXME: should have an ID
+        $this->byName('pass')->value('admin');
+        // FIXME: should have an ID
+        $this->byName('pass_verif')->value(self::$dol_admin_pass);
+        // FIXME: the button itself should have an ID
+        $this->byId('nextbutton')->byTag('input')->click();
+        $this->assertContains('/install/step5.php', $this->url());
+    }
 
     /**
      * testStep5
      *
      * @return  void
      */
-	public function testStep5()
-	{
-		// FIXME: this button should have an ID
-		$this->byTag('a')->click();
-		$this->assertContains('/admin/index.php', $this->url());
-	}
+    public function testStep5()
+    {
+        // FIXME: this button should have an ID
+        $this->byTag('a')->click();
+        $this->assertContains('/admin/index.php', $this->url());
+    }
 
     /**
      * testFirstLogin
      *
      * @return  void
      */
-	public function testFirstLogin()
-	{
-		$this->assertEquals('login', $this->byTag('form')->attribute('id'));
-		$this->assertEquals(self::$dol_admin_user, $this->byId('username')->value());
-		$this->byId('password')->value(self::$dol_admin_pass);
-		// FIXME: login button should have an ID
-		$this->byId('login')->submit();
-		$this->assertEquals('mainbody', $this->byTag('body')->attribute('id'));
-	}
+    public function testFirstLogin()
+    {
+        $this->assertEquals('login', $this->byTag('form')->attribute('id'));
+        $this->assertEquals(self::$dol_admin_user, $this->byId('username')->value());
+        $this->byId('password')->value(self::$dol_admin_pass);
+        // FIXME: login button should have an ID
+        $this->byId('login')->submit();
+        $this->assertEquals('mainbody', $this->byTag('body')->attribute('id'));
+    }
 }

dolibarr/test/phpunit/functional/TakePosFunctionalTest.php

Indentation +248 added lines, -248 removed lines

@@ -41,32 +41,32 @@ 
  */
 class TakePosFunctionalTest extends \PHPUnit_Extensions_Selenium2TestCase
 {
-	// TODO: move to a global configuration file?
-	/** @var string Base URL of the webserver under test */
-	protected static $base_url = 'http://dev.zenfusion.fr';
-	/**
-	 * @var string Dolibarr admin username
-	 * @see authenticate
-	 */
-	protected static $dol_admin_user = 'admin';
-	/**
-	 * @var string Dolibarr admin password
-	 * @see authenticate
-	 */
-	protected static $dol_admin_pass = 'admin';
-	/** @var int Dolibarr module ID */
-	private static $module_id = 500000; // TODO: autodetect?
+    // TODO: move to a global configuration file?
+    /** @var string Base URL of the webserver under test */
+    protected static $base_url = 'http://dev.zenfusion.fr';
+    /**
+     * @var string Dolibarr admin username
+     * @see authenticate
+     */
+    protected static $dol_admin_user = 'admin';
+    /**
+     * @var string Dolibarr admin password
+     * @see authenticate
+     */
+    protected static $dol_admin_pass = 'admin';
+    /** @var int Dolibarr module ID */
+    private static $module_id = 500000; // TODO: autodetect?
 
-	/** @var array Browsers to test with */
-	public static $browsers = array(
-		array(
-			'browser' => 'Google Chrome on Linux',
-			'browserName' => 'chrome',
-			'sessionStrategy' => 'shared',
-			'desiredCapabilities' => array()
-		),
-		// Geckodriver does not keep the session at the moment?!
-		// XPath selectors also don't seem to work
+    /** @var array Browsers to test with */
+    public static $browsers = array(
+        array(
+            'browser' => 'Google Chrome on Linux',
+            'browserName' => 'chrome',
+            'sessionStrategy' => 'shared',
+            'desiredCapabilities' => array()
+        ),
+        // Geckodriver does not keep the session at the moment?!
+        // XPath selectors also don't seem to work
 //        array(
 //            'browser' => 'Mozilla Firefox on Linux',
 //            'browserName' => 'firefox',
@@ -75,243 +75,243 @@ 
 //                'marionette' => true
 //            )
 //        )
-	);
+    );
 
-	/**
-	 * Helper function to select links by href
-	 *
-	 * @param  string  $value      Href
-	 * @return mixed               Helper string
-	 */
-	protected function byHref($value)
-	{
-		$anchor = null;
-		$anchors = $this->elements($this->using('tag name')->value('a'));
-		foreach ($anchors as $anchor) {
-			if (strstr($anchor->attribute('href'), $value)) {
-				break;
-			}
-		}
-		return $anchor;
-	}
+    /**
+     * Helper function to select links by href
+     *
+     * @param  string  $value      Href
+     * @return mixed               Helper string
+     */
+    protected function byHref($value)
+    {
+        $anchor = null;
+        $anchors = $this->elements($this->using('tag name')->value('a'));
+        foreach ($anchors as $anchor) {
+            if (strstr($anchor->attribute('href'), $value)) {
+                break;
+            }
+        }
+        return $anchor;
+    }
 
-	/**
-	 * Global test setup
-	 *
-	 * @return	void
-	 */
-	public static function setUpBeforeClass()
-	{
-	}
+    /**
+     * Global test setup
+     *
+     * @return	void
+     */
+    public static function setUpBeforeClass()
+    {
+    }
 
-	/**
-	 * Unit test setup
-	 *
-	 * @return	void
-	 */
-	public function setUp()
-	{
-		$this->setSeleniumServerRequestsTimeout(3600);
-		$this->setBrowserUrl(self::$base_url);
-	}
+    /**
+     * Unit test setup
+     *
+     * @return	void
+     */
+    public function setUp()
+    {
+        $this->setSeleniumServerRequestsTimeout(3600);
+        $this->setBrowserUrl(self::$base_url);
+    }
 
-	/**
-	 * Verify pre conditions
-	 *
-	 * @return	void
-	 */
-	protected function assertPreConditions()
-	{
-	}
+    /**
+     * Verify pre conditions
+     *
+     * @return	void
+     */
+    protected function assertPreConditions()
+    {
+    }
 
-	/**
-	 * Handle Dolibarr authentication
-	 *
-	 * @return	void
-	 */
-	private function authenticate()
-	{
-		try {
-			if ($this->byId('login')) {
-				$login = $this->byId('username');
-				$login->clear();
-				$login->value('admin');
-				$password = $this->byId('password');
-				$password->clear();
-				$password->value('admin');
-				$this->byId('login')->submit();
-			}
-		} catch (PHPUnit_Extensions_Selenium2TestCase_WebDriverException $e) {
-			// Login does not exist. Assume we are already authenticated
-		}
-	}
+    /**
+     * Handle Dolibarr authentication
+     *
+     * @return	void
+     */
+    private function authenticate()
+    {
+        try {
+            if ($this->byId('login')) {
+                $login = $this->byId('username');
+                $login->clear();
+                $login->value('admin');
+                $password = $this->byId('password');
+                $password->clear();
+                $password->value('admin');
+                $this->byId('login')->submit();
+            }
+        } catch (PHPUnit_Extensions_Selenium2TestCase_WebDriverException $e) {
+            // Login does not exist. Assume we are already authenticated
+        }
+    }
 
-	/**
-	 * Test enabling developer mode
-	 *
-	 * @return	void
-	 */
-	public function testEnableDeveloperMode()
-	{
-		$this->url('/admin/const.php');
-		$this->authenticate();
-		$main_features_level_path='//input[@value="MAIN_FEATURES_LEVEL"]/following::input[@type="text"]';
-		$main_features_level = $this->byXPath($main_features_level_path);
-		$main_features_level->clear();
-		$main_features_level->value('2');
-		$this->byName('update')->click();
-		// Page reloaded, we need a new XPath
-		$main_features_level = $this->byXPath($main_features_level_path);
-		$this->assertEquals('2', $main_features_level->value(), "MAIN_FEATURES_LEVEL value is 2");
-	}
+    /**
+     * Test enabling developer mode
+     *
+     * @return	void
+     */
+    public function testEnableDeveloperMode()
+    {
+        $this->url('/admin/const.php');
+        $this->authenticate();
+        $main_features_level_path='//input[@value="MAIN_FEATURES_LEVEL"]/following::input[@type="text"]';
+        $main_features_level = $this->byXPath($main_features_level_path);
+        $main_features_level->clear();
+        $main_features_level->value('2');
+        $this->byName('update')->click();
+        // Page reloaded, we need a new XPath
+        $main_features_level = $this->byXPath($main_features_level_path);
+        $this->assertEquals('2', $main_features_level->value(), "MAIN_FEATURES_LEVEL value is 2");
+    }
 
-	/**
-	 * Test enabling the module
-	 *
-	 * @return	void
-	 *
-	 * @depends testEnableDeveloperMode
-	 */
-	public function testModuleEnabled()
-	{
-		$this->url('/admin/modules.php');
-		$this->authenticate();
-		$module_status_image_path='//a[contains(@href, "' . self::$module_id . '")]/img';
-		$module_status_image = $this->byXPath($module_status_image_path);
-		if (strstr($module_status_image->attribute('src'), 'switch_off.png')) {
-			// Enable the module
-			$this->byHref('modTakePos')->click();
-		} else {
-			// Disable the module
-			$this->byHref('modTakePos')->click();
-			// Reenable the module
-			$this->byHref('modTakePos')->click();
-		}
-		// Page reloaded, we need a new Xpath
-		$module_status_image = $this->byXPath($module_status_image_path);
-		$this->assertContains('switch_on.png', $module_status_image->attribute('src'), "Module enabled");
-	}
+    /**
+     * Test enabling the module
+     *
+     * @return	void
+     *
+     * @depends testEnableDeveloperMode
+     */
+    public function testModuleEnabled()
+    {
+        $this->url('/admin/modules.php');
+        $this->authenticate();
+        $module_status_image_path='//a[contains(@href, "' . self::$module_id . '")]/img';
+        $module_status_image = $this->byXPath($module_status_image_path);
+        if (strstr($module_status_image->attribute('src'), 'switch_off.png')) {
+            // Enable the module
+            $this->byHref('modTakePos')->click();
+        } else {
+            // Disable the module
+            $this->byHref('modTakePos')->click();
+            // Reenable the module
+            $this->byHref('modTakePos')->click();
+        }
+        // Page reloaded, we need a new Xpath
+        $module_status_image = $this->byXPath($module_status_image_path);
+        $this->assertContains('switch_on.png', $module_status_image->attribute('src'), "Module enabled");
+    }
 
-	/**
-	 * Test access to the configuration page
-	 *
-	 * @return	void
-	 *
-	 * @depends testModuleEnabled
-	 */
-	public function testConfigurationPage()
-	{
-		$this->url('/custom/takepos/admin/setup.php');
-		$this->authenticate();
-		$this->assertContains('takepos/admin/setup.php', $this->url(), 'Configuration page');
-	}
+    /**
+     * Test access to the configuration page
+     *
+     * @return	void
+     *
+     * @depends testModuleEnabled
+     */
+    public function testConfigurationPage()
+    {
+        $this->url('/custom/takepos/admin/setup.php');
+        $this->authenticate();
+        $this->assertContains('takepos/admin/setup.php', $this->url(), 'Configuration page');
+    }
 
-	/**
-	 * Test access to the about page
-	 *
-	 * @return	void
-	 *
-	 * @depends testConfigurationPage
-	 */
-	public function testAboutPage()
-	{
-		$this->url('/custom/takepos/admin/about.php');
-		$this->authenticate();
-		$this->assertContains('takepos/admin/about.php', $this->url(), 'About page');
-	}
+    /**
+     * Test access to the about page
+     *
+     * @return	void
+     *
+     * @depends testConfigurationPage
+     */
+    public function testAboutPage()
+    {
+        $this->url('/custom/takepos/admin/about.php');
+        $this->authenticate();
+        $this->assertContains('takepos/admin/about.php', $this->url(), 'About page');
+    }
 
-	/**
-	 * Test about page is rendering Markdown
-	 *
-	 * @return	void
-	 *
-	 * @depends testAboutPage
-	 */
-	public function testAboutPageRendersMarkdownReadme()
-	{
-		$this->url('/custom/takepos/admin/about.php');
-		$this->authenticate();
-		$this->assertEquals(
-			'Dolibarr Module Template (aka My Module)',
-			$this->byTag('h1')->text(),
-			"Readme title"
-		);
-	}
+    /**
+     * Test about page is rendering Markdown
+     *
+     * @return	void
+     *
+     * @depends testAboutPage
+     */
+    public function testAboutPageRendersMarkdownReadme()
+    {
+        $this->url('/custom/takepos/admin/about.php');
+        $this->authenticate();
+        $this->assertEquals(
+            'Dolibarr Module Template (aka My Module)',
+            $this->byTag('h1')->text(),
+            "Readme title"
+        );
+    }
 
-	/**
-	 * Test box is properly declared
-	 *
-	 * @return	void
-	 *
-	 * @depends testModuleEnabled
-	 */
-	public function testBoxDeclared()
-	{
-		$this->url('/admin/boxes.php');
-		$this->authenticate();
-		$this->assertContains('takeposwidget1', $this->source(), "Box enabled");
-	}
+    /**
+     * Test box is properly declared
+     *
+     * @return	void
+     *
+     * @depends testModuleEnabled
+     */
+    public function testBoxDeclared()
+    {
+        $this->url('/admin/boxes.php');
+        $this->authenticate();
+        $this->assertContains('takeposwidget1', $this->source(), "Box enabled");
+    }
 
-	/**
-	 * Test trigger is properly enabled
-	 *
-	 * @return	void
-	 *
-	 * @depends testModuleEnabled
-	 */
-	public function testTriggerDeclared()
-	{
-		$this->url('/admin/triggers.php');
-		$this->authenticate();
-		$this->assertContains(
-			'interface_99_modTakePos_TakePosTriggers.class.php',
-			$this->byTag('body')->text(),
-			"Trigger declared"
-		);
-	}
+    /**
+     * Test trigger is properly enabled
+     *
+     * @return	void
+     *
+     * @depends testModuleEnabled
+     */
+    public function testTriggerDeclared()
+    {
+        $this->url('/admin/triggers.php');
+        $this->authenticate();
+        $this->assertContains(
+            'interface_99_modTakePos_TakePosTriggers.class.php',
+            $this->byTag('body')->text(),
+            "Trigger declared"
+        );
+    }
 
-	/**
-	 * Test trigger is properly declared
-	 *
-	 * @return	void
-	 *
-	 * @depends testTriggerDeclared
-	 */
-	public function testTriggerEnabled()
-	{
-		$this->url('/admin/triggers.php');
-		$this->authenticate();
-		$this->assertContains(
-			'tick.png',
-			$this->byXPath('//td[text()="interface_99_modTakePos_MyTrigger.class.php"]/following::img')->attribute('src'),
-			"Trigger enabled"
-		);
-	}
+    /**
+     * Test trigger is properly declared
+     *
+     * @return	void
+     *
+     * @depends testTriggerDeclared
+     */
+    public function testTriggerEnabled()
+    {
+        $this->url('/admin/triggers.php');
+        $this->authenticate();
+        $this->assertContains(
+            'tick.png',
+            $this->byXPath('//td[text()="interface_99_modTakePos_MyTrigger.class.php"]/following::img')->attribute('src'),
+            "Trigger enabled"
+        );
+    }
 
-	/**
-	 * Verify post conditions
-	 *
-	 * @return	void
-	 */
-	protected function assertPostConditions()
-	{
-	}
+    /**
+     * Verify post conditions
+     *
+     * @return	void
+     */
+    protected function assertPostConditions()
+    {
+    }
 
-	/**
-	 * Unit test teardown
-	 *
-	 * @return	void
-	 */
-	public function tearDown()
-	{
-	}
+    /**
+     * Unit test teardown
+     *
+     * @return	void
+     */
+    public function tearDown()
+    {
+    }
 
-	/**
-	 * Global test teardown
-	 *
-	 * @return	void
-	 */
-	public static function tearDownAfterClass()
-	{
-	}
+    /**
+     * Global test teardown
+     *
+     * @return	void
+     */
+    public static function tearDownAfterClass()
+    {
+    }
 }

dolibarr/test/phpunit/CategorieTest.php

Indentation +2 added lines, -3 removed lines

@@ -60,9 +60,9 @@ 
      */
     function __construct()
     {
-    	parent::__construct();
+        parent::__construct();
 
-    	//$this->sharedFixture
+        //$this->sharedFixture
         global $conf,$user,$langs,$db;
         $this->savconf=$conf;
         $this->savuser=$user;
@@ -238,7 +238,6 @@ 
      *
      * @param   Category        $localobject        Category
      * @return  int
-
      * @depends testCategorieFetch
      * The depends says test is run only if previous is ok
      */

dolibarr/test/phpunit/FormAdminTest.php

Indentation +68 added lines, -68 removed lines

@@ -31,9 +31,9 @@ 
 
 if (empty($user->id))
 {
-	print "Load permissions for admin user nb 1\n";
-	$user->fetch(1);
-	$user->getrights();
+    print "Load permissions for admin user nb 1\n";
+    $user->fetch(1);
+    $user->getrights();
 }
 $conf->global->MAIN_DISABLE_ALL_MAILS=1;
 
@@ -47,75 +47,75 @@ 
  */
 class FormAdminTest extends PHPUnit_Framework_TestCase
 {
-	protected $savconf;
-	protected $savuser;
-	protected $savlangs;
-	protected $savdb;
-
-	/**
-	 * Constructor
-	 * We save global variables into local variables
-	 *
-	 * @return FactureTest
-	 */
-	function __construct()
-	{
-		parent::__construct();
-
-		//$this->sharedFixture
-		global $conf,$user,$langs,$db;
-		$this->savconf=$conf;
-		$this->savuser=$user;
-		$this->savlangs=$langs;
-		$this->savdb=$db;
-
-		print __METHOD__." db->type=".$db->type." user->id=".$user->id;
-		//print " - db ".$db->db;
-		print "\n";
-	}
-
-	// Static methods
-  	public static function setUpBeforeClass()
+    protected $savconf;
+    protected $savuser;
+    protected $savlangs;
+    protected $savdb;
+
+    /**
+     * Constructor
+     * We save global variables into local variables
+     *
+     * @return FactureTest
+     */
+    function __construct()
+    {
+        parent::__construct();
+
+        //$this->sharedFixture
+        global $conf,$user,$langs,$db;
+        $this->savconf=$conf;
+        $this->savuser=$user;
+        $this->savlangs=$langs;
+        $this->savdb=$db;
+
+        print __METHOD__." db->type=".$db->type." user->id=".$user->id;
+        //print " - db ".$db->db;
+        print "\n";
+    }
+
+    // Static methods
+        public static function setUpBeforeClass()
     {
-    	global $conf,$user,$langs,$db;
-		$db->begin();	// This is to have all actions inside a transaction even if test launched without suite.
+        global $conf,$user,$langs,$db;
+        $db->begin();	// This is to have all actions inside a transaction even if test launched without suite.
 
-    	print __METHOD__."\n";
+        print __METHOD__."\n";
     }
 
     // tear down after class
     public static function tearDownAfterClass()
     {
-    	global $conf,$user,$langs,$db;
-		$db->rollback();
+        global $conf,$user,$langs,$db;
+        $db->rollback();
 
-		print __METHOD__."\n";
+        print __METHOD__."\n";
     }
 
-	/**
-	 * Init phpunit tests
-	 *
-	 * @return	void
-	 */
+    /**
+     * Init phpunit tests
+     *
+     * @return	void
+     */
     protected function setUp()
     {
-    	global $conf,$user,$langs,$db;
-		$conf=$this->savconf;
-		$user=$this->savuser;
-		$langs=$this->savlangs;
-		$db=$this->savdb;
+        global $conf,$user,$langs,$db;
+        $conf=$this->savconf;
+        $user=$this->savuser;
+        $langs=$this->savlangs;
+        $db=$this->savdb;
 
-		print __METHOD__."\n";
+        print __METHOD__."\n";
     }
 
-	/**
-	 * End phpunit tests
-	 *
-	 * @return	void
-	 */
-	protected function tearDown()
+    /**
+     * End phpunit tests
+     *
+     * @return	void
+     */
+    protected function tearDown()
     {
-    	print __METHOD__."\n";
+        print __METHOD__."\n";
     }
 
     /**
@@ -125,17 +125,17 @@ 
      */
     public function testSelectPaperFormat()
     {
-    	global $conf,$user,$langs,$db;
-		$conf=$this->savconf;
-		$user=$this->savuser;
-		$langs=$this->savlangs;
-		$db=$this->savdb;
-
-		$localobject=new FormAdmin($this->savdb);
-    	$result=$localobject->select_paper_format('','paperformat_id','A4');
-
-    	$this->assertEquals($result, '<select class="flat" id="paperformat_id" name="paperformat_id"><option value="EUA4">Format A4 - 210x297 mm</option></select>');
-    	print __METHOD__." result=".$result."\n";
-    	return $result;
+        global $conf,$user,$langs,$db;
+        $conf=$this->savconf;
+        $user=$this->savuser;
+        $langs=$this->savlangs;
+        $db=$this->savdb;
+
+        $localobject=new FormAdmin($this->savdb);
+        $result=$localobject->select_paper_format('','paperformat_id','A4');
+
+        $this->assertEquals($result, '<select class="flat" id="paperformat_id" name="paperformat_id"><option value="EUA4">Format A4 - 210x297 mm</option></select>');
+        print __METHOD__." result=".$result."\n";
+        return $result;
     }
 }

dolibarr/test/phpunit/ProjectTest.php

Indentation +107 added lines, -107 removed lines

@@ -32,9 +32,9 @@ 
 
 if (empty($user->id))
 {
-	print "Load permissions for admin user nb 1\n";
-	$user->fetch(1);
-	$user->getrights();
+    print "Load permissions for admin user nb 1\n";
+    $user->fetch(1);
+    $user->getrights();
 }
 $conf->global->MAIN_DISABLE_ALL_MAILS=1;
 
@@ -48,75 +48,75 @@ 
  */
 class ProjectTest extends PHPUnit_Framework_TestCase
 {
-	protected $savconf;
-	protected $savuser;
-	protected $savlangs;
-	protected $savdb;
-
-	/**
-	 * Constructor
-	 * We save global variables into local variables
-	 *
-	 * @return ProjectTest
-	 */
-	function __construct()
-	{
-		parent::__construct();
-
-		//$this->sharedFixture
-		global $conf,$user,$langs,$db;
-		$this->savconf=$conf;
-		$this->savuser=$user;
-		$this->savlangs=$langs;
-		$this->savdb=$db;
-
-		print __METHOD__." db->type=".$db->type." user->id=".$user->id;
-		//print " - db ".$db->db;
-		print "\n";
-	}
-
-	// Static methods
-  	public static function setUpBeforeClass()
+    protected $savconf;
+    protected $savuser;
+    protected $savlangs;
+    protected $savdb;
+
+    /**
+     * Constructor
+     * We save global variables into local variables
+     *
+     * @return ProjectTest
+     */
+    function __construct()
     {
-    	global $conf,$user,$langs,$db;
-		$db->begin();	// This is to have all actions inside a transaction even if test launched without suite.
+        parent::__construct();
 
-    	print __METHOD__."\n";
+        //$this->sharedFixture
+        global $conf,$user,$langs,$db;
+        $this->savconf=$conf;
+        $this->savuser=$user;
+        $this->savlangs=$langs;
+        $this->savdb=$db;
+
+        print __METHOD__." db->type=".$db->type." user->id=".$user->id;
+        //print " - db ".$db->db;
+        print "\n";
+    }
+
+    // Static methods
+        public static function setUpBeforeClass()
+    {
+        global $conf,$user,$langs,$db;
+        $db->begin();	// This is to have all actions inside a transaction even if test launched without suite.
+
+        print __METHOD__."\n";
     }
 
     // tear down after class
     public static function tearDownAfterClass()
     {
-    	global $conf,$user,$langs,$db;
-		$db->rollback();
+        global $conf,$user,$langs,$db;
+        $db->rollback();
 
-		print __METHOD__."\n";
+        print __METHOD__."\n";
     }
 
-	/**
-	 * Init phpunit tests
-	 *
-	 * @return	void
-	 */
+    /**
+     * Init phpunit tests
+     *
+     * @return	void
+     */
     protected function setUp()
     {
-    	global $conf,$user,$langs,$db;
-		$conf=$this->savconf;
-		$user=$this->savuser;
-		$langs=$this->savlangs;
-		$db=$this->savdb;
-
-		print __METHOD__."\n";
-		//print $db->getVersion()."\n";
+        global $conf,$user,$langs,$db;
+        $conf=$this->savconf;
+        $user=$this->savuser;
+        $langs=$this->savlangs;
+        $db=$this->savdb;
+
+        print __METHOD__."\n";
+        //print $db->getVersion()."\n";
     }
-	/**
-	 * End phpunit tests
-	 *
-	 * @return	void
-	 */
+    /**
+     * End phpunit tests
+     *
+     * @return	void
+     */
     protected function tearDown()
     {
-    	print __METHOD__."\n";
+        print __METHOD__."\n";
     }
 
     /**
@@ -126,19 +126,19 @@ 
      */
     public function testProjectCreate()
     {
-    	global $conf,$user,$langs,$db;
-		$conf=$this->savconf;
-		$user=$this->savuser;
-		$langs=$this->savlangs;
-		$db=$this->savdb;
-
-		$localobject=new Project($this->savdb);
-    	$localobject->initAsSpecimen();
-    	$result=$localobject->create($user);
-
-    	$this->assertLessThan($result, 0);
-    	print __METHOD__." result=".$result."\n";
-    	return $result;
+        global $conf,$user,$langs,$db;
+        $conf=$this->savconf;
+        $user=$this->savuser;
+        $langs=$this->savlangs;
+        $db=$this->savdb;
+
+        $localobject=new Project($this->savdb);
+        $localobject->initAsSpecimen();
+        $result=$localobject->create($user);
+
+        $this->assertLessThan($result, 0);
+        print __METHOD__." result=".$result."\n";
+        return $result;
     }
 
     /**
@@ -152,18 +152,18 @@ 
      */
     public function testProjectFetch($id)
     {
-    	global $conf,$user,$langs,$db;
-		$conf=$this->savconf;
-		$user=$this->savuser;
-		$langs=$this->savlangs;
-		$db=$this->savdb;
-
-		$localobject=new Project($this->savdb);
-    	$result=$localobject->fetch($id);
-
-    	$this->assertLessThan($result, 0);
-    	print __METHOD__." id=".$id." result=".$result."\n";
-    	return $localobject;
+        global $conf,$user,$langs,$db;
+        $conf=$this->savconf;
+        $user=$this->savuser;
+        $langs=$this->savlangs;
+        $db=$this->savdb;
+
+        $localobject=new Project($this->savdb);
+        $result=$localobject->fetch($id);
+
+        $this->assertLessThan($result, 0);
+        print __METHOD__." id=".$id." result=".$result."\n";
+        return $localobject;
     }
 
     /**
@@ -177,20 +177,20 @@ 
      */
     public function testProjectValid($localobject)
     {
-    	global $conf,$user,$langs,$db;
-		$conf=$this->savconf;
-		$user=$this->savuser;
-		$langs=$this->savlangs;
-		$db=$this->savdb;
+        global $conf,$user,$langs,$db;
+        $conf=$this->savconf;
+        $user=$this->savuser;
+        $langs=$this->savlangs;
+        $db=$this->savdb;
 
-    	$result=$localobject->setValid($user);
+        $result=$localobject->setValid($user);
 
-    	print __METHOD__." id=".$localobject->id." result=".$result."\n";
-    	$this->assertLessThan($result, 0);
-    	return $localobject;
+        print __METHOD__." id=".$localobject->id." result=".$result."\n";
+        $this->assertLessThan($result, 0);
+        return $localobject;
     }
 
-	/**
+    /**
      * testProjectClose
      *
      * @param	Project	$localobject	Project
@@ -209,8 +209,8 @@ 
 
         $result=$localobject->setClose($user);
 
-    	print __METHOD__." id=".$localobject->id." result=".$result."\n";
-    	$this->assertLessThan($result, 0);
+        print __METHOD__." id=".$localobject->id." result=".$result."\n";
+        $this->assertLessThan($result, 0);
         return $localobject->id;
     }
 
@@ -225,18 +225,18 @@ 
      */
     public function testProjectDelete($id)
     {
-    	global $conf,$user,$langs,$db;
-		$conf=$this->savconf;
-		$user=$this->savuser;
-		$langs=$this->savlangs;
-		$db=$this->savdb;
-
-		$localobject=new Project($this->savdb);
-    	$result=$localobject->fetch($id);
-		$result=$localobject->delete($user);
-
-		print __METHOD__." id=".$id." result=".$result."\n";
-    	$this->assertLessThan($result, 0);
-    	return $result;
+        global $conf,$user,$langs,$db;
+        $conf=$this->savconf;
+        $user=$this->savuser;
+        $langs=$this->savlangs;
+        $db=$this->savdb;
+
+        $localobject=new Project($this->savdb);
+        $result=$localobject->fetch($id);
+        $result=$localobject->delete($user);
+
+        print __METHOD__." id=".$id." result=".$result."\n";
+        $this->assertLessThan($result, 0);
+        return $result;
     }
 }

dolibarr/test/phpunit/Functions2LibTest.php

Indentation +71 added lines, -71 removed lines

@@ -63,9 +63,9 @@ 
      */
     function __construct()
     {
-    	parent::__construct();
+        parent::__construct();
 
-    	//$this->sharedFixture
+        //$this->sharedFixture
         global $conf,$user,$langs,$db;
         $this->savconf=$conf;
         $this->savuser=$user;
@@ -95,11 +95,11 @@ 
         print __METHOD__."\n";
     }
 
-	/**
-	 * Init phpunit tests
-	 *
-	 * @return	void
-	 */
+    /**
+     * Init phpunit tests
+     *
+     * @return	void
+     */
     protected function setUp()
     {
         global $conf,$user,$langs,$db;
@@ -110,11 +110,11 @@ 
 
         print __METHOD__."\n";
     }
-	/**
-	 * End phpunit tests
-	 *
-	 * @return	void
-	 */
+    /**
+     * End phpunit tests
+     *
+     * @return	void
+     */
     protected function tearDown()
     {
         print __METHOD__."\n";
@@ -149,62 +149,62 @@ 
      */
     public function testIsValidUrl()
     {
-	    //Simple check
-	    $result = isValidUrl('http://google.com');
-	    $this->assertEquals(1, $result);
+        //Simple check
+        $result = isValidUrl('http://google.com');
+        $this->assertEquals(1, $result);
 
-	    $result = isValidUrl('goo=gle');	// This is good, it might be an alias of hostname
-	    $this->assertEquals(1, $result);
+        $result = isValidUrl('goo=gle');	// This is good, it might be an alias of hostname
+        $this->assertEquals(1, $result);
 
-	    //With scheme check
-    	$result = isValidUrl('http://www.google.com', 1);
-	    $this->assertEquals(1, $result);
+        //With scheme check
+        $result = isValidUrl('http://www.google.com', 1);
+        $this->assertEquals(1, $result);
 
-	    $result = isValidUrl('ftp://www.google.com', 1);
-	    $this->assertEquals(0, $result);
+        $result = isValidUrl('ftp://www.google.com', 1);
+        $this->assertEquals(0, $result);
 
-	    //With password check invalid. This test should be ko but currently it is not
-	    //$result = isValidUrl('http://user:password@http://www.google.com', 1, 1);
-	    //$this->assertEquals(0, $result);
+        //With password check invalid. This test should be ko but currently it is not
+        //$result = isValidUrl('http://user:password@http://www.google.com', 1, 1);
+        //$this->assertEquals(0, $result);
 
-	    //With password check valid
-	    $result = isValidUrl('http://user:password@www.google.com', 1, 1);
-	    $this->assertEquals(1, $result);
+        //With password check valid
+        $result = isValidUrl('http://user:password@www.google.com', 1, 1);
+        $this->assertEquals(1, $result);
 
-	    $result = isValidUrl('http://www.google.com', 1, 1);
-	    $this->assertEquals(0, $result);
+        $result = isValidUrl('http://www.google.com', 1, 1);
+        $this->assertEquals(0, $result);
 
-	    //With port check
-	    $result = isValidUrl('http://google.com:8080', 0, 0, 1);
-	    $this->assertEquals(1, $result);
+        //With port check
+        $result = isValidUrl('http://google.com:8080', 0, 0, 1);
+        $this->assertEquals(1, $result);
 
-	    $result = isValidUrl('http://google.com', 0, 0, 1);
-	    $this->assertEquals(0, $result);
+        $result = isValidUrl('http://google.com', 0, 0, 1);
+        $this->assertEquals(0, $result);
 
-	    //With path check
-	    $result = isValidUrl('http://google.com/search', 0, 0, 0, 1);
-	    $this->assertEquals(1, $result);
+        //With path check
+        $result = isValidUrl('http://google.com/search', 0, 0, 0, 1);
+        $this->assertEquals(1, $result);
 
-	    $result = isValidUrl('http://google.com', 0, 0, 0, 0);
-	    $this->assertEquals(1, $result);
+        $result = isValidUrl('http://google.com', 0, 0, 0, 0);
+        $this->assertEquals(1, $result);
 
-	    //With query check
-	    $result = isValidUrl('http://google.com/search?test=test', 0, 0, 0, 0, 1);
-	    $this->assertEquals(1, $result);
+        //With query check
+        $result = isValidUrl('http://google.com/search?test=test', 0, 0, 0, 0, 1);
+        $this->assertEquals(1, $result);
 
-	    //With query check
-	    $result = isValidUrl('http://google.com?test=test', 0, 0, 0, 0, 1);
-	    $this->assertEquals(1, $result);
+        //With query check
+        $result = isValidUrl('http://google.com?test=test', 0, 0, 0, 0, 1);
+        $this->assertEquals(1, $result);
 
-	    $result = isValidUrl('http://google.com', 0, 0, 0, 0, 1);
-	    $this->assertEquals(0, $result);
+        $result = isValidUrl('http://google.com', 0, 0, 0, 0, 1);
+        $this->assertEquals(0, $result);
 
-	    //With anchor check
-	    $result = isValidUrl('http://google.com/search#done', 0, 0, 0, 0, 0, 1);
-	    $this->assertEquals(1, $result);
+        //With anchor check
+        $result = isValidUrl('http://google.com/search#done', 0, 0, 0, 0, 0, 1);
+        $this->assertEquals(1, $result);
 
-	    $result = isValidUrl('http://google.com/search', 0, 0, 0, 0, 0, 1);
-	    $this->assertEquals(0, $result);
+        $result = isValidUrl('http://google.com/search', 0, 0, 0, 0, 0, 1);
+        $this->assertEquals(0, $result);
     }
 
     /**
@@ -214,33 +214,33 @@ 
      */
     public function testIsIP()
     {
-    	// Not valid
-    	$ip='a299.299.299.299';
-    	$result=is_ip($ip);
+        // Not valid
+        $ip='a299.299.299.299';
+        $result=is_ip($ip);
         print __METHOD__." for ".$ip." result=".$result."\n";
-    	$this->assertEquals(0,$result,$ip);
+        $this->assertEquals(0,$result,$ip);
 
-    	// Reserved IP range (not checked by is_ip function)
-    	$ip='169.254.0.0';
-    	$result=is_ip($ip);
+        // Reserved IP range (not checked by is_ip function)
+        $ip='169.254.0.0';
+        $result=is_ip($ip);
         print __METHOD__." for ".$ip." result=".$result."\n";
-    	//$this->assertEquals(2,$result,$ip);      // Assertion disabled because returned value differs between PHP patch version
+        //$this->assertEquals(2,$result,$ip);      // Assertion disabled because returned value differs between PHP patch version
 
-    	$ip='1.2.3.4';
-    	$result=is_ip($ip);
+        $ip='1.2.3.4';
+        $result=is_ip($ip);
         print __METHOD__." for ".$ip." result=".$result."\n";
-    	$this->assertEquals(1,$result,$ip);
+        $this->assertEquals(1,$result,$ip);
 
-    	// Private IP ranges
-    	$ip='10.0.0.0';
-    	$result=is_ip($ip);
+        // Private IP ranges
+        $ip='10.0.0.0';
+        $result=is_ip($ip);
         print __METHOD__." for ".$ip." result=".$result."\n";
-    	$this->assertEquals(2,$result,$ip);
+        $this->assertEquals(2,$result,$ip);
 
-    	$ip='172.16.0.0';
-    	$result=is_ip($ip);
+        $ip='172.16.0.0';
+        $result=is_ip($ip);
         print __METHOD__." for ".$ip." result=".$result."\n";
-    	$this->assertEquals(2,$result,$ip);
+        $this->assertEquals(2,$result,$ip);
 
         $ip='192.168.0.0';
         $result=is_ip($ip);

dolibarr/test/phpunit/SecurityTest.php

Indentation +68 added lines, -68 removed lines

@@ -59,75 +59,75 @@ 
  */
 class SecurityTest extends PHPUnit_Framework_TestCase
 {
-	protected $savconf;
-	protected $savuser;
-	protected $savlangs;
-	protected $savdb;
-
-	/**
-	 * Constructor
-	 * We save global variables into local variables
-	 *
-	 * @return SecurityTest
-	 */
-	function __construct()
-	{
-		parent::__construct();
-
-		//$this->sharedFixture
-		global $conf,$user,$langs,$db;
-		$this->savconf=$conf;
-		$this->savuser=$user;
-		$this->savlangs=$langs;
-		$this->savdb=$db;
-
-		print __METHOD__." db->type=".$db->type." user->id=".$user->id;
-		//print " - db ".$db->db;
-		print "\n";
-	}
-
-	// Static methods
-  	public static function setUpBeforeClass()
+    protected $savconf;
+    protected $savuser;
+    protected $savlangs;
+    protected $savdb;
+
+    /**
+     * Constructor
+     * We save global variables into local variables
+     *
+     * @return SecurityTest
+     */
+    function __construct()
     {
-    	global $conf,$user,$langs,$db;
-		$db->begin();	// This is to have all actions inside a transaction even if test launched without suite.
+        parent::__construct();
+
+        //$this->sharedFixture
+        global $conf,$user,$langs,$db;
+        $this->savconf=$conf;
+        $this->savuser=$user;
+        $this->savlangs=$langs;
+        $this->savdb=$db;
+
+        print __METHOD__." db->type=".$db->type." user->id=".$user->id;
+        //print " - db ".$db->db;
+        print "\n";
+    }
+
+    // Static methods
+        public static function setUpBeforeClass()
+    {
+        global $conf,$user,$langs,$db;
+        $db->begin();	// This is to have all actions inside a transaction even if test launched without suite.
 
-    	print __METHOD__."\n";
+        print __METHOD__."\n";
     }
 
     // tear down after class
     public static function tearDownAfterClass()
     {
-    	global $conf,$user,$langs,$db;
-		$db->rollback();
+        global $conf,$user,$langs,$db;
+        $db->rollback();
 
-		print __METHOD__."\n";
+        print __METHOD__."\n";
     }
 
-	/**
-	 * Init phpunit tests
-	 *
-	 * @return	void
-	 */
+    /**
+     * Init phpunit tests
+     *
+     * @return	void
+     */
     protected function setUp()
     {
-    	global $conf,$user,$langs,$db;
-		$conf=$this->savconf;
-		$user=$this->savuser;
-		$langs=$this->savlangs;
-		$db=$this->savdb;
+        global $conf,$user,$langs,$db;
+        $conf=$this->savconf;
+        $user=$this->savuser;
+        $langs=$this->savlangs;
+        $db=$this->savdb;
 
-		print __METHOD__."\n";
+        print __METHOD__."\n";
     }
 
-	/**
-	 * End phpunit tests
-	 *
-	 * @return	void
-	 */
+    /**
+     * End phpunit tests
+     *
+     * @return	void
+     */
     protected function tearDown()
     {
-    	print __METHOD__."\n";
+        print __METHOD__."\n";
     }
 
     /**
@@ -137,16 +137,16 @@ 
      */
     public function testGETPOST()
     {
-    	global $conf,$user,$langs,$db;
-		$conf=$this->savconf;
-		$user=$this->savuser;
-		$langs=$this->savlangs;
-		$db=$this->savdb;
+        global $conf,$user,$langs,$db;
+        $conf=$this->savconf;
+        $user=$this->savuser;
+        $langs=$this->savlangs;
+        $db=$this->savdb;
 
         $_COOKIE["id"]=111;
-		$_GET["param1"]="222";
+        $_GET["param1"]="222";
         $_POST["param1"]="333";
-		$_GET["param2"]='a/b#e(pr)qq-rr\cc';
+        $_GET["param2"]='a/b#e(pr)qq-rr\cc';
         $_GET["param3"]='"a/b#e(pr)qq-rr\cc';    // Same than param2 + "
         $_GET["param4"]='../dir';
         $_GET["param5"]="a_1-b";
@@ -288,16 +288,16 @@ 
      */
     public function testRestrictedArea()
     {
-    	global $conf,$user,$langs,$db;
-		$conf=$this->savconf;
-		$user=$this->savuser;
-		$langs=$this->savlangs;
-		$db=$this->savdb;
+        global $conf,$user,$langs,$db;
+        $conf=$this->savconf;
+        $user=$this->savuser;
+        $langs=$this->savlangs;
+        $db=$this->savdb;
 
-		//$dummyuser=new User($db);
-		//$result=restrictedArea($dummyuser,'societe');
+        //$dummyuser=new User($db);
+        //$result=restrictedArea($dummyuser,'societe');
 
-		$result=restrictedArea($user,'societe');
-		$this->assertEquals(1,$result);
+        $result=restrictedArea($user,'societe');
+        $this->assertEquals(1,$result);
     }
 }

dolibarr/test/phpunit/ContratTest.php

Indentation +90 added lines, -90 removed lines

@@ -31,9 +31,9 @@ 
 
 if (empty($user->id))
 {
-	print "Load permissions for admin user nb 1\n";
-	$user->fetch(1);
-	$user->getrights();
+    print "Load permissions for admin user nb 1\n";
+    $user->fetch(1);
+    $user->getrights();
 }
 $conf->global->MAIN_DISABLE_ALL_MAILS=1;
 
@@ -47,74 +47,74 @@ 
  */
 class ContratTest extends PHPUnit_Framework_TestCase
 {
-	protected $savconf;
-	protected $savuser;
-	protected $savlangs;
-	protected $savdb;
-
-	/**
-	 * Constructor
-	 * We save global variables into local variables
-	 *
-	 * @return ContratTest
-	 */
-	function __construct()
-	{
-		parent::__construct();
-
-		//$this->sharedFixture
-		global $conf,$user,$langs,$db;
-		$this->savconf=$conf;
-		$this->savuser=$user;
-		$this->savlangs=$langs;
-		$this->savdb=$db;
-
-		print __METHOD__." db->type=".$db->type." user->id=".$user->id;
-		//print " - db ".$db->db;
-		print "\n";
-	}
-
-	// Static methods
-  	public static function setUpBeforeClass()
+    protected $savconf;
+    protected $savuser;
+    protected $savlangs;
+    protected $savdb;
+
+    /**
+     * Constructor
+     * We save global variables into local variables
+     *
+     * @return ContratTest
+     */
+    function __construct()
     {
-    	global $conf,$user,$langs,$db;
-		$db->begin();	// This is to have all actions inside a transaction even if test launched without suite.
+        parent::__construct();
 
-    	print __METHOD__."\n";
+        //$this->sharedFixture
+        global $conf,$user,$langs,$db;
+        $this->savconf=$conf;
+        $this->savuser=$user;
+        $this->savlangs=$langs;
+        $this->savdb=$db;
+
+        print __METHOD__." db->type=".$db->type." user->id=".$user->id;
+        //print " - db ".$db->db;
+        print "\n";
+    }
+
+    // Static methods
+        public static function setUpBeforeClass()
+    {
+        global $conf,$user,$langs,$db;
+        $db->begin();	// This is to have all actions inside a transaction even if test launched without suite.
+
+        print __METHOD__."\n";
     }
 
     // tear down after class
     public static function tearDownAfterClass()
     {
-    	global $conf,$user,$langs,$db;
-		$db->rollback();
+        global $conf,$user,$langs,$db;
+        $db->rollback();
 
-		print __METHOD__."\n";
+        print __METHOD__."\n";
     }
 
-	/**
-	 * Init phpunit tests
-	 *
-	 * @return	void
-	 */
+    /**
+     * Init phpunit tests
+     *
+     * @return	void
+     */
     protected function setUp()
     {
-    	global $conf,$user,$langs,$db;
-		$conf=$this->savconf;
-		$user=$this->savuser;
-		$langs=$this->savlangs;
-		$db=$this->savdb;
+        global $conf,$user,$langs,$db;
+        $conf=$this->savconf;
+        $user=$this->savuser;
+        $langs=$this->savlangs;
+        $db=$this->savdb;
 
-		print __METHOD__."\n";
+        print __METHOD__."\n";
     }
-	/**
-	 * End phpunit tests
-	 *
-	 * @return	void
-	 */
+    /**
+     * End phpunit tests
+     *
+     * @return	void
+     */
     protected function tearDown()
     {
-    	print __METHOD__."\n";
+        print __METHOD__."\n";
     }
 
     /**
@@ -124,20 +124,20 @@ 
      */
     public function testContratCreate()
     {
-    	global $conf,$user,$langs,$db;
-		$conf=$this->savconf;
-		$user=$this->savuser;
-		$langs=$this->savlangs;
-		$db=$this->savdb;
+        global $conf,$user,$langs,$db;
+        $conf=$this->savconf;
+        $user=$this->savuser;
+        $langs=$this->savlangs;
+        $db=$this->savdb;
 
-		$localobject=new Contrat($this->savdb);
-    	$localobject->initAsSpecimen();
-    	$result=$localobject->create($user);
+        $localobject=new Contrat($this->savdb);
+        $localobject->initAsSpecimen();
+        $result=$localobject->create($user);
 
-    	print __METHOD__." result=".$result."\n";
-    	$this->assertLessThan($result, 0);
+        print __METHOD__." result=".$result."\n";
+        $this->assertLessThan($result, 0);
 
-    	return $result;
+        return $result;
     }
 
     /**
@@ -151,22 +151,22 @@ 
      */
     public function testContratFetch($id)
     {
-    	global $conf,$user,$langs,$db;
-		$conf=$this->savconf;
-		$user=$this->savuser;
-		$langs=$this->savlangs;
-		$db=$this->savdb;
+        global $conf,$user,$langs,$db;
+        $conf=$this->savconf;
+        $user=$this->savuser;
+        $langs=$this->savlangs;
+        $db=$this->savdb;
 
-		$localobject=new Contrat($this->savdb);
-    	$result=$localobject->fetch($id);
+        $localobject=new Contrat($this->savdb);
+        $result=$localobject->fetch($id);
 
-    	print __METHOD__." id=".$id." result=".$result."\n";
-    	$this->assertLessThan($result, 0);
+        print __METHOD__." id=".$id." result=".$result."\n";
+        $this->assertLessThan($result, 0);
 
-    	return $localobject;
+        return $localobject;
     }
 
-   /**
+    /**
      * testContratOther
      *
      * @param	Object	$localobject	Object contract
@@ -206,18 +206,18 @@ 
      */
     public function testContratDelete($id)
     {
-    	global $conf,$user,$langs,$db;
-		$conf=$this->savconf;
-		$user=$this->savuser;
-		$langs=$this->savlangs;
-		$db=$this->savdb;
-
-		$localobject=new Contrat($this->savdb);
-    	$result=$localobject->fetch($id);
-		$result=$localobject->delete($user);
-
-		print __METHOD__." id=".$id." result=".$result."\n";
-    	$this->assertLessThan($result, 0);
-    	return $result;
+        global $conf,$user,$langs,$db;
+        $conf=$this->savconf;
+        $user=$this->savuser;
+        $langs=$this->savlangs;
+        $db=$this->savdb;
+
+        $localobject=new Contrat($this->savdb);
+        $result=$localobject->fetch($id);
+        $result=$localobject->delete($user);
+
+        print __METHOD__." id=".$id." result=".$result."\n";
+        $this->assertLessThan($result, 0);
+        return $result;
     }
 }