Description
Alex Buckley
2022-09-27 03:50:29 UTC
Created attachment 141017 [details] [review] Bug 31631: Add new system preference Sponsored-by: Waikato Institute of Technology, New Zealand Created attachment 141018 [details] [review] Bug 31631: Optionally use tax-exclusive values for calculating fund values Test plan: 1. Apply patches 2. sudo koha-upgrade-schema <instance> 3. Restart services 4. Set TaxRates syspref = 0.15 5. Create a new fund with amount = 1000 6. Add an order to a basket with the following values: - Fund = Choose the fund from step #5 - Quantity = 1 - Vendor price = 10 - Tax rate = 15% - Discount = 20% - Retail price = 10.00 - Replacement cost = 10.00 - Actual cost = 10.00 7. Go to the Acquisitions home page 8. Confirm the 'Ordered' column for the step #5 fund contains 9.20 9. Click on the 9.20 link and confirm the ordered.pl page that loads also contains 9.20 as the 'Subtotal' 10. Change 'CalculateFundValuesIncludingTax' = 'Exclude' 11. On the Acquisitions home page confirm the 'Ordered' column now contains 8.00 12. Click on the 8.00 link and confirm the ordered.pl page also contains 8.00 as the 'Subtotal' 13. Close your basket and receive your order keeping all the default values 14. Go to the Acquisitions home page and confirm the 'Spent' column now contains 8.00 15. Click on the 8.00 link and confirm the spent.pl page also contains 8.00 as the 'Subtotal' 16. Change 'CalculateFundValuesIncludingTax' = 'Include' 17. Confirm the 'Spent' column on the Acquisitions home page now shows 9.20 18. Click on the 9.20 link and confirm the spent.pl page also contains 9.20 as the 'Subtotal' Sponsored-by: Waikato Institute of Technology, New Zealand Created attachment 141019 [details] [review] Bug 31631: Unit tests Test plan: 1. Run tests in koha-shell sudo koha-shell <instance> prove t/db_dependent/Budgets.t Sponsored-by: Waikato Institute of Technology, New Zealand Created attachment 142597 [details] [review] Bug 31631: (follow-up) Use CalculateFundValuesIncludingTax when modifying existing order This patch depends on Bug 31840. To test: 1) Edit an order and increase the price so that you would expect it to trigger a warning that you will exceed the amount allowed for this Fund 2) Confirm the warning is triggered Ready for testing. Created attachment 142661 [details] Screenshot - Receiving order - formatting and default values Testing notes and issues/query (tested using koha-testing-docker). Three things I noticed: 1. Tax Rate system preference warning message --------------------------------------------- Even though I defined the Tax Rate system preference as 0.15, when adding an item to a basket: for tax rate there is a message saying "Tax rate not defined in system preference TaxRates!". Despite the warning, everything is calculated correctly using the 15% tax rate. I've logged bug 31984 for this, as this was happening before the patches were applied (includes a screenshot to show what I mean). 2. Step 13 - Formatting things ------------------------------ When receiving order (see screenshot): - For the Retail price field: no space after value and the opening bracket; no space between comma and tax exclusive - For the Budgeted cost field: no space after value and the opening bracket 3. For step 13 - Query about using the default values ----------------------------------------------------- I received the order with the default values (see the screenshot). As the actual cost defaulted to $10 (as entered in step 6), then the 10.00 is the value excluding tax, so: - For step 14: spent = 10.00 - For step 15: subtotal = 10.00 - For step 17: spent = 11.50 - For step 18: subtotal = 11.50 These values are correctly calculated, but don't match up with the test plan. So should I have entered the actual costs as 8.00 (the $10 retail price less the 20% discount, tax exclusive)? Thanks for testing David (In reply to David Nind from comment #6) > Created attachment 142661 [details] > Screenshot - Receiving order - formatting and default values > > Testing notes and issues/query (tested using koha-testing-docker). > > Three things I noticed: > > 1. Tax Rate system preference warning message > --------------------------------------------- > Unrelated (exists before patches), see bug 31984 > > 2. Step 13 - Formatting things > ------------------------------ > Unrelated (exists before patches), maybe open another bug for this? > > 3. For step 13 - Query about using the default values > ----------------------------------------------------- > > These values are correctly calculated, but don't match up with the test > plan. So should I have entered the actual costs as 8.00 (the $10 retail > price less the 20% discount, tax exclusive)? Yes I think that's right sorry! Created attachment 143156 [details] [review] Bug 31631: Add new system preference Sponsored-by: Waikato Institute of Technology, New Zealand Signed-off-by: David Nind <david@davidnind.com> Created attachment 143157 [details] [review] Bug 31631: Optionally use tax-exclusive values for calculating fund values Test plan: 1. Apply patches 2. sudo koha-upgrade-schema <instance> 3. Restart services 4. Set TaxRates syspref = 0.15 5. Create a new fund with amount = 1000 6. Add an order to a basket with the following values: - Fund = Choose the fund from step #5 - Quantity = 1 - Vendor price = 10 - Tax rate = 15% - Discount = 20% - Retail price = 10.00 - Replacement cost = 10.00 - Actual cost = 10.00 7. Go to the Acquisitions home page 8. Confirm the 'Ordered' column for the step #5 fund contains 9.20 9. Click on the 9.20 link and confirm the ordered.pl page that loads also contains 9.20 as the 'Subtotal' 10. Change 'CalculateFundValuesIncludingTax' = 'Exclude' 11. On the Acquisitions home page confirm the 'Ordered' column now contains 8.00 12. Click on the 8.00 link and confirm the ordered.pl page also contains 8.00 as the 'Subtotal' 13. Close your basket and receive your order keeping all the default values 14. Go to the Acquisitions home page and confirm the 'Spent' column now contains 8.00 15. Click on the 8.00 link and confirm the spent.pl page also contains 8.00 as the 'Subtotal' 16. Change 'CalculateFundValuesIncludingTax' = 'Include' 17. Confirm the 'Spent' column on the Acquisitions home page now shows 9.20 18. Click on the 9.20 link and confirm the spent.pl page also contains 9.20 as the 'Subtotal' Sponsored-by: Waikato Institute of Technology, New Zealand Signed-off-by: David Nind <david@davidnind.com> Created attachment 143158 [details] [review] Bug 31631: Unit tests Test plan: 1. Run tests in koha-shell sudo koha-shell <instance> prove t/db_dependent/Budgets.t Sponsored-by: Waikato Institute of Technology, New Zealand Signed-off-by: David Nind <david@davidnind.com> Created attachment 143159 [details] [review] Bug 31631: (follow-up) Use CalculateFundValuesIncludingTax when modifying existing order This patch depends on Bug 31840. To test: 1) Edit an order and increase the price so that you would expect it to trigger a warning that you will exceed the amount allowed for this Fund 2) Confirm the warning is triggered Signed-off-by: David Nind <david@davidnind.com> Unit tests are failing randomly: root@kohadevbox:koha(bug31631-qa)$ prove t/db_dependent/Budgets.t t/db_dependent/Budgets.t .. 153/154 Use of uninitialized value in multiplication (*) at /kohadevbox/koha/Koha/Acquisition/Order.pm line 515. Use of uninitialized value in numeric eq (==) at /kohadevbox/koha/Koha/Acquisition/Order.pm line 526. Use of uninitialized value in multiplication (*) at /kohadevbox/koha/Koha/Acquisition/Order.pm line 515. Use of uninitialized value in numeric eq (==) at /kohadevbox/koha/Koha/Acquisition/Order.pm line 526. # Failed test 'We expect this to be the tax exclusive value' # at t/db_dependent/Budgets.t line 1243. # got: '157.6' # expected: '147.36' # Failed test 'We expect this to be the tax exclusive value' # at t/db_dependent/Budgets.t line 1247. # got: '169.44' # expected: '157.6' # Failed test 'We expect this to be the tax exclusive value' # at t/db_dependent/Budgets.t line 1270. # got: '157.6' # expected: '147.36' # Failed test 'We expect this value to be the tax exclusive value' # at t/db_dependent/Budgets.t line 1272. # got: '157.6' # expected: '147.36' # Failed test 'We expect this to be the tax exclusive value' # at t/db_dependent/Budgets.t line 1276. # got: '169.44' # expected: '157.6' # Failed test 'We expect this value to be the tax exclusive value' # at t/db_dependent/Budgets.t line 1278. # got: '169.44' # expected: '157.6' # Looks like you failed 6 tests of 34. t/db_dependent/Budgets.t .. 154/154 # Failed test 'GetBudgetSpent GetBudgetOrdered GetBudgetsPlanCell FieldsForCalculatingFundValues tests' # at t/db_dependent/Budgets.t line 1290. # Looks like you failed 1 test of 154. t/db_dependent/Budgets.t .. Dubious, test returned 1 (wstat 256, 0x100) Failed 1/154 subtests Test Summary Report ------------------- t/db_dependent/Budgets.t (Wstat: 256 Tests: 154 Failed: 1) Failed test: 154 Non-zero exit status: 1 Files=1, Tests=154, 4 wallclock secs ( 0.02 usr 0.00 sys + 3.43 cusr 0.65 csys = 4.10 CPU) Result: FAIL root@kohadevbox:koha(bug31631-qa)$ git log commit 2a3c15c91467b01802c31a65f3436c9175ef1103 (HEAD -> bug31631-qa) Author: Aleisha Amohia <aleisha@catalystacademy.net.nz> Date: Tue Oct 25 03:08:03 2022 +0000 Bug 31631: (follow-up) Use CalculateFundValuesIncludingTax when modifying existing order This patch depends on Bug 31840. To test: 1) Edit an order and increase the price so that you would expect it to trigger a warning that you will exceed the amount allowed for this Fund 2) Confirm the warning is triggered Signed-off-by: David Nind <david@davidnind.com> commit cc6bcfe54ea55691bd4ed749b58fac2f9331f8fd Author: Alex Buckley <alexbuckley@catalyst.net.nz> Date: Tue Sep 27 17:01:51 2022 +1300 Bug 31631: Unit tests Test plan: 1. Run tests in koha-shell sudo koha-shell <instance> prove t/db_dependent/Budgets.t Sponsored-by: Waikato Institute of Technology, New Zealand Signed-off-by: David Nind <david@davidnind.com> commit a50a301eacde061376c6a3c0cafc03b56e8d1604 Author: Alex Buckley <alexbuckley@catalyst.net.nz> Date: Tue Sep 27 16:59:00 2022 +1300 Bug 31631: Optionally use tax-exclusive values for calculating fund values Test plan: 1. Apply patches 2. sudo koha-upgrade-schema <instance> 3. Restart services 4. Set TaxRates syspref = 0.15 5. Create a new fund with amount = 1000 6. Add an order to a basket with the following values: - Fund = Choose the fund from step #5 - Quantity = 1 - Vendor price = 10 - Tax rate = 15% - Discount = 20% - Retail price = 10.00 - Replacement cost = 10.00 - Actual cost = 10.00 7. Go to the Acquisitions home page 8. Confirm the 'Ordered' column for the step #5 fund contains 9.20 9. Click on the 9.20 link and confirm the ordered.pl page that loads also contains 9.20 as the 'Subtotal' 10. Change 'CalculateFundValuesIncludingTax' = 'Exclude' root@kohadevbox:koha(bug31631-qa)$ prove t/db_dependent/Budgets.t t/db_dependent/Budgets.t .. 153/154 Use of uninitialized value in addition (+) at /kohadevbox/koha/Koha/Acquisition/Order.pm line 490. Use of uninitialized value in addition (+) at /kohadevbox/koha/Koha/Acquisition/Order.pm line 490. t/db_dependent/Budgets.t .. ok All tests successful. Files=1, Tests=154, 4 wallclock secs ( 0.02 usr 0.01 sys + 2.99 cusr 0.54 csys = 3.56 CPU) Result: PASS root@kohadevbox:koha(bug31631-qa)$ prove t/db_dependent/Budgets.t t/db_dependent/Budgets.t .. 153/154 Use of uninitialized value in multiplication (*) at /kohadevbox/koha/Koha/Acquisition/Order.pm line 515. Use of uninitialized value in numeric eq (==) at /kohadevbox/koha/Koha/Acquisition/Order.pm line 526. Use of uninitialized value in multiplication (*) at /kohadevbox/koha/Koha/Acquisition/Order.pm line 515. Use of uninitialized value in numeric eq (==) at /kohadevbox/koha/Koha/Acquisition/Order.pm line 526. # Failed test 'We expect this to be the tax exclusive value' # at t/db_dependent/Budgets.t line 1243. # got: '157.6' # expected: '147.36' # Failed test 'We expect this to be the tax exclusive value' # at t/db_dependent/Budgets.t line 1247. # got: '169.44' # expected: '157.6' # Failed test 'We expect this to be the tax exclusive value' # at t/db_dependent/Budgets.t line 1270. # got: '157.6' # expected: '147.36' # Failed test 'We expect this value to be the tax exclusive value' # at t/db_dependent/Budgets.t line 1272. # got: '157.6' # expected: '147.36' # Failed test 'We expect this to be the tax exclusive value' # at t/db_dependent/Budgets.t line 1276. # got: '169.44' # expected: '157.6' # Failed test 'We expect this value to be the tax exclusive value' # at t/db_dependent/Budgets.t line 1278. # got: '169.44' # expected: '157.6' # Looks like you failed 6 tests of 34. t/db_dependent/Budgets.t .. 154/154 # Failed test 'GetBudgetSpent GetBudgetOrdered GetBudgetsPlanCell FieldsForCalculatingFundValues tests' # at t/db_dependent/Budgets.t line 1290. # Looks like you failed 1 test of 154. t/db_dependent/Budgets.t .. Dubious, test returned 1 (wstat 256, 0x100) Failed 1/154 subtests Test Summary Report ------------------- t/db_dependent/Budgets.t (Wstat: 256 Tests: 154 Failed: 1) Failed test: 154 Non-zero exit status: 1 Files=1, Tests=154, 5 wallclock secs ( 0.02 usr 0.01 sys + 3.41 cusr 0.68 csys = 4.12 CPU) Result: FAIL root@kohadevbox:koha(bug31631-qa)$ prove t/db_dependent/Budgets.t t/db_dependent/Budgets.t .. 2/154 Use of uninitialized value in multiplication (*) at /kohadevbox/koha/Koha/Acquisition/Order.pm line 515. Use of uninitialized value in numeric eq (==) at /kohadevbox/koha/Koha/Acquisition/Order.pm line 526. Use of uninitialized value in multiplication (*) at /kohadevbox/koha/Koha/Acquisition/Order.pm line 515. Use of uninitialized value in numeric eq (==) at /kohadevbox/koha/Koha/Acquisition/Order.pm line 526. # Failed test 'We expect this to be the tax exclusive value' # at t/db_dependent/Budgets.t line 1243. # got: '157.6' # expected: '147.36' # Failed test 'We expect this to be the tax exclusive value' # at t/db_dependent/Budgets.t line 1247. # got: '169.44' # expected: '157.6' # Looks like you failed 2 tests of 34. t/db_dependent/Budgets.t .. 154/154 # Failed test 'GetBudgetSpent GetBudgetOrdered GetBudgetsPlanCell FieldsForCalculatingFundValues tests' # at t/db_dependent/Budgets.t line 1290. # Looks like you failed 1 test of 154. t/db_dependent/Budgets.t .. Dubious, test returned 1 (wstat 256, 0x100) Failed 1/154 subtests Test Summary Report ------------------- t/db_dependent/Budgets.t (Wstat: 256 Tests: 154 Failed: 1) Failed test: 154 Non-zero exit status: 1 Files=1, Tests=154, 2 wallclock secs ( 0.02 usr 0.00 sys + 2.17 cusr 0.24 csys = 2.43 CPU) Result: FAIL root@kohadevbox:koha(bug31631-qa)$ prove t/db_dependent/Budgets.t t/db_dependent/Budgets.t .. 100/154 Use of uninitialized value in addition (+) at /kohadevbox/koha/Koha/Acquisition/Order.pm line 490. Use of uninitialized value in addition (+) at /kohadevbox/koha/Koha/Acquisition/Order.pm line 490. # Failed test 'We expect this to be the tax exclusive value' # at t/db_dependent/Budgets.t line 1270. # got: '157.6' # expected: '147.36' # Failed test 'We expect this value to be the tax exclusive value' # at t/db_dependent/Budgets.t line 1272. # got: '157.6' # expected: '147.36' # Failed test 'We expect this to be the tax exclusive value' # at t/db_dependent/Budgets.t line 1276. # got: '169.44' # expected: '157.6' # Failed test 'We expect this value to be the tax exclusive value' # at t/db_dependent/Budgets.t line 1278. # got: '169.44' # expected: '157.6' # Looks like you failed 4 tests of 34. t/db_dependent/Budgets.t .. 154/154 # Failed test 'GetBudgetSpent GetBudgetOrdered GetBudgetsPlanCell FieldsForCalculatingFundValues tests' # at t/db_dependent/Budgets.t line 1290. # Looks like you failed 1 test of 154. t/db_dependent/Budgets.t .. Dubious, test returned 1 (wstat 256, 0x100) Failed 1/154 subtests Test Summary Report ------------------- t/db_dependent/Budgets.t (Wstat: 256 Tests: 154 Failed: 1) Failed test: 154 Non-zero exit status: 1 Files=1, Tests=154, 3 wallclock secs ( 0.03 usr 0.00 sys + 2.31 cusr 0.37 csys = 2.71 CPU) Result: FAIL root@kohadevbox:koha(bug31631-qa)$ prove t/db_dependent/Budgets.t t/db_dependent/Budgets.t .. 2/154 Use of uninitialized value in addition (+) at /kohadevbox/koha/Koha/Acquisition/Order.pm line 490. Use of uninitialized value in addition (+) at /kohadevbox/koha/Koha/Acquisition/Order.pm line 490. # Failed test 'We expect this to be the tax exclusive value' # at t/db_dependent/Budgets.t line 1270. # got: '157.6' # expected: '147.36' # Failed test 'We expect this value to be the tax exclusive value' # at t/db_dependent/Budgets.t line 1272. # got: '157.6' # expected: '147.36' # Failed test 'We expect this to be the tax exclusive value' # at t/db_dependent/Budgets.t line 1276. # got: '169.44' # expected: '157.6' # Failed test 'We expect this value to be the tax exclusive value' # at t/db_dependent/Budgets.t line 1278. # got: '169.44' # expected: '157.6' # Looks like you failed 4 tests of 34. t/db_dependent/Budgets.t .. 154/154 # Failed test 'GetBudgetSpent GetBudgetOrdered GetBudgetsPlanCell FieldsForCalculatingFundValues tests' # at t/db_dependent/Budgets.t line 1290. # Looks like you failed 1 test of 154. t/db_dependent/Budgets.t .. Dubious, test returned 1 (wstat 256, 0x100) Failed 1/154 subtests Test Summary Report ------------------- t/db_dependent/Budgets.t (Wstat: 256 Tests: 154 Failed: 1) Failed test: 154 Non-zero exit status: 1 Files=1, Tests=154, 2 wallclock secs ( 0.03 usr 0.00 sys + 2.09 cusr 0.28 csys = 2.40 CPU) Result: FAIL root@kohadevbox:koha(bug31631-qa)$ prove t/db_dependent/Budgets.t t/db_dependent/Budgets.t .. 2/154 Use of uninitialized value in addition (+) at /kohadevbox/koha/Koha/Acquisition/Order.pm line 490. Use of uninitialized value in addition (+) at /kohadevbox/koha/Koha/Acquisition/Order.pm line 490. t/db_dependent/Budgets.t .. ok All tests successful. Files=1, Tests=154, 2 wallclock secs ( 0.01 usr 0.00 sys + 2.15 cusr 0.23 csys = 2.39 CPU) Result: PASS |