Skip to content

use run-test command#15

Merged
Guru107 merged 23 commits intodevelopfrom
support-related-changes
Dec 21, 2025
Merged

use run-test command#15
Guru107 merged 23 commits intodevelopfrom
support-related-changes

Conversation

@Guru107
Copy link
Owner

@Guru107 Guru107 commented Dec 18, 2025

Note

Switches CI to use bench run-tests (enabling allow_tests) and removes india_compliance, while strengthening test bootstrap to reliably create companies/warehouses and conditionally set GST settings.

  • CI:
    • Use bench --site test_site run-tests with allow_tests instead of run-parallel-tests in .github/workflows/ci.yml.
    • Remove fetching/installing india_compliance app from CI install steps.
  • Tests setup (gate_entry/tests/__init__.py):
    • Add ensure_test_company_exists() and call it early; create _Test Company if missing.
    • Create Company only if "Wind Power LLP" is absent; populate extra fields (e.g., company_gstin, default_gst_rate, enable_audit_trail).
    • Make GST sandbox toggle conditional on DocType GST Settings existing; wrap in try/except.
    • Improve warehouse setup: trigger company.save(ignore_permissions=True) when none exist; create parent and child warehouses with ignore_permissions and without extra commits.
    • Reorder and refine set_default_company_for_tests(); save Global Defaults with ignore_permissions.
    • Minor refactors: relocate add_companies_to_fiscal_year, remove admin role enabling, extra cache/commit calls.

Written by Cursor Bugbot for commit d15f3ff. This will update automatically on new commits. Configure here.

Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This PR is being reviewed by Cursor Bugbot

Details

You are on the Bugbot Free tier. On this plan, Bugbot will review limited PRs each billing cycle.

To receive Bugbot reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.

@Guru107 Guru107 merged commit af2e4a8 into develop Dec 21, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant