test: change of test utility (backport #2660)#4276
test: change of test utility (backport #2660)#4276mergify[bot] wants to merge 4 commits intoversion-15-hotfixfrom
Conversation
(cherry picked from commit 48ba8b6) # Conflicts: # india_compliance/gst_india/doctype/bill_of_entry/test_bill_of_entry.py # india_compliance/gst_india/doctype/gst_return_log/test_gst_return_log.py # india_compliance/gst_india/doctype/gst_settings/test_gst_settings.py # india_compliance/gst_india/doctype/gstin/test_gstin.py # india_compliance/gst_india/doctype/gstr_3b_report/test_gstr_3b_report.py # india_compliance/gst_india/overrides/test_advance_payment_entry.py # india_compliance/gst_india/overrides/test_ineligible_itc.py # india_compliance/gst_india/overrides/test_purchase_invoice.py # india_compliance/gst_india/overrides/test_setup_wizard.py # india_compliance/gst_india/overrides/test_subcontracting_transaction.py # india_compliance/gst_india/overrides/test_transaction.py # india_compliance/gst_india/utils/gstr_2/test_gstr_2a.py # india_compliance/gst_india/utils/gstr_2/test_gstr_2b_v3_0.py # india_compliance/gst_india/utils/test_e_invoice.py # india_compliance/gst_india/utils/test_e_waybill.py # india_compliance/gst_india/utils/test_utils.py
(cherry picked from commit c67e9aa) # Conflicts: # .github/workflows/server-tests.yml
(cherry picked from commit 9d7ce33)
(cherry picked from commit 98e77f4) # Conflicts: # india_compliance/gst_india/report/hsn_wise_summary_of_outward_supplies/test_hsn_wise_summary_of_outward_supplies.py # india_compliance/gst_india/utils/test_gstin_info.py
|
Cherry-pick of 48ba8b6 has failed: Cherry-pick of c67e9aa has failed: Cherry-pick of 98e77f4 has failed: To fix up this pull request, you can check it out locally. See documentation: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/checking-out-pull-requests-locally |
Not up to standards ⛔🔴 Issues
|
| Category | Results |
|---|---|
| Security | 34 critical |
| CodeStyle | 21 minor |
🟢 Metrics 0 complexity
Metric Results Complexity 0
NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.
Confidence Score: 0/5This PR is not safe to merge — unresolved merge conflict markers in 21 files will cause immediate SyntaxErrors and CI workflow failure. Multiple P0 findings: raw Git conflict markers committed into Python source files and the CI YAML make every affected module unparseable. No test in those files can run, and the GitHub Actions workflow is invalid. All ~20 Python test files with conflict markers, and
|
| Filename | Overview |
|---|---|
| .github/workflows/server-tests.yml | Contains a raw Git merge conflict block — invalid YAML that will prevent the CI workflow from running. |
| india_compliance/gst_india/overrides/test_transaction.py | Contains 3 unresolved Git conflict blocks; Python cannot parse this file and all tests within it will fail. |
| india_compliance/gst_india/overrides/test_purchase_invoice.py | Contains 2 unresolved Git conflict blocks making the file unparseable. |
| india_compliance/gst_india/overrides/test_advance_payment_entry.py | Contains 2 unresolved Git conflict blocks making the file unparseable. |
| india_compliance/gst_india/overrides/test_subcontracting_transaction.py | Contains 2 unresolved Git conflict blocks making the file unparseable. |
| india_compliance/tests/init.py | Updates import from frappe.test_runner to frappe.tests.utils — clean change, no conflicts. |
| india_compliance/tests/test_records.json | Adds stock_uom: "Nos" to 4 item records and removes 3 POS Profile test records. |
| india_compliance/gst_india/overrides/test_sales_invoice.py | Migrates base class from unittest.TestCase to IntegrationTestCase — no conflicts, clean change. |
| india_compliance/gst_india/overrides/test_setup_wizard.py | New test file using IntegrationTestCase — no conflicts. |
Reviews (1): Last reviewed commit: "test: fix other dependencies for test ca..." | Re-trigger Greptile
| <<<<<<< HEAD | ||
| services: | ||
| mariadb: | ||
| image: mariadb:10.6 |
| <<<<<<< HEAD | ||
| ======= | ||
| from frappe.tests import IntegrationTestCase, change_settings | ||
| from frappe.utils import today | ||
| >>>>>>> 48ba8b67 (test: change of test utility) | ||
| from erpnext.accounts.doctype.purchase_invoice.purchase_invoice import ( |
There was a problem hiding this comment.
Unresolved Git merge conflict markers throughout the file
This file contains 3 unresolved conflict blocks (<<<<<<< HEAD / ======= / >>>>>>> 48ba8b67). Python will raise a SyntaxError when importing this module, so none of the test classes (TestTransaction, TestQuotationTransaction, TestSpecificTransactions, TestItemUpdate) will run. The same pattern appears in ~20 other files in this PR: test_purchase_invoice.py, test_advance_payment_entry.py, test_subcontracting_transaction.py, and others.
This is an automatic backport of pull request #2660 done by [Mergify](https://mergify.com).