aws-mock-metadata icon indicating copy to clipboard operation
aws-mock-metadata copied to clipboard

Create package

Open totallyunknown opened this issue 7 years ago • 2 comments

I think it make sense to make this as a package, so we can use it as a mock for unit testing in kube2iam. What do you think?

totallyunknown avatar Jun 24 '17 10:06 totallyunknown

I definitely use the docker image to be able to test locally with minikube and we could use it for some kind of end to end tests but I think that would be pretty hard to setup and maintain. I'm looking at increasing unit test coverage first for which I think, I'd just use unit level mocks rather than running aws-mock-metadata. But I'm opened to suggestion, did you have something else in mind?

jtblin avatar Jun 25 '17 01:06 jtblin

https://github.com/jtblin/aws-mock-metadata/pull/10 adds a lot of mocked test coverage, which will get merged sometime shortly.

@totallyunknown can you elaborate on what the testing topology might look like with kube2iam, the way you vision things currently? I can see both sides of the argument (with limited exposure right now), re moving the source code into a subdirectory that can be used as a package, vs using a Docker based aws-mock-metadata to expose the mocks where required.

CpuID avatar Feb 28 '18 19:02 CpuID