|
From: | Laurent Destailleur (aka Eldy) |
Subject: | Re: [Dolibarr-dev] Dolibarr branch develop : phpunit test failed |
Date: | Tue, 2 Feb 2016 13:02:56 +0100 |
Ok, but still, anywhere in core dolibarr we have code to deal with tax like -19:-15:-9
Dataset coming from mysqldump_dolibarr_3.5.0.sql migrate to 3.8 is not the same as dataset coming from mysqldump_dolibarr_3.8.0.sql.
It can be confusing for local phpunit test before submit PR to github.
Regards
Florian Henry +33 6 03 76 48 07 address@hidden http://www.open-concept.pro Twitter : @_Open_Concept_ Google+ : https://www.google.com/+Open-conceptProLe 31/01/2016 19:41, Laurent Destailleur (aka Eldy) a écrit :
Yes, having x:y:z into localtax is a feature used for spanish companies.The datadump is not broken it is just not the datadump that match the phpunit. The phpunit is done acoording to data define into the dump of 3.5 version with migration done from 3.5 to 3.8.Having phpunit tests that does not depends on database data/setup is very difficult. We must just know it works with the dump of 3.5 migrated
2016-01-30 18:06 GMT+01:00 Florian HENRY <address@hidden>:
Hello all,
The current dev branch failed on travis cause of
1) FunctionsLibTest::testGetDefaultLocalTax
Failed asserting that '-19:-15:-9' matches expected -19.
/home/bobby/Eclipse/workspace/dolibarr/test/phpunit/FunctionsLibTest.php:858
When I dig on dev/initdata/mysqldump_dolibarr_3.8.0.sql, I see
-19:-15:-9 into localtax_2 lot's of time.
My question is data_dump broken or it is a feature to declare multiple
tva level, but if it is the case it's still ongoing dev. I can't find
anywhere a piece of code to deal with this VAT rate parate by ":" .In
htdcos/admin/dict.php it try to do a price('-19:-15:-9')=>0.
Shall we avoid to merge PR that cause travis crash even in dev branch ?
Regards
--
Florian Henry
+33 6 03 76 48 07
address@hidden
http://www.open-concept.pro
Twitter : @_Open_Concept_
Google+ : https://www.google.com/+Open-conceptPro
_______________________________________________
Dolibarr-dev mailing list
address@hidden
https://lists.nongnu.org/mailman/listinfo/dolibarr-dev
--
EMail: address@hidden------------------------------------------------------------------------------------Google+: https://plus.google.com/+LaurentDestailleur/
Facebook: https://www.facebook.com/Destailleur.Laurent
------------------------------------------------------------------------------------* Dolibarr (Project leader): http://www.dolibarr.org (make a donation for Dolibarr project via Paypal: address@hidden)* AWStats (Author) : http://awstats.sourceforge.net (make a donation for AWStats project via Paypal: address@hidden)* AWBot (Author) : http://awbot.sourceforge.net
* CVSChangeLogBuilder (Author) : http://cvschangelogb.sourceforge.net
_______________________________________________ Dolibarr-dev mailing list address@hidden https://lists.nongnu.org/mailman/listinfo/dolibarr-dev
_______________________________________________
Dolibarr-dev mailing list
address@hidden
https://lists.nongnu.org/mailman/listinfo/dolibarr-dev
[Prev in Thread] | Current Thread | [Next in Thread] |