From 51b03e358e711bfbf303fb55ecc5a45e948d85f2 Mon Sep 17 00:00:00 2001
From: Laurent Destailleur <eldy@destailleur.fr>
Date: Sun, 20 Jan 2019 13:45:29 +0100
Subject: [PATCH] Fix phpunit

---
 .../class/supplier_proposal.class.php               |  2 +-
 test/phpunit/SupplierProposalTest.php               | 13 ++++++++++---
 2 files changed, 11 insertions(+), 4 deletions(-)

diff --git a/htdocs/supplier_proposal/class/supplier_proposal.class.php b/htdocs/supplier_proposal/class/supplier_proposal.class.php
index 5d4fbed5e72..a32b973d00a 100644
--- a/htdocs/supplier_proposal/class/supplier_proposal.class.php
+++ b/htdocs/supplier_proposal/class/supplier_proposal.class.php
@@ -843,7 +843,7 @@ class SupplierProposal extends CommonObject
      */
     function create($user, $notrigger=0)
     {
-        global $langs,$conf,$mysoc,$hookmanager;
+        global $langs, $conf, $mysoc, $hookmanager;
         $error=0;
 
         $now=dol_now();
diff --git a/test/phpunit/SupplierProposalTest.php b/test/phpunit/SupplierProposalTest.php
index 3648e0e61da..c284eb61178 100644
--- a/test/phpunit/SupplierProposalTest.php
+++ b/test/phpunit/SupplierProposalTest.php
@@ -203,8 +203,8 @@ class SupplierProposalTest extends PHPUnit_Framework_TestCase
     /**
      * testSupplierProposalValid
      *
-     * @param	Proposal	$localobject	Proposal
-     * @return	Proposal
+     * @param	SupplierProposal	$localobject	Proposal
+     * @return	SupplierProposal
      *
      * @depends	testSupplierProposalAddLine
      * The depends says test is run only if previous is ok
@@ -217,7 +217,14 @@ class SupplierProposalTest extends PHPUnit_Framework_TestCase
 		$langs=$this->savlangs;
 		$db=$this->savdb;
 
+		$result = $user->addrights(0, 'supplier_proposal');
+		$this->assertLessThan($result, 0);
+
+		$result = $user->getrights('supplier_proposal', 1);
+		//$this->assertLessThan($result, 0);
+
 		$result=$localobject->valid($user);
+		$this->assertLessThan($result, 0);
 
     	print __METHOD__." id=".$localobject->id." result=".$result."\n";
     	$this->assertLessThan($result, 0);
@@ -227,7 +234,7 @@ class SupplierProposalTest extends PHPUnit_Framework_TestCase
    /**
      * testSupplierProposalOther
      *
-     * @param	Proposal	$localobject	Proposal
+     * @param	SupplierProposal	$localobject	Proposal
      * @return	int
      *
      * @depends testSupplierProposalValid