diff --git a/test/phpunit/CodingSqlTest.php b/test/phpunit/CodingSqlTest.php index 245a7869811..3e065bb141c 100644 --- a/test/phpunit/CodingSqlTest.php +++ b/test/phpunit/CodingSqlTest.php @@ -170,7 +170,7 @@ class CodingSqlTest extends PHPUnit\Framework\TestCase $filecontent = str_replace('`rank`', '_rank_', $filecontent); $result=strpos($filecontent, '`'); - print __METHOD__." Result for checking we don't have back quote = ".$result."\n"; + //print __METHOD__." Result for checking we don't have back quote = ".$result."\n"; $this->assertTrue($result===false, 'Found back quote into '.$file.'. Bad.'); $result=strpos($filecontent, '"'); @@ -178,27 +178,27 @@ class CodingSqlTest extends PHPUnit\Framework\TestCase { $result=(! strpos($filecontent, '["') && ! strpos($filecontent, '{"')); } - print __METHOD__." Result for checking we don't have double quote = ".$result."\n"; + //print __METHOD__." Result for checking we don't have double quote = ".$result."\n"; $this->assertTrue($result===false, 'Found double quote that is not [" neither {" (used for json content) into '.$file.'. Bad.'); $result=strpos($filecontent, 'int('); - print __METHOD__." Result for checking we don't have 'int(' instead of 'integer' = ".$result."\n"; + //print __METHOD__." Result for checking we don't have 'int(' instead of 'integer' = ".$result."\n"; $this->assertTrue($result===false, 'Found int(x) or tinyint(x) instead of integer or tinyint into '.$file.'. Bad.'); $result=strpos($filecontent, 'ON DELETE CASCADE'); - print __METHOD__." Result for checking we don't have 'ON DELETE CASCADE' = ".$result."\n"; + //print __METHOD__." Result for checking we don't have 'ON DELETE CASCADE' = ".$result."\n"; $this->assertTrue($result===false, 'Found ON DELETE CASCADE into '.$file.'. Bad.'); $result=strpos($filecontent, 'NUMERIC('); - print __METHOD__." Result for checking we don't have 'NUMERIC(' = ".$result."\n"; + //print __METHOD__." Result for checking we don't have 'NUMERIC(' = ".$result."\n"; $this->assertTrue($result===false, 'Found NUMERIC( into '.$file.'. Bad.'); $result=strpos($filecontent, 'NUMERIC('); - print __METHOD__." Result for checking we don't have 'curdate(' = ".$result."\n"; + //print __METHOD__." Result for checking we don't have 'curdate(' = ".$result."\n"; $this->assertTrue($result===false, 'Found curdate( into '.$file.'. Bad. Current date must be generated with PHP.'); $result=strpos($filecontent, 'integer('); - print __METHOD__." Result for checking we don't have 'integer(' = ".$result."\n"; + //print __METHOD__." Result for checking we don't have 'integer(' = ".$result."\n"; $this->assertTrue($result===false, 'Found value in parenthesis after the integer. It must be integer not integer(x) into '.$file.'. Bad.'); if ($dir == DOL_DOCUMENT_ROOT.'/install/mysql/migration') @@ -214,11 +214,11 @@ class CodingSqlTest extends PHPUnit\Framework\TestCase } else { // Test for non key files only $result=(strpos($filecontent, 'KEY ') && strpos($filecontent, 'PRIMARY KEY') == 0); - print __METHOD__." Result for checking we don't have ' KEY ' instead of a sql file to create index = ".$result."\n"; + //print __METHOD__." Result for checking we don't have ' KEY ' instead of a sql file to create index = ".$result."\n"; $this->assertTrue($result===false, 'Found KEY into '.$file.'. Bad.'); $result=stripos($filecontent, 'ENGINE=innodb'); - print __METHOD__." Result for checking we have the ENGINE=innodb string = ".$result."\n"; + //print __METHOD__." Result for checking we have the ENGINE=innodb string = ".$result."\n"; $this->assertGreaterThan(0, $result, 'The ENGINE=innodb was not found into '.$file.'. Add it or just fix syntax to match case.'); } }