Skip to content
This repository has been archived by the owner on Jul 16, 2021. It is now read-only.

API Server - Add "Repo Charts Fetcher" interface #53

Open
jackfrancis opened this issue Oct 27, 2016 · 0 comments
Open

API Server - Add "Repo Charts Fetcher" interface #53

jackfrancis opened this issue Oct 27, 2016 · 0 comments

Comments

@jackfrancis
Copy link
Contributor

We're currently relying on a black box implementation inside the data.Charts.Refresh method. For the data.cache.cachedCharts implementation of data.Charts specifically, we have an http.Get implementation that operates against the URLs in the knownRepos instance slice map.

Less specifically, what we want is to be able to pass an interface into Refresh so that we can provide a fetcher implementation that is backed by mock data, for tests. Until then we're relying on actual HTTP connections to the prod repos to test data.cache.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants