New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Unit test did not break, but should have #73

Open

redstreet opened this issue Jul 7, 2023 · 1 comment

Owner

redstreet commented Jul 7, 2023

#72 should have broken the schwab_csv_brokerage unit test, which it did not. Fix.

redstreet changed the title ~~Unit test did not break~~ Unit test did not break, but should have

Owner Author

redstreet commented Jul 8, 2023

This is because Beancount regression tester calls file_account() with an already initialized importer, while bean-file calls it with an uninitialized importer.

Probably doesn't matter for Beancount, but matters for beancount_reds_importers.

m-d-brown mentioned this issue

Filing account incorrect with schwab_csv_brokerage #72

Closed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment