search

macxred / pyledger

Python package to streamline the implementation and management of accounting systems.
MIT License
0 stars 0 forks source link

Coherent naming in base tests #63

Closed lasuk closed 6 days ago

lasuk commented 6 days ago

This PR follows up on #62 and align names in ledger base test with other base tests.

Changes:

  1. Use engine rather than pristine_engine to name the fixture that instantiates the accounting engine, as in other base tests.
  2. Rename fixture that represents a pre-populated accounting engine for testing to restore_engine.
  3. Clear any existing ledger entries in restore_engine. This will be helpful when testing connections to external systems such as CashCtrl, that will not start from a pristine state.
codecov-commenter commented 6 days ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Flag Coverage Δ
unittests 83.67% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

AlexTheWizardL commented 6 days ago

Thanks for the changes