search

agritheory / test_utils

Common Test Utilities and Fixtures for AgriTheory Projects
2 stars 3 forks source link

feat(fixtures): fixtures data for customers #39

Closed MyuddinKhatri closed 4 weeks ago

MyuddinKhatri commented 1 month ago

Issue - https://github.com/agritheory/test_utils/issues/10 Collected fixtures data for customers from Autoreader, beam, inventory_tools, electronic_payments

MyuddinKhatri commented 1 month ago

@agritheory for tax_id under customer i have directly stores in fixtures instead of generating it randomly. Please let me know if I should keep this as it is change it to generate randomly.

MyuddinKhatri commented 1 month ago

@agritheory should we add separate create_address() and create_contact() as both suppliers and customers have addresses?We can call create_suppliers, create_customers first then create_address and create_contact ?

Alchez commented 1 month ago

@MyuddinKhatri I think it's better to just have the create_suppliers and create_customers commands create the downstream documents, since it seems less likely that an app would need just addresses or contacts.

agritheory commented 4 weeks ago

@MyuddinKhatri Can you rebase this to fix the pre-commit spelling issue?

MyuddinKhatri commented 4 weeks ago

@MyuddinKhatri Can you rebase this to fix the pre-commit spelling issue?

@agritheory codespell is complaining about pheonix Is this intentional?

agritheory commented 4 weeks ago

@agritheory codespell is complaining about pheonix Is this intentional?

No, let's change it to the most standard spelling. The number and variation of spellings of this particular word is wild. Here's an example that's local to me:

image