Skip to content
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.

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

Dqb 28 choose meal #41

Merged
merged 6 commits into from
Apr 23, 2024
Merged

Dqb 28 choose meal #41

merged 6 commits into from
Apr 23, 2024

Conversation

jplhanna
Copy link
Owner

No description provided.

Command can choose first item on the menu or a random item
Replace command args with flag converter
Add pytest-mock to pyproject.toml to support rare cases of needing to patch code.
Run poetry upgrade to upgrade existing installed packages
Install factoryboy into pyproject.toml, and update ruff rules accordingly.
Installed and implemented factory boy through out tests
Maybe clause in factory was not working. will come back to this
@jplhanna jplhanna merged commit 9535cc4 into main Apr 23, 2024
1 check passed
@jplhanna jplhanna deleted the DQB-28-choose-meal branch April 23, 2024 03:12
jplhanna added a commit that referenced this pull request Jul 20, 2024
* DQB-28 Created command to choose from menu

Command can choose first item on the menu or a random item

* DQB-28 Introduce more of select command

Replace command args with flag converter

* DQB-28 Clean up select command code and create tests for controller

Add pytest-mock to pyproject.toml to support rare cases of needing to patch code.

* DQB-28 Upgrade python packages

Run poetry upgrade to upgrade existing installed packages

* DQB-28 Update tests to use factory boy and introduce factories

Install factoryboy into pyproject.toml, and update ruff rules accordingly.
Installed and implemented factory boy through out tests

* DQB-28 Delete broken factory

Maybe clause in factory was not working. will come back to this
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant