tree dde41c0081014b7dac9621f69cf5b3faf038bffe
parent d6f32a18bda2c942c72dd533dcac3af73b48b0a2
author Edward Lee <edwarddl@google.com> 1750798229 +0000
committer Edward Lee <edwarddl@google.com> 1750819049 -0700

Always move subresponse to final response

Currently if the subresponse has a badly formatted error message,
then the subresponse gets cleared without adding the message.

Prevent this case from happening.

Also prevent returning out early before adding messageAnnotation.

We can have a default error code as our base before we override it
from the subresponse error code.

Added unit test to test this.

Query_test had to be updated, since it was wrong the entire time.

The subresponse should always have a error object and the message
info should be in the annotations.

Otherwise it does not follow the redfish spec for errors.

Tested:
Unit test

Google-Bug-Id: 426668117
Change-Id: I5c8c96daed37638967c3330f7bf03c0ff32e85d3
Signed-off-by: Edward Lee <edwarddl@google.com>