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

Unit tests and mock support for Client.get_inventory() is missing #444

Closed
andy-maier opened this issue Sep 18, 2017 · 0 comments · Fixed by #1549
Closed

Unit tests and mock support for Client.get_inventory() is missing #444

andy-maier opened this issue Sep 18, 2017 · 0 comments · Fixed by #1549

Comments

@andy-maier
Copy link
Member

PR #330 added support for Client.get_inventory(), without adding unit test code or mock support.

This issue is about adding:

  • mock support for Client.get_inventory()
  • unit tests for that mock support
  • unit tests for Client.get_inventory(), based upon its mock support
@andy-maier andy-maier self-assigned this Sep 18, 2017
@andy-maier andy-maier added this to the 0.17.0 milestone Sep 18, 2017
@andy-maier andy-maier modified the milestones: 0.17.0, 0.19.0 Sep 21, 2017
@andy-maier andy-maier modified the milestones: 0.19.0, 0.20.0 Mar 16, 2018
@andy-maier andy-maier modified the milestones: 0.20.0, 0.21.0 Oct 24, 2018
@andy-maier andy-maier modified the milestones: 0.21.0, 0.23.0 Jan 7, 2019
@andy-maier andy-maier modified the milestones: 0.23.0, 0.24.0 Apr 4, 2019
@andy-maier andy-maier modified the milestones: 0.24.0, 0.25.0 Aug 15, 2019
@andy-maier andy-maier removed this from the 0.25.0 milestone Dec 18, 2019
@andy-maier andy-maier removed their assignment Jan 12, 2024
@andy-maier andy-maier self-assigned this Jun 23, 2024
@andy-maier andy-maier added this to the 1.17.0 milestone Jun 23, 2024
@andy-maier andy-maier linked a pull request Jun 23, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant