CatalogService: GetItemsByIds wrong behavior

[dmytrohridin]

Hi all,
In case when items list will be empty - Status will be set to 'NotFound', but then the value will be written to 'Ok'. Seems it's wrong behavior. If yes - PR has been created.

[dmytrohridin]

Hello @mvelosop
Can I ask you to look at this issue? Seems it's a bug. If not - feel free to close it.
Thank you :)

[sughosneo]

Hi @dmytrohridin, thank you for pointing that out. Yes, that's definitely a scope of improvement.
As the client await GrpcCallerService.CallService( relying on return this.MapToResponse( statement that's the reason it's not doing any check on ServerCallContext status. The correct status is useful for the unit test and for the client as well.

Looks good. Accepting the PR and closing the issue.