* chore: remove deprecated and empty QUnit tests
* ci: fix UI test config
Testing library was added on Frappe, in order to reuse command testing
library has to be installed during setup process.
(cherry picked from commit ae55eab599)
Co-authored-by: Ankush Menat <ankush@iwebnotes.com>
* fix(Sales Invoice): Let invoice be created for Sold Assets if it's a return invoice
* fix(Sales Invoice): Print appropriate message if item.asset is missing when the Item is a Fixed Asset
* fix(Sales Invoice): Fix GL Entry creation for Return Invoices linked with Assets
* fix(Sales Invoice): Print appropriate message if Asset isn't specified when the Item is a Fixed Asset
* fix(Sales Invoice): Let invoice be created for Sold Assets if it's a return invoice
* fix(Sales Invoice): Reset disposal_date on returning the Asset
* fix: Rename get_gl_entries_on_asset_movement to get_gl_entries_on_asset_disposal_and_regain
* fix: Make functions more readable
* fix: Let create_item() make items that are fixed assets
* fix: Test GL Entries made when an Asset is returned
* fix: Create asset data
* fix: Test
Co-authored-by: Saqib <nextchamp.saqib@gmail.com>
Summary : unable to create manual / auto asset depreciation entry when cost_center is mandatory
Reason: Though we are calculating value for depreciation_cost_center, it is not passed in credit_entry(it is passed in debit_entry) and this prevents creation of manual / auto asset depreciation entry when cost_center is mandatory
Solution : pass already calculated depreciation_cost_center value in credit_entry (in line with, already done as in debit_entry)
(cherry picked from commit b99c011947)
Co-authored-by: Ashish Shah <mr.ashish.shah@gmail.com>
* test: fix test due to rename change
* test: fix attendance request tests
- Use `frappe.db.get_value` instead of `get_doc` for asserting values
- Get values after cancellation as reloading attendance doc breaks due to stale doc (primary key changed after cancel of attendance request)
- rollback everything on tearDown
* test: fix Shift Request test
- Use `get_value` instead of `get_doc`
- Remove unnecessary loop, only one shift assignment is made against a shift request
- Get value after cancel again. Get doc is not reliable since primary key changed after cancel
* test: fix POS Closing Entry Test
- Separated into two tests, one checks if SI cancelling is blocked, the other checks PCE cancel impact
- This is done because after cancel via assertRaises, damage done by cancel still exists or is partially comitted
- Dont use this partially cancelled doc for any assertions further, end test at exception assertion
- Use `get_value` to check SI docstatus, as its primary key changes after cancel
* test: fixed asset movement tests
- set cwip account in company to avoid value missing
- removed unused statement
- removed trailing spaces
* Revert "test: fix POS Closing Entry Test"
This reverts commit 8f1a3aef2e.
Co-authored-by: marination <maricadsouza221197@gmail.com>
