jrwrigh / csv2cash

Python package for importing CSV files to GNUCash

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Check Compatibility with updated piecash version

jrwrigh opened this issue · comments

piecash has now updated to include support for GNUCash 3.0 (version 1.0.0 of piecash). Just need to double check to make sure it works.

any partucular features or just see if the example runs? I could write tests if its particular features. Or we could do a general compatibilty check (does it run) and test critical features (Transaction access, validation, etc)

Just checking to see if the example works is probably good enough. The previous version of csv2cash was running an early of branch of piecash that was merged without any changes iirc, so that should be good enough.

I just ran the example with piecash<=1.1.2 and it worked. All transactions are in test.gnucash and are categorized, paired, or ignored (in the duplicate test case) as needed. Building a test script to test the features and tell the users which, if any of the tests failed specifically would still be an improvement on running this example script.

Edit: I'll put together some test scripts and make a pull request.

I'll put together some test scripts and make a pull request.

That'd be awesome! I honestly haven't put too much priority on this project since it was kind of a one-use thing I made (and was primarily an excuse for playing around with Python). Implementing an actual testing script (even if it's not full unit testing) would be a great addition.