Side by Side Diff: mojo/public/interfaces/bindings/tests/data/validation/conformance_msghdr_invalid_request_flags2.data
Issue 921223002:
Cpp bindings: Return false from Validator::Accept() on unrecognized message or invalid flags (Closed)
Base URL: https://github.com/domokit/mojo.git@master
Patch Set: Fix comments in JS test.
Created 5 years, 9 months ago
Use n/p to move between diff chunks;
N/P to move between comments.
Draft comments are only viewable by you.
On 2015/03/13 22:55:03, yzshen1 (OOO Mar 2 - 22) wrote:
> nit: please align the comment with other lines.
Done.
4 [u4]0 // flags: kMessageExpectsResponse is not set but expected
.
yzshen1
2015/03/13 22:55:03
Does it make sense to name this test conformance_m
Does it make sense to name this test conformance_mthd12_xxx? Because it is not
something that the message header validator can detect by itself. Instead,
knowledge about the actual interface running is required to detect the error.
Similarly, conformance_msghdr_invalid_request_flags1 may be renamed as
conformance_mthd0_xxx. (etc.)
What do you think?
Issue 921223002: Cpp bindings: Return false from Validator::Accept() on unrecognized message or invalid flags
(Closed)
Created 5 years, 10 months ago by rudominer
Modified 5 years, 8 months ago
Reviewers: yzshen1, hansmuller, hansmuller1
Base URL: https://github.com/domokit/mojo.git@master
Comments: 29