macxred / cashctrl_ledger

Implementation of the abstract pyledger.LegderEngine interface with CashCtrl accounting service.
MIT License
0 stars 0 forks source link

Maintenance of GitHub Actions #47

Closed lasuk closed 4 months ago

lasuk commented 4 months ago

This PR combines several commits for the maintenance of continuous integration scripts. It is best reviewed commit by commit.

  1. a3692119 Clarifies the file name of the 'Unit/Integration Tests' GitHub workflow:

    check-release.yml -> run-tests.yml
  2. 0f5fed24 Modifies the configuration of the 'Restore CashCtrl initial state' workflow, such that the action fails if the restore_initial_state.py issues a warning. This makes a pre-existing 'invalid escape sequence' warning apparent, the job now fails. See job #59.

  3. 858fbb11 fixes the aforementioned warning. The job is now successful. See job #60.

    invalid escape sequence '\ '
      ACCOUNTS = """
  4. fa4d7ae0 Improves robustness of the 'Restore CashCtrl initial state' action. The change addresses a common Excpetion:

    RequestException: API call failed. The account '22000 Input Tax' is already referenced (e.g. in book entries)
    and cannot be deleted. You can set it as inactive instead.

    This small fix is very helpful in everyday life: After a failed test, we can now easily reset the test server to the original state by running below command in the console and do not need to log into the GUI any more.

    CC_API_ORGANISATION=<your-org-here> CC_API_KEY=<your-api-key> restore_initial_state.py