Feat: move txn_to_str() to ledger engine

macxred / pyledger

Python package to streamline the implementation and management of accounting systems.

MIT License

0 stars 0 forks source link

Closed AlexTheWizardL closed 2 months ago

AlexTheWizardL commented 2 months ago

Summary of Changes:

Moved the txn_to_str() method to the LedgerEngine class to provide a consistent, unique representation of ledger transactions in CSV-like string format.
Updated the method to ensure that both single and collective transactions are correctly represented and sortable for comparison.
Created comprehensive test cases for txn_to_str()

@lasuk Please review the changes and provide feedback or approval for merging.

codecov-commenter commented 2 months ago

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

Flag	Coverage Δ
unittests	`64.54% <100.00%> (+0.26%)`	:arrow_up:

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

Files with missing lines	Coverage Δ
pyledger/ledger_engine.py	`65.43% <100.00%> (+0.56%)`	:arrow_up:

lasuk commented 2 months ago

Ready to merge.

I've added four additional commits to beautify and improve. @AlexTheWizardL: Please shout if you disagree.