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

[Fuel backend] Mock LocalCache and REST #17

Open
osrf-migration opened this issue Jul 14, 2017 · 0 comments
Open

[Fuel backend] Mock LocalCache and REST #17

osrf-migration opened this issue Jul 14, 2017 · 0 comments
Labels
bug Something isn't working tests Broken or missing tests / testing infra

Comments

@osrf-migration
Copy link

Original report (archived issue) by Shane Loretz (Bitbucket: Shane Loretz, GitHub: sloretz).


The classes LocalCache and REST can be optionally provided to FuelClient. This is meant to allow these classes to be mocked for unit tests. Currently the test directory has gtest, but not gmock, so the classes cannot be mocked. gmock should be added to this repo and unit tests using mocks of these classes should be written to test edge cases that are difficult to test with a real file system or REST server.

@osrf-migration osrf-migration added major bug Something isn't working labels Apr 15, 2020
@chapulina chapulina removed the major label May 26, 2020
@chapulina chapulina added the tests Broken or missing tests / testing infra label Jun 11, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working tests Broken or missing tests / testing infra
Projects
None yet
Development

No branches or pull requests

2 participants