required output can be mocked #247
Conversation
| ? result | ||
| : throw new KeyNotFoundException( | ||
| $"Required output '{v.Item1}' does not exist on stack '{v.Item2}'.")); | ||
| $"Required output '{v.Item1}' does not exist on stack '{base.GetResourceName()}'.")); |
There was a problem hiding this comment.
This will return the logical name of the stack reference resource rather than the name of the stack. Given the phrasing of the message that's probably not what's really wanted?
I think the issue here is that FakeStackOutputMocks doesn't return the name output.
There was a problem hiding this comment.
Yup raised #260 to add a test showing how to do this. Just requires the mock to return the "name" output as well as the "outputs" for the stack.
There was a problem hiding this comment.
Thanks, this test example is much clearer. I think I missed it as I was debugging through looking for .Name for resources and not .Type is there any documentation on the meaning of these different props on the args to mocks?
I am still getting a lot of errors in my test code, but I think they are probably just more missing mocks.
There was a problem hiding this comment.
is there any documentation on the meaning of these different props on the args to mocks?
Not that I know of, certainly doesn't look like anything on https://www.pulumi.com/docs/using-pulumi/testing/unit/#add-mocks. Worth raising a bug to the docs repo (https://github.com/pulumi/docs) asking for more written on this.
fixes the same issue as seen in issue #9212 for the typescript sdk.