tree f91ce3cf7cb08c038233f7c7c7f39fd97a84cd0f
parent 03c51984b59a74e0cb9460d7df7b8594b2dc34b7
author Nick Maniscalco <maniscalco@google.com> 1560809752 +0000
committer CQ bot account: commit-bot@chromium.org <commit-bot@chromium.org> 1560809752 +0000

[zircon][ulib][zxtest] Swap args in ASSERT_OK to match (expected, actual) order

Change the order of the arguments passed to the underlying
_ASSERT_VAR_STATUS macros so that "expected" comes before
"actual". This matches what we do in the other macros and matches
_ASSERT_VAR_STATUS's "prototype".

Now, when an ASSERT_OK fails, the printed message correctly identifies
the tested argument as "actual".  For example,

    zx_status_t status = ZX_ERR_BAD_STATE;
    EXPECT_OK(ZX_ERR_BAD_STATE);

now yields:

    Expected: ZX_OK
    Actual  : status
    Which is: ZX_ERR_BAD_STATE

Test: Ran exiting integration tests and observed the output
Change-Id: I9e1af9e8c1d78e77f39506465282aa8bc5ae8745