From fc62b1c9c798ff3519563b8268c1ee66d1099550 Mon Sep 17 00:00:00 2001 From: Laurent Destailleur Date: Sat, 18 Feb 2012 01:07:52 +0100 Subject: [PATCH] Removed all checkstyle errors into phpunit files --- test/phpunit/CMailFileTest.php | 3 +-- test/phpunit/CommandeTest.php | 24 +++++++++++---------- test/phpunit/CompanyBankAccountTest.php | 17 ++++++++------- test/phpunit/ContactTest.php | 25 +++++++++++++++++++--- test/phpunit/ContratTest.php | 20 +++++++++--------- test/phpunit/DiscountTest.php | 11 +++++----- test/phpunit/FactureFournisseurTest.php | 28 +++++++++++++++++++++++++ test/phpunit/FactureTest.php | 17 +++++++++++++++ test/phpunit/ProductTest.php | 21 ++++++++++--------- test/phpunit/UserTest.php | 14 ++++++------- 10 files changed, 124 insertions(+), 56 deletions(-) diff --git a/test/phpunit/CMailFileTest.php b/test/phpunit/CMailFileTest.php index 8e28276bfb8..6891401179b 100755 --- a/test/phpunit/CMailFileTest.php +++ b/test/phpunit/CMailFileTest.php @@ -126,8 +126,7 @@ class CMailFileTest extends PHPUnit_Framework_TestCase $langs=$this->savlangs; $db=$this->savdb; - $localobject=new CMailFile('Test','test@test.com','from@from.com', - 'Message txt',array(),array(),array(),'','',1,0); + $localobject=new CMailFile('Test','test@test.com','from@from.com','Message txt',array(),array(),array(),'','',1,0); $result=$localobject->sendfile(); print __METHOD__." result=".$result."\n"; diff --git a/test/phpunit/CommandeTest.php b/test/phpunit/CommandeTest.php index 135908dfe41..dc5e7363f89 100644 --- a/test/phpunit/CommandeTest.php +++ b/test/phpunit/CommandeTest.php @@ -116,7 +116,7 @@ class CommandeTest extends PHPUnit_Framework_TestCase /** * testCommandeCreate - * + * * @return void */ public function testCommandeCreate() @@ -138,8 +138,10 @@ class CommandeTest extends PHPUnit_Framework_TestCase /** * testCommandeFetch + * * @param int $id Id order - * + * @return Commande + * * @depends testCommandeCreate * The depends says test is run only if previous is ok */ @@ -161,10 +163,10 @@ class CommandeTest extends PHPUnit_Framework_TestCase /** * testCommandeValid - * + * * @param Object $localobject Order * @return void - * + * * @depends testCommandeFetch * The depends says test is run only if previous is ok */ @@ -185,10 +187,10 @@ class CommandeTest extends PHPUnit_Framework_TestCase /** * testCommandeCancel - * - * @param Object $localobject Order + * + * @param Object $localobject Order * @return void - * + * * @depends testCommandeValid * The depends says test is run only if previous is ok */ @@ -209,10 +211,10 @@ class CommandeTest extends PHPUnit_Framework_TestCase /** * testCommandeOther - * + * * @param Object $localobject Order * @return void - * + * * @depends testCommandeCancel * The depends says test is run only if previous is ok */ @@ -238,10 +240,10 @@ class CommandeTest extends PHPUnit_Framework_TestCase /** * testCommandeDelete - * + * * @param int $id Id of order * @return void - * + * * @depends testCommandeOther * The depends says test is run only if previous is ok */ diff --git a/test/phpunit/CompanyBankAccountTest.php b/test/phpunit/CompanyBankAccountTest.php index f3f6b36857b..5595c919768 100644 --- a/test/phpunit/CompanyBankAccountTest.php +++ b/test/phpunit/CompanyBankAccountTest.php @@ -116,7 +116,7 @@ class CompanyBankAccountTest extends PHPUnit_Framework_TestCase /** * testCompanyBankAccountCreate - * + * * @return int */ public function testCompanyBankAccountCreate() @@ -138,10 +138,10 @@ class CompanyBankAccountTest extends PHPUnit_Framework_TestCase /** * testCompanyBankAccountFetch - * + * * @param int $id Id of bank account * @return void - * + * * @depends testCompanyBankAccountCreate * The depends says test is run only if previous is ok */ @@ -163,9 +163,10 @@ class CompanyBankAccountTest extends PHPUnit_Framework_TestCase /** * testCompanyBankAccountUpdate - * + * * @param Object $localobject Bank account object - * + * @return int + * * @depends testCompanyBankAccountFetch * The depends says test is run only if previous is ok */ @@ -187,10 +188,10 @@ class CompanyBankAccountTest extends PHPUnit_Framework_TestCase /** * testCompanyBankAccountOther - * + * * @param Object $localobject Bank account - * @return int - * + * @return int + * * @depends testCompanyBankAccountFetch * The depends says test is run only if previous is ok */ diff --git a/test/phpunit/ContactTest.php b/test/phpunit/ContactTest.php index 49ccbcb4aad..84ad96e5e98 100755 --- a/test/phpunit/ContactTest.php +++ b/test/phpunit/ContactTest.php @@ -117,7 +117,7 @@ class ContactTest extends PHPUnit_Framework_TestCase /** * testContactCreate - * + * * @return int */ public function testContactCreate() @@ -140,8 +140,9 @@ class ContactTest extends PHPUnit_Framework_TestCase /** * testContactFetch - * - * @param int Id of contact + * + * @param int $id Id of contact + * @return int * @depends testContactCreate * The depends says test is run only if previous is ok */ @@ -163,6 +164,11 @@ class ContactTest extends PHPUnit_Framework_TestCase } /** + * testContactUpdate + * + * @param Contact $localobject Contact + * @return int + * * @depends testContactFetch * The depends says test is run only if previous is ok */ @@ -244,6 +250,11 @@ class ContactTest extends PHPUnit_Framework_TestCase } /** + * testContactOther + * + * @param Contact $localobject Contact + * @return void + * * @depends testContactUpdate * The depends says test is run only if previous is ok */ @@ -273,6 +284,11 @@ class ContactTest extends PHPUnit_Framework_TestCase } /** + * testContactDelete + * + * @param int $id Id of contact + * @return void + * * @depends testContactOther * The depends says test is run only if previous is ok */ @@ -295,6 +311,9 @@ class ContactTest extends PHPUnit_Framework_TestCase } /** + * testContactStatic + * + * @return void */ public function testContactStatic() { diff --git a/test/phpunit/ContratTest.php b/test/phpunit/ContratTest.php index fa4a2cf4362..824f463034f 100644 --- a/test/phpunit/ContratTest.php +++ b/test/phpunit/ContratTest.php @@ -115,7 +115,7 @@ class ContratTest extends PHPUnit_Framework_TestCase /** * testContratCreate - * + * * @return int */ public function testContratCreate() @@ -138,10 +138,10 @@ class ContratTest extends PHPUnit_Framework_TestCase /** * testContratFetch - * + * * @param int $id Id of contract * @return int - * + * * @depends testContratCreate * The depends says test is run only if previous is ok */ @@ -164,10 +164,10 @@ class ContratTest extends PHPUnit_Framework_TestCase /** * testContratValid - * - * @param int $id Id of contract + * + * @param Contrat $localobject Contract * @return int - * + * * @depends testContratFetch * The depends says test is run only if previous is ok */ @@ -188,10 +188,10 @@ class ContratTest extends PHPUnit_Framework_TestCase /** * testContratValid - * + * * @param Object $localobject Object contract * @return int - * + * * @depends testContratValid * The depends says test is run only if previous is ok */ @@ -217,10 +217,10 @@ class ContratTest extends PHPUnit_Framework_TestCase /** * testContratDelete - * + * * @param int $id Id of contract * @return int - * + * * @depends testContratOther * The depends says test is run only if previous is ok */ diff --git a/test/phpunit/DiscountTest.php b/test/phpunit/DiscountTest.php index 976dd1d42f1..7d9a1dde834 100755 --- a/test/phpunit/DiscountTest.php +++ b/test/phpunit/DiscountTest.php @@ -116,7 +116,7 @@ class DiscountTest extends PHPUnit_Framework_TestCase /** * testDiscountCreate - * + * * @return int */ public function testDiscountCreate() @@ -138,10 +138,10 @@ class DiscountTest extends PHPUnit_Framework_TestCase /** * testDiscountFetch - * + * * @param int $id Id of discount * @return int - * + * * @depends testDiscountCreate * The depends says test is run only if previous is ok */ @@ -163,9 +163,10 @@ class DiscountTest extends PHPUnit_Framework_TestCase /** * testDiscountDelete - * + * * @param int $id Id of discount - * + * @return int + * * @depends testDiscountFetch * The depends says test is run only if previous is ok */ diff --git a/test/phpunit/FactureFournisseurTest.php b/test/phpunit/FactureFournisseurTest.php index 470fa6c49d6..9a1afe73f53 100644 --- a/test/phpunit/FactureFournisseurTest.php +++ b/test/phpunit/FactureFournisseurTest.php @@ -114,6 +114,9 @@ class FactureFournisseurTest extends PHPUnit_Framework_TestCase } /** + * testFactureFournisseurCreate + * + * @return int */ public function testFactureFournisseurCreate() { @@ -133,6 +136,11 @@ class FactureFournisseurTest extends PHPUnit_Framework_TestCase } /** + * testFactureFournisseurFetch + * + * @param int $id If supplier invoice + * @return void + * * @depends testFactureFournisseurCreate * The depends says test is run only if previous is ok */ @@ -153,6 +161,11 @@ class FactureFournisseurTest extends PHPUnit_Framework_TestCase } /** + * testFactureFournisseurUpdate + * + * @param Object $localobject Supplier invoice + * @return int + * * @depends testFactureFournisseurFetch * The depends says test is run only if previous is ok */ @@ -173,6 +186,11 @@ class FactureFournisseurTest extends PHPUnit_Framework_TestCase } /** + * testFactureFournisseurValid + * + * @param Object $localobject Supplier invoice + * @return void + * * @depends testFactureFournisseurUpdate * The depends says test is run only if previous is ok */ @@ -192,6 +210,11 @@ class FactureFournisseurTest extends PHPUnit_Framework_TestCase } /** + * testFactureFournisseurOther + * + * @param Object $localobject Supplier invoice + * @return void + * * @depends testFactureFournisseurValid * The depends says test is run only if previous is ok */ @@ -216,6 +239,11 @@ class FactureFournisseurTest extends PHPUnit_Framework_TestCase } /** + * testFactureFournisseurDelete + * + * @param int $id Id of supplier invoice + * @return void + * * @depends testFactureFournisseurOther * The depends says test is run only if previous is ok */ diff --git a/test/phpunit/FactureTest.php b/test/phpunit/FactureTest.php index 8c0b66ae181..7aede976e83 100644 --- a/test/phpunit/FactureTest.php +++ b/test/phpunit/FactureTest.php @@ -140,6 +140,7 @@ class FactureTest extends PHPUnit_Framework_TestCase * testFactureFetch * * @param int $id Id invoice + * @return int * * @depends testFactureCreate * The depends says test is run only if previous is ok @@ -164,6 +165,7 @@ class FactureTest extends PHPUnit_Framework_TestCase * testFactureFetch * * @param Object $localobject Invoice + * @return int * * @depends testFactureFetch * The depends says test is run only if previous is ok @@ -185,6 +187,11 @@ class FactureTest extends PHPUnit_Framework_TestCase } /** + * testFactureValid + * + * @param Object $localobject Invoice + * @return void + * * @depends testFactureUpdate * The depends says test is run only if previous is ok */ @@ -211,6 +218,11 @@ class FactureTest extends PHPUnit_Framework_TestCase } /** + * testFactureOther + * + * @param Object $localobject Invoice + * @return int + * * @depends testFactureValid * The depends says test is run only if previous is ok */ @@ -235,6 +247,11 @@ class FactureTest extends PHPUnit_Framework_TestCase } /** + * testFactureDelete + * + * @param int $id Id of invoice + * @return int + * * @depends testFactureOther * The depends says test is run only if previous is ok */ diff --git a/test/phpunit/ProductTest.php b/test/phpunit/ProductTest.php index d1aad437c8e..799b16da232 100755 --- a/test/phpunit/ProductTest.php +++ b/test/phpunit/ProductTest.php @@ -142,9 +142,10 @@ class ProductTest extends PHPUnit_Framework_TestCase /** * testProductFetch - * - * @param int $id Id product - * + * + * @param int $id Id product + * @return Product + * * @depends testProductCreate * The depends says test is run only if previous is ok */ @@ -166,10 +167,10 @@ class ProductTest extends PHPUnit_Framework_TestCase /** * testProductUpdate - * + * * @param Product $localobject Product * @return void - * + * * @depends testProductFetch * The depends says test is run only if previous is ok */ @@ -191,10 +192,10 @@ class ProductTest extends PHPUnit_Framework_TestCase /** * testProductOther - * + * * @param Product $localobject Product * @return void - * + * * @depends testProductUpdate * The depends says test is run only if previous is ok */ @@ -212,10 +213,10 @@ class ProductTest extends PHPUnit_Framework_TestCase /** * testProductDelete - * + * * @param int $id Id of product * @return void - * + * * @depends testProductOther * The depends says test is run only if previous is ok */ @@ -239,7 +240,7 @@ class ProductTest extends PHPUnit_Framework_TestCase /** * testProductStatic - * + * * @return void */ public function testProductStatic() diff --git a/test/phpunit/UserTest.php b/test/phpunit/UserTest.php index 288cea791de..223b94397e3 100644 --- a/test/phpunit/UserTest.php +++ b/test/phpunit/UserTest.php @@ -116,7 +116,7 @@ class UserTest extends PHPUnit_Framework_TestCase /** * testUserCreate - * + * * @return void */ public function testUserCreate() @@ -138,7 +138,7 @@ class UserTest extends PHPUnit_Framework_TestCase /** * testUserFetch - * + * * @param int $id Id of user * @return void * @depends testUserCreate @@ -162,7 +162,7 @@ class UserTest extends PHPUnit_Framework_TestCase /** * testUserUpdate - * + * * @param Object $localobject User * @return void * @depends testUserFetch @@ -186,7 +186,7 @@ class UserTest extends PHPUnit_Framework_TestCase /** * testUserDisable - * + * * @param Object $localobject User * @return void * @depends testUserUpdate @@ -209,7 +209,7 @@ class UserTest extends PHPUnit_Framework_TestCase /** * testUserOther - * + * * @param Object $localobject User * @return void * @depends testUserDisable @@ -237,8 +237,8 @@ class UserTest extends PHPUnit_Framework_TestCase /** * testUserDelete - * - * @param Object $localobject User + * + * @param Object $id User * @return void * @depends testUserOther * The depends says test is run only if previous is ok