[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Noalyss-commit] [noalyss] 02/02: Code improved : test and fix load func
|
From: |
Dany De Bontridder |
|
Subject: |
[Noalyss-commit] [noalyss] 02/02: Code improved : test and fix load functions |
|
Date: |
Mon, 6 Dec 2021 12:53:32 -0500 (EST) |
sparkyx pushed a commit to branch devel
in repository noalyss.
commit fc3c1057e341d05894635a1ce292521a4628ead8
Author: sparkyx <danydb@noalyss.eu>
AuthorDate: Mon Dec 6 18:53:16 2021 +0100
Code improved : test and fix load functions
---
include/class/acc_bilan.class.php | 6 +-
include/class/acc_ledger_info.class.php | 16 +-
include/class/acc_payment.class.php | 25 ++-
include/class/anc_group.class.php | 29 ++--
include/class/dossier.class.php | 10 +-
include/class/follow_up_detail.class.php | 18 ++-
include/class/forecast.class.php | 25 +--
include/class/todo_list.class.php | 13 +-
unit-test/include/class/acc_bilan.Test.php | 1 -
unit-test/include/class/acc_ledger_info.Test.php | 32 +++-
unit-test/include/class/acc_ledger_search.Test.php | 15 +-
unit-test/include/class/acc_payment.Test.php | 180 +++++++++++++++++++++
unit-test/include/class/anc_group.Test.php | 126 +++++++++++++++
unit-test/include/class/dossier.Test.php | 24 ++-
unit-test/include/class/pdf_anc_acc_list.Test.php | 4 +-
unit-test/include/class/user.Test.php | 9 +-
16 files changed, 453 insertions(+), 80 deletions(-)
diff --git a/include/class/acc_bilan.class.php
b/include/class/acc_bilan.class.php
index dcae0b8..85c8dda 100644
--- a/include/class/acc_bilan.class.php
+++ b/include/class/acc_bilan.class.php
@@ -273,8 +273,10 @@ class Acc_Bilan
if ( Database::num_row($res)==0)
throw new Exception (_('Aucun enregistrement trouve'));
$array=Database::fetch_array($res,0);
- foreach ($array as $name=>$value)
- $this->$name=$value;
+ $this->b_name=$array['b_name'];
+ $this->b_file_template=$array['b_file_template'];
+ $this->b_file_form=$array['b_file_form'];
+ $this->b_type=$array['b_type'];
}
catch(Exception $Ex)
diff --git a/include/class/acc_ledger_info.class.php
b/include/class/acc_ledger_info.class.php
index b67b971..376c993 100644
--- a/include/class/acc_ledger_info.class.php
+++ b/include/class/acc_ledger_info.class.php
@@ -90,26 +90,30 @@ class Acc_Ledger_Info
throw $e;
}
}
+ /**
+ * @brief load the todo_list row thanks it's ID
+ * @return boolean true if found else false
+ */
function load()
{
- $sql="select jr_id,id_type,ji_value from jrn_info where
ji_id=".$this->ji_id;
- $r=$this->cn->exec_sql($sql);
+ $sql="select jr_id,id_type,ji_value from jrn_info where ji_id=$1";
+ $r=$this->cn->exec_sql($sql,[$this->ji_id]);
if (Database::num_row ($r) > 0 )
{
$this->from_array(Database::fetch_array($r,0));
- return 0;
+ return true;
}
else
{
- return 1;
+ return false;
}
}
function from_array($p_array)
{
- foreach ($p_array as $col=>$value)
+ foreach (array("jr_id","id_type","ji_value") as $col)
{
- $this->$col=$value;
+ $this->$col=$p_array[$col];
}
}
function set_id($p_ji_id)
diff --git a/include/class/acc_payment.class.php
b/include/class/acc_payment.class.php
index a35f2b8..4358b36 100644
--- a/include/class/acc_payment.class.php
+++ b/include/class/acc_payment.class.php
@@ -93,13 +93,15 @@ class Acc_Payment
array($this->mp_id)
);
- if ( Database::num_row($res) == 0 ) return;
+ if ( Database::num_row($res) == 0 ) return false;
+
$row=Database::fetch_array($res,0);
- foreach ($row as $idx=>$value)
+ $a_index=array_values(self::$variable);
+ foreach ($a_index as $idx)
{
- $this->$idx=$value;
+ $this->$idx=$row[$idx];
}
-
+ return true;
}
/*!\brief retrieve all the data for all ledgers
@@ -289,9 +291,11 @@ class Acc_Payment
*/
public function from_array($p_array)
{
-
$idx=array('mp_id','mp_lib','mp_fd_id','mp_jrn_def_id','mp_qcode','jrn_def_id');
- foreach ($idx as $l)
- if (isset($p_array[$l])) $this->$l=$p_array[$l];
+ $a_index=array_values(self::$variable);
+ //
$idx=array('mp_id','mp_lib','mp_fd_id','mp_jrn_def_id','mp_qcode','jrn_def_id');
+ foreach ($a_index as $l) {
+ if (isset($p_array[$l])) $this->$l=$p_array[$l];
+ }
}
/**
*@brief return an html with a form to add a new middle of payment
@@ -333,13 +337,6 @@ class Acc_Payment
ob_end_clean();
return $r;
}
- /*!\brief test function
- */
- static function test_me()
- {
-
- }
-
}
diff --git a/include/class/anc_group.class.php
b/include/class/anc_group.class.php
index 5991568..50ffb94 100644
--- a/include/class/anc_group.class.php
+++ b/include/class/anc_group.class.php
@@ -52,19 +52,19 @@ class Anc_Group extends Anc_Print
{
if (strlen ($this->ga_id) > 10 ) return '<span
class="notice">'.
_('Taille de la code trop long maximum 10
caractères').'</span>';
- $sql=" insert into groupe_analytique (ga_id,ga_description,pa_id)
values ('%s','%s',%d)";
- $sql=sprintf($sql,Database::escape_string($this->ga_id),
- Database::escape_string($this->ga_description),
- $this->pa_id);
+ $sql=" insert into groupe_analytique (ga_id,ga_description,pa_id)
values ($1,$2,$3)";
try
{
- $this->db->exec_sql($sql);
+ $this->db->exec_sql($sql,[$this->ga_id,
+ $this->ga_description,
+ $this->pa_id]);
+ return true;
}
catch (Exception $a)
{
return '<span class="notice">Doublon !!</span>';
+ return false;
}
- return "";
}
/*!
* \brief remove from the database
@@ -74,26 +74,29 @@ class Anc_Group extends Anc_Print
{
$this->ga_id=str_replace(' ','',$this->ga_id);
$this->ga_id=strtoupper($this->ga_id);
- $sql=" delete from groupe_analytique where
ga_id='".Database::escape_string($this->ga_id)."'";
+ $sql=" delete from groupe_analytique where ga_id=$1";
- $this->db->exec_sql($sql);
+ $this->db->exec_sql($sql,array($this->ga_id));
}
- /*!
- * \brief load from the database and make an object
+ /**
+ * @brief load the todo_list row thanks it's ID
+ * @return boolean true if found else false
*/
function load()
{
$sql="select ga_id, ga_description,pa_id from groupe_analytique where".
- " ga_id = ".$this->ga_id;
- $res=$this->db->exec_sql($sql);
- $array=Database::fetch_all($res);
+ " ga_id = $1";
+ $res=$this->db->exec_sql($sql,[$this->ga_id]);
+ $array=Database::fetch_array($res);
if ( ! empty($array) )
{
$this->ga_id=$array['ga_id'];
$this->ga_description=$array['ga_description'];
$this->pa_id=$array['pa_id'];
+ return true;
}
+ return false;
}
/*!
diff --git a/include/class/dossier.class.php b/include/class/dossier.class.php
index 1eaa0a1..d666a0a 100644
--- a/include/class/dossier.class.php
+++ b/include/class/dossier.class.php
@@ -55,7 +55,9 @@ class Dossier
static function id()
{
self::check();
- return $_REQUEST['gDossier'];
+ $http=new HttpInput();
+
+ return $http->request('gDossier','number');
}
/**
@@ -259,7 +261,7 @@ class Dossier
public function load()
{
- $sql="select dos_name,dos_description,dos_email from ac_dossier where
dos_id=$1";
+ $sql="select * from ac_dossier where dos_id=$1";
$res=$this->cn->exec_sql(
$sql, array($this->dos_id)
@@ -268,9 +270,9 @@ class Dossier
if (Database::num_row($res)==0)
return;
$row=Database::fetch_array($res, 0);
- foreach ($row as $idx=> $value)
+ foreach ( self::$variable as $idx)
{
- $this->$idx=$value;
+ $this->$idx=$row[$idx];
}
}
diff --git a/include/class/follow_up_detail.class.php
b/include/class/follow_up_detail.class.php
index 9b5b0ee..dccfc07 100644
--- a/include/class/follow_up_detail.class.php
+++ b/include/class/follow_up_detail.class.php
@@ -22,7 +22,8 @@
* \brief Follow_Up details are the details for a actions
*/
-/*!\brief Follow_Up Details are the details for an actions, it means
+/*!
+ * \brief Follow_Up Details are the details for an actions, it means
* the details of an order, delivery order, submit a quote...
* this class is linked to the table action_detail
* - "id"=>"ad_id", primary key
@@ -164,24 +165,27 @@ class Follow_Up_Detail
}
return $aRet;
}
-
+ /**
+ * @brief load the todo_list row thanks it's ID
+ * @return boolean true if found else false
+ */
public function load()
{
$sql="SELECT ad_id, f_id, ad_text, ad_pu, ad_quant, ad_tva_id,
ad_tva_amount,
ad_total_amount, ag_id FROM action_detail".
" where ad_id=$1";
- $res=$this->db->get_array($this->db,
+ $res=$this->db->get_array(
$sql,
array($this->ad_id)
);
- if ( $this->db->count() == 0 ) return;
+ if ( $this->db->count() == 0 ) return false;
$row=$res[0];
- foreach ($row as $idx=>$value)
+ foreach (self::$variable as $idx)
{
- $this->$idx=$value;
+ $this->$idx=$row[$idx];
}
-
+ return true;
}
public function delete()
{
diff --git a/include/class/forecast.class.php b/include/class/forecast.class.php
index 166dac1..49fd909 100644
--- a/include/class/forecast.class.php
+++ b/include/class/forecast.class.php
@@ -27,7 +27,9 @@
*/
class Forecast
{
- private static $variable=array
("id"=>"f_id","name"=>"f_name","start_date"=>"f_start_date","end_date"=>"f_end_date");
+ private static $variable=array ("id"=>"f_id",
+ "name"=>"f_name","start_date"=>"f_start_date","end_date"=>"f_end_date"
+ );
private $cn;
/**
* @brief constructor
@@ -118,20 +120,25 @@ class Forecast
$ret=$p_cn->get_array($sql);
return $ret;
}
+ /**
+ * @brief load from db
+ * @return boolean true if found else false
+ */
public function load()
{
- $sql="select f_name,f_start_date ,f_end_date from forecast where
f_id=$1";
+ $sql="select f_id, f_name,f_start_date ,f_end_date from forecast where
f_id=$1";
$res=$this->cn->exec_sql(
$sql,
array($this->f_id)
);
- if ( Database::num_row($res) == 0 ) return -1;
+ if ( Database::num_row($res) == 0 ) return false;
$row=Database::fetch_array($res,0);
- foreach ($row as $idx=>$value)
+ $a_index=array_values(self::$variable);
+ foreach ( $a_index as $idx)
{
- $this->$idx=$value;
+ $this->$idx=$row[$idx];
}
-
+ return true;
}
public function delete()
{
@@ -158,11 +165,7 @@ class Forecast
" from forecast_item where
fc_id=$2",array($array[$i]['fc_id'],$old[$i]['fc_id']));
}
}
- /**
- * @brief unit test
- */
- static function test_me()
- {}
+
}
?>
diff --git a/include/class/todo_list.class.php
b/include/class/todo_list.class.php
index d42747e..d0bec9d 100644
--- a/include/class/todo_list.class.php
+++ b/include/class/todo_list.class.php
@@ -206,6 +206,10 @@ class Todo_List
return $array;
}
+ /**
+ * @brief load the todo_list row thanks it's ID
+ * @return boolean true if found else false
+ */
public function load()
{
@@ -217,13 +221,14 @@ class Todo_List
array($this->tl_id)
);
- if ( Database::num_row($res) == 0 ) return;
+ if ( Database::num_row($res) == 0 ) return false;
$row=Database::fetch_array($res,0);
- foreach ($row as $idx=>$value)
+ $aIndex=array_values(self::$variable);
+ foreach ($aIndex as $idx)
{
- $this->$idx=$value;
+ $this->$idx=$row[$idx];
}
-
+ return true;
}
public function delete()
{
diff --git a/unit-test/include/class/acc_bilan.Test.php
b/unit-test/include/class/acc_bilan.Test.php
index 90637dd..217b178 100644
--- a/unit-test/include/class/acc_bilan.Test.php
+++ b/unit-test/include/class/acc_bilan.Test.php
@@ -80,7 +80,6 @@ class Acc_BilanTest extends TestCase
/**
* @covers Acc_Bilan::load
- * @todo Implement testLoad().
*/
public function testLoad()
{
diff --git a/unit-test/include/class/acc_ledger_info.Test.php
b/unit-test/include/class/acc_ledger_info.Test.php
index a918598..ad06660 100644
--- a/unit-test/include/class/acc_ledger_info.Test.php
+++ b/unit-test/include/class/acc_ledger_info.Test.php
@@ -59,11 +59,12 @@ class Acc_Ledger_InfoTest extends TestCase
$this->object->id_type='BON_COMMANDE';
$this->object->ji_value='BON';
- $cnt=$g_connection->get_array("select * from jrn_info where
jr_id=$1",[$this->object->jr_id]);
- $this->assertSame(count($cnt),0);
+ $g_connection->exec_sql("delete from jrn_info where
jr_id=$1",[$this->object->jr_id]);
+ $cnt=$g_connection->get_value("select count(*) from jrn_info where
jr_id=$1",[$this->object->jr_id]);
+ $this->assertEquals($cnt,0,"Error : bon_commande already exist");
$this->object->insert();
- $cnt=$g_connection->get_array("select * from jrn_info where
jr_id=$1",[$this->object->jr_id]);
- $this->assertSame(count($cnt),1);
+ $cnt=$g_connection->get_value("select count(*) from jrn_info where
jr_id=$1",[$this->object->jr_id]);
+ $this->assertEquals($cnt,1,'Error : not inserted');
$g_connection->exec_sql("delete from jrn_info where
jr_id=$1",[$this->object->jr_id]);
}
@@ -81,5 +82,28 @@ class Acc_Ledger_InfoTest extends TestCase
$this->assertSame(count($cnt),1);
$g_connection->exec_sql("delete from jrn_info where
jr_id=$1",[$this->object->jr_id]);
}
+ /**
+ * @testdox test loading
+ */
+ function test_load()
+ {
+ global $g_connection;
+ $this->object->jr_id=3;
+ $this->object->id_type='BON_COMMANDE';
+ $this->object->ji_value='BONi 11111111';
+
+ $cnt=$g_connection->get_array("select * from jrn_info where
jr_id=$1",[$this->object->jr_id]);
+ $this->assertSame(count($cnt),0);
+ $this->object->insert();
+ $cnt=$g_connection->get_array("select * from jrn_info where
jr_id=$1",[$this->object->jr_id]);
+ $this->assertSame(count($cnt),1);
+ $reload_id=$g_connection->get_value("select ji_id from jrn_info where
jr_id=$1",[$this->object->jr_id]);
+ $reload=new Acc_Ledger_Info($g_connection,$reload_id);
+ $reload->load();
+ $this->assertSame($reload->ji_value,$this->object->ji_value,"Error
cannot load");
+
+ $g_connection->exec_sql("delete from jrn_info where
jr_id=$1",[$this->object->jr_id]);
+
+ }
}
diff --git a/unit-test/include/class/acc_ledger_search.Test.php
b/unit-test/include/class/acc_ledger_search.Test.php
index f3f0b03..cb4e28e 100644
--- a/unit-test/include/class/acc_ledger_search.Test.php
+++ b/unit-test/include/class/acc_ledger_search.Test.php
@@ -88,7 +88,7 @@ class Acc_Ledger_Test extends TestCase
$ledger=new Acc_Ledger_Search('ALL');
$r=$ledger->display_search_form();
\Noalyss\Facility::save_file(__DIR__."/file",
"acc_ledger_search-test_display_search_form.html", $r);
- $this->assertEquals(9665,strlen($r),"Size of the html string for
display_search_form see "
+ $this->assertEquals(9669,strlen($r),"Size of the html string for
display_search_form see "
.
__DIR__."/file/acc_ledger_search-test_display_search_form.html ");
}
/**
@@ -98,35 +98,34 @@ class Acc_Ledger_Test extends TestCase
{
$ledger=new Acc_Ledger_Search('ALL');
$ret = $ledger->build_search_filter();
-
$result=sprintf("manage_search_filter({'div':'','ledger_type':'ALL','all_type':1,'dossier':%d})"
+
$result=sprintf("manage_search_filter({'div':'','ledger_type':'ALL','all_type':1,'dossier':'%d'})"
,Dossier::id());
- $this->assertEquals($ret,$result,"Build filter for ALL");
-
+ $this->assertEquals($ret,$result,"Build filter for ALL");
$ledger=new Acc_Ledger_Search('FIN');
$this->assertEquals($ret,$result);
$ret = $ledger->build_search_filter();
-
$result=sprintf("manage_search_filter({'div':'','ledger_type':'FIN','all_type':1,'dossier':%d})"
+
$result=sprintf("manage_search_filter({'div':'','ledger_type':'FIN','all_type':1,'dossier':'%d'})"
,Dossier::id());
$this->assertEquals($ret,$result,"Build filter for FIN");
$ledger=new Acc_Ledger_Search('VEN');
$this->assertEquals($ret,$result);
$ret = $ledger->build_search_filter();
-
$result=sprintf("manage_search_filter({'div':'','ledger_type':'VEN','all_type':1,'dossier':%d})"
+
$result=sprintf("manage_search_filter({'div':'','ledger_type':'VEN','all_type':1,'dossier':'%d'})"
,Dossier::id());
$this->assertEquals($ret,$result,"Build filter for VEN");
$ledger=new Acc_Ledger_Search('ACH');
$this->assertEquals($ret,$result);
$ret = $ledger->build_search_filter();
-
$result=sprintf("manage_search_filter({'div':'','ledger_type':'ACH','all_type':1,'dossier':%d})"
+
$result=sprintf("manage_search_filter({'div':'','ledger_type':'ACH','all_type':1,'dossier':'%d'})"
,Dossier::id());
$this->assertEquals($ret,$result,"Build filter for ACH");
$ledger=new Acc_Ledger_Search('ODS');
$this->assertEquals($ret,$result);
$ret = $ledger->build_search_filter();
-
$result=sprintf("manage_search_filter({'div':'','ledger_type':'ODS','all_type':1,'dossier':%d})"
+
$result=sprintf("manage_search_filter({'div':'','ledger_type':'ODS','all_type':1,'dossier':'%d'})"
,Dossier::id());
$this->assertEquals($ret,$result,"Build filter for ODS");
diff --git a/unit-test/include/class/acc_payment.Test.php
b/unit-test/include/class/acc_payment.Test.php
new file mode 100644
index 0000000..dfbc81a
--- /dev/null
+++ b/unit-test/include/class/acc_payment.Test.php
@@ -0,0 +1,180 @@
+<?php
+
+/*
+ * * Copyright (C) 2021 Dany De Bontridder <dany@alchimerys.be>
+ *
+ * This program is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU General Public License
+ * as published by the Free Software Foundation; either version 2
+ * of the License, or (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write to the Free Software
+ * Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA.
+
+ *
+ * Author : Dany De Bontridder danydb@noalyss.eu
+ *
+ */
+
+/**
+ * @file
+ * @brief
+ */
+use PHPUnit\Framework\TestCase;
+
+/**
+ * @backupGlobals enabled
+ * @coversDefaultClass \Acc_Payment
+ */
+require 'global.php';
+
+class AccPaymentTest extends TestCase
+{
+
+ /**
+ * @var Fiche
+ */
+ protected $object;
+
+ /**
+ * Sets up the fixture, for example, opens a network connection.
+ * This method is called before a test method is executed.
+ */
+ protected function setUp()
+ {
+ include 'global.php';
+ }
+
+ public function dataLoad()
+ {
+ $a_json=array();
+ $a_json[0]=['{
+ "mp_id" : 4,
+ "mp_lib" : "Caisse",
+ "mp_jrn_def_id" : 1,
+ "mp_fd_id" : null,
+ "mp_qcode" : null,
+ "jrn_def_id" : 3
+ }'];
+
+ $a_json[1]=['{
+ "mp_id" : 3,
+ "mp_lib" : "Par gérant ou administrateur",
+ "mp_jrn_def_id" : 2,
+ "mp_fd_id" : null,
+ "mp_qcode" : null,
+ "jrn_def_id" : 3
+ }'];
+ $a_json[2]=['{
+ "mp_id" : 1,
+ "mp_lib" : "Paiement électronique",
+ "mp_jrn_def_id" : 1,
+ "mp_fd_id" : 3,
+ "mp_qcode" : "COMPTE",
+ "jrn_def_id" : 2
+ }'];
+ $a_json[3]=[' {
+ "mp_id" : 2,
+ "mp_lib" : "Caisse",
+ "mp_jrn_def_id" : 4,
+ "mp_fd_id" : 2,
+ "mp_qcode" : null,
+ "jrn_def_id" : 2
+ }'];
+ $a_json[4]=['{
+ "mp_id" : 11,
+ "mp_lib" : "Caisse",
+ "mp_jrn_def_id" : 1,
+ "mp_fd_id" : 3,
+ "mp_qcode" : "COMPTE",
+ "jrn_def_id" : 35
+ }'];
+ return $a_json;
+ }
+
+ /**
+ * @brief
+ * @testdox load record
+ * @dataProvider dataLoad
+ * @param json $json
+ * @covers ::load ::get_parameter
+ */
+ public function testLoad($json)
+ {
+ global $g_connection;
+ $acc_pay=json_decode($json, true);
+ $object=new Acc_Payment($g_connection, $acc_pay['mp_id']);
+ $object->load();
+ $this->assertEquals($acc_pay['mp_id'], $object->get_parameter('id'),
"Error for column mp_id");
+ $this->assertEquals($acc_pay['mp_lib'], $object->get_parameter('lib'),
"Error for column mp_lib");
+ $this->assertEquals($acc_pay['mp_qcode'],
$object->get_parameter('qcode'), "Error for column mp_qcode");
+ $this->assertEquals($acc_pay['mp_jrn_def_id'],
$object->get_parameter('ledger_target'),
+ "Error for column mp_jrn_def_id");
+ $this->assertEquals($acc_pay['jrn_def_id'],
$object->get_parameter('ledger_source'),
+ "Error for column jrn_def_id");
+ $this->assertEquals($acc_pay['mp_fd_id'],
$object->get_parameter('fiche_def'), "Error for column mp_fd_id");
+ }
+
+ /**
+ * @brief
+ * @testdox Test from array
+ * @dataProvider dataLoad
+ * @param json $json
+ * @covers ::from_array ::get_parameter
+ */
+ public function testFromArray($json)
+ {
+ global $g_connection;
+ $acc_pay=json_decode($json, true);
+ $object=new Acc_Payment($g_connection);
+ $object->from_array($acc_pay);
+ $this->assertEquals($acc_pay['mp_id'], $object->get_parameter('id'),
"Error for column mp_id");
+ $this->assertEquals($acc_pay['mp_lib'], $object->get_parameter('lib'),
"Error for column mp_lib");
+ $this->assertEquals($acc_pay['mp_qcode'],
$object->get_parameter('qcode'), "Error for column mp_qcode");
+ $this->assertEquals($acc_pay['mp_jrn_def_id'],
$object->get_parameter('ledger_target'),
+ "Error for column mp_jrn_def_id");
+ $this->assertEquals($acc_pay['jrn_def_id'],
$object->get_parameter('ledger_source'),
+ "Error for column jrn_def_id");
+ $this->assertEquals($acc_pay['mp_fd_id'],
$object->get_parameter('fiche_def'), "Error for column mp_fd_id");
+ }
+ /**
+ * @brief
+ * @testdox function get_valide
+ * @covers ::set_parameter ::get_valide
+ */
+ public function testGet_Valide()
+ {
+ global $g_connection;
+ $object=new Acc_Payment($g_connection);
+ $object->set_parameter("ledger_source", 2);
+ $valid_payment=$object->get_valide();
+ $this->assertEquals(2, count($valid_payment), 'Error different payment
method');
+ $this->assertEquals($valid_payment[0]->get_parameter('qcode'),
'COMPTE',
+ 'Propose method no correct
['.$valid_payment[0]->get_parameter('qcode').']');
+ $this->assertEquals($valid_payment[1]->get_parameter('lib'), 'Caisse',
+ 'label no correct
['.$valid_payment[0]->get_parameter('lib').']');
+ }
+ /**
+ * @brief
+ * @testdox function get_all
+ * @covers ::set_parameter ::get_all
+ */
+ public function testGet_All()
+ {
+ global $g_connection;
+ $object=new Acc_Payment($g_connection);
+ $valid_payment=$object->get_all();
+ $this->assertEquals(5, count($valid_payment), 'Error different payment
method');
+ $this->assertEquals($valid_payment[0]->get_parameter('qcode'), null,
+ 'Propose method no correct
['.$valid_payment[0]->get_parameter('qcode').']');
+ $this->assertEquals($valid_payment[1]->get_parameter('lib'), 'Caisse',
+ 'label no correct
['.$valid_payment[0]->get_parameter('lib').']');
+ }
+
+}
diff --git a/unit-test/include/class/anc_group.Test.php
b/unit-test/include/class/anc_group.Test.php
new file mode 100644
index 0000000..4d50ba5
--- /dev/null
+++ b/unit-test/include/class/anc_group.Test.php
@@ -0,0 +1,126 @@
+<?php
+
+/*
+ * * Copyright (C) 2021 Dany De Bontridder <dany@alchimerys.be>
+ *
+ * This program is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU General Public License
+ * as published by the Free Software Foundation; either version 2
+ * of the License, or (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write to the Free Software
+ * Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA.
+
+ *
+ * Author : Dany De Bontridder danydb@noalyss.eu
+ *
+ */
+
+/**
+ * @file
+ * @brief Test Anc_Group
+ */
+use PHPUnit\Framework\TestCase;
+
+/**
+ * @backupGlobals enabled
+ * @coversDefaultClass \Anc_Group
+ */
+require 'global.php';
+
+class Anc_GroupTest extends TestCase
+{
+
+
+ /**
+ * Sets up the fixture, for example, opens a network connection.
+ * This method is called before a test method is executed.
+ */
+ protected function setUp()
+ {
+ include 'global.php';
+ }
+
+ /**
+ * Tears down the fixture, for example, closes a network connection.
+ * This method is called after a test method is executed.
+ */
+ protected function tearDown()
+ {
+
+ }
+
+ /**
+ * the setUpBeforeClass() template methods is called before the first test
of the test case
+ * class is run
+ */
+ public static function setUpBeforeClass()
+ {
+ include 'global.php';
+ global $g_connection;
+ $g_connection->exec_sql("delete from plan_analytique where pa_id>1");
+ $g_connection->exec_sql("INSERT INTO plan_analytique
(pa_id,pa_name,pa_description) VALUES
+ (2,'AXE2','Axe 2'),
+ (3,'AXE3','Axe 3')");
+
+ }
+
+ /**
+ * tearDownAfterClass() template methods is calleafter the last test of
the test case class is run,
+ *
+ */
+ static function tearDownAfterClass()
+ {
+ include 'global.php';
+ global $g_connection;
+ $g_connection->exec_sql("delete from plan_analytique where pa_id>1");
+ }
+
+ public function datasqlStmt()
+ {
+ return array([1,"Group 1",1],
+ [2,"Group 2",1],
+ [3,"Group 3",1],
+ [4,"Group 4",null]
+ );
+ }
+
+ /**
+ * @brief
+ * @testdox insert, load and remove
+ * @dataProvider datasqlStmt
+ * @param type $p_param
+ * @covers
+ */
+ public function test_sqlStmt($p_ga_id,$pga_description,$ppa_id)
+ {
+ global $g_connection;
+ // check group empty
+ $this->assertEquals(0,$g_connection->get_value("select count(*) from
groupe_analytique"),
+ "Table not empty");
+ $obj=new Anc_Group($g_connection);
+ $obj->ga_id=$p_ga_id;
+ $obj->ga_description=$pga_description;
+ $obj->pa_id=$ppa_id;
+ $obj->insert();
+ $this->assertEquals(1,$g_connection->get_value("select count(*) from
groupe_analytique"),
+ "record not inserted");
+ // load it
+ $reload=new Anc_Group($g_connection);
+ $reload->ga_id=$obj->ga_id;
+ $reload->load();
+
$this->assertEquals($reload->ga_description,$obj->ga_description,"Cannot load
from db");
+
+ // remove it
+ $reload->remove();
+ $this->assertEquals(0,$g_connection->get_value("select count(*) from
groupe_analytique"),
+ "Cannot remove");
+ }
+
+}
diff --git a/unit-test/include/class/dossier.Test.php
b/unit-test/include/class/dossier.Test.php
index afba77c..f0d6273 100644
--- a/unit-test/include/class/dossier.Test.php
+++ b/unit-test/include/class/dossier.Test.php
@@ -2,6 +2,8 @@
use PHPUnit\Framework\TestCase;
/**
* Generated by PHPUnit_SkeletonGenerator on 2016-02-11 at 15:39:21.
+ * @coversDefaultClass \Dossier
+ * @backupGlobals enable
*/
class DossierTest extends TestCase
{
@@ -17,6 +19,7 @@ class DossierTest extends TestCase
*/
protected function setUp()
{
+ require_once 'global.php';
$this->object=new Dossier(DOSSIER);
}
@@ -35,6 +38,25 @@ class DossierTest extends TestCase
public function testSynchro_admin()
{
Dossier::synchro_admin(DOSSIER);
+ // nothing to test
+ $this->assertTrue(true);
+ }
+ /**
+ * @testdox Test Dossier::id()
+ * @covers ::id
+ */
+ public function testID()
+ {
+ $this->assertEquals(DOSSIER,Dossier::id(),"Id() doesn't work");
+ }
+ /**
+ * @testdox Load
+ */
+ public function testLoad()
+ {
+ global $g_connection;
+ $obj=new Dossier(DOSSIER);
+ $obj->load();
+ $this->assertEquals(DOSSIER,$obj->get_parameter("id"),"Not the right
folder");
}
-
}
diff --git a/unit-test/include/class/pdf_anc_acc_list.Test.php
b/unit-test/include/class/pdf_anc_acc_list.Test.php
index fbbfd3b..1bd8067 100644
--- a/unit-test/include/class/pdf_anc_acc_list.Test.php
+++ b/unit-test/include/class/pdf_anc_acc_list.Test.php
@@ -145,7 +145,7 @@ EOF;
$anc_acc_list->card_poste=3;
$pdf_anc_acc=new PDF_Anc_Acc_List($anc_acc_list);
$pdf_anc_acc->export_pdf()->Output(__DIR__."/file/pdf_anc_acc_list-activity-card.pdf","F");
-
$this->assertEquals(77690,filesize(__DIR__."/file/pdf_anc_acc_list-activity-card.pdf"),
+
$this->assertEquals(77689,filesize(__DIR__."/file/pdf_anc_acc_list-activity-card.pdf"),
__DIR__."/file/pdf_anc_acc_list-activity-card.pdf incorrect");
// By Activity / Account
@@ -156,7 +156,7 @@ EOF;
$anc_acc_list->card_poste=4;
$pdf_anc_acc=new PDF_Anc_Acc_List($anc_acc_list);
$pdf_anc_acc->export_pdf()->Output(__DIR__."/file/pdf_anc_acc_list-activity-account.pdf","F");
-
$this->assertEquals(76411,filesize(__DIR__."/file/pdf_anc_acc_list-activity-account.pdf"),
+
$this->assertEquals(76410,filesize(__DIR__."/file/pdf_anc_acc_list-activity-account.pdf"),
__DIR__."/file/pdf_anc_acc_list-activity-account.pdf
incorrect");
}
diff --git a/unit-test/include/class/user.Test.php
b/unit-test/include/class/user.Test.php
index 68836b6..0e98279 100644
--- a/unit-test/include/class/user.Test.php
+++ b/unit-test/include/class/user.Test.php
@@ -11,8 +11,11 @@ define('USE_ADMIN', 0);
define('USE_EMAIL', 'none@dev.null.eu');
/**
- * Generated by PHPUnit_SkeletonGenerator on 2016-02-10 at 22:48:23.
+ * @backupGlobals enabled
+ * @coversDefaultClass \User
*/
+require 'global.php';
+
class UserTest extends TestCase
{
@@ -132,8 +135,8 @@ class UserTest extends TestCase
*/
public function testPeriode($p_id)
{
- global $g_connection;
- $this->object->db=$g_connection;
+ $this->object=new User(Dossier::connect(), USE_ID);
+
$restore=$this->object->get_periode();
$this->assertTrue(is_numeric($restore),"Old periode id is not an
integer");
$this->object->set_periode($p_id);