-
Notifications
You must be signed in to change notification settings - Fork 974
Sample Ledger data files (ledger-*.json) for testing & development? #4199
Comments
Perhaps we could also have an environment variable which can be used to tell Brave to use the mock Ledger data (and not try to update it). |
@willy-b - sorry for the delay in replying. it seems to me that by setting the NODE_ENV environment variable to either 'test' or 'dev', the browser will look in different folders for all the configuration files. on macOS, you'll get |
Thanks for the reply, @mrose17. The separation of user data by NODE_ENV is good, but in this issue I'm looking for your thoughts on a versioned sample data set. This would be useful for unit, integration, and manual testing to ensure that all features are working without forcing the tester to generate new data (which costs Bitcoin(!) and time). Also when making changes to the transaction data format, you could know right away whether any features were broken. I did add some mock data for the But it isn't generally useful like a sample set toggled by environment variable would be, like for manual testing of Payment History, which right now requires you to spend Bitcoin to see. |
Did you search for similar issues before submitting this one?
Yes
Describe the issue you encountered:
After updates to ledger-client / userdata formats, it takes a long time to fund a wallet, generate a browsing history, and trigger multiple reconciliations in order to have Ledger userdata to test against.
For multiple reconciliations, even triggering manually one can easily spend an hour on this (each time).
Expected behavior
Perhaps we could maintain a sample set of ledger-*.json files (short browsing history, a few reconciliations/transactions, always in latest format) for testing and development purposes. When updating userdata format or ledger-client this would be updated as well.
(Obviously the wallet associated with the sample data files would be unfunded but we need a standard to test against.)
Lately I've been running into some ledger errors and since it takes significant time to clear state, fund a wallet, visit pages, and get reconciliation to trigger (this is the long step), this would be extremely helpful for testing / UI work.
cc @mrose17
#4286
The text was updated successfully, but these errors were encountered: