[PATCH] lib/audit_logging/test: fix typos
Philipp Gesang
philipp.gesang at intra2net.com
Thu Oct 25 06:36:19 UTC 2018
-<| Quoting Andrew Bartlett <abartlet at samba.org>, on Thursday, 2018-10-25 05:33:41 PM |>-
> Thanks for sending this in.
>
> Just a reminder to attach patches in future, rather than use git-send-
> email, as samba-technical rewrites the From and you get credited with
> an e-mail of samba-technical unless whoever merges this might notices
> and amends the author to fix it up.
Ok, thanks for the heads up.
Philipp
> Otherwise, the patch is fine.
>
> Reviewed-by: Andrew Bartlett <abartlet at samba.org>
>
> Andrew Bartlett
>
> On Fri, 2018-10-19 at 16:38 +0200, Philipp Gesang via samba-technical
> wrote:
> > Signed-off-by: Philipp Gesang <philipp.gesang at intra2net.com>
> > ---
> > lib/audit_logging/audit_logging.c | 4 +-
> > .../tests/audit_logging_error_test.c | 50 +++++++++------
> > ----
> > 2 files changed, 27 insertions(+), 27 deletions(-)
> >
> > diff --git a/lib/audit_logging/audit_logging.c
> > b/lib/audit_logging/audit_logging.c
> > index c274e971925..976b1c9a9fc 100644
> > --- a/lib/audit_logging/audit_logging.c
> > +++ b/lib/audit_logging/audit_logging.c
> > @@ -282,7 +282,7 @@ void audit_message_send(
> > * Create a new json object, the json_object wraps the underlying
> > json
> > * implementations JSON Object representation.
> > *
> > - * Free with a call to json_free_object, note that the jansson
> > inplementation
> > + * Free with a call to json_free_object, note that the jansson
> > implementation
> > * allocates memory with malloc and not talloc.
> > *
> > * @return a struct json_object, valid will be set to false if the
> > object
> > @@ -309,7 +309,7 @@ struct json_object json_new_object(void) {
> > * Create a new json object, the json_object wraps the underlying
> > json
> > * implementations JSON Array representation.
> > *
> > - * Free with a call to json_free_object, note that the jansson
> > inplementation
> > + * Free with a call to json_free_object, note that the jansson
> > implementation
> > * allocates memory with malloc and not talloc.
> > *
> > * @return a struct json_object, error will be set to true if the
> > array
> > diff --git a/lib/audit_logging/tests/audit_logging_error_test.c
> > b/lib/audit_logging/tests/audit_logging_error_test.c
> > index 1c0929a1b99..153e4f5b1fa 100644
> > --- a/lib/audit_logging/tests/audit_logging_error_test.c
> > +++ b/lib/audit_logging/tests/audit_logging_error_test.c
> > @@ -55,7 +55,7 @@
> >
> > #include "lib/audit_logging/audit_logging.h"
> >
> > -const int JANNASON_FAILURE = -1;
> > +const int JANSSON_FAILURE = -1;
> > const int CALL_ORIG = -2;
> >
> > /*
> > @@ -300,7 +300,7 @@ static void test_json_add_int(void **state)
> > * Test json object set new failure
> > */
> > will_return(__wrap_json_integer, false);
> > - will_return(__wrap_json_object_set_new, JANNASON_FAILURE);
> > + will_return(__wrap_json_object_set_new, JANSSON_FAILURE);
> > rc = json_add_int(&object, "name", 2);
> >
> > assert_false(json_is_invalid(&object));
> > @@ -320,7 +320,7 @@ static void test_json_add_bool(void **state)
> > * json_boolean does not return an error code.
> > * Test json object set new failure
> > */
> > - will_return(__wrap_json_object_set_new, JANNASON_FAILURE);
> > + will_return(__wrap_json_object_set_new, JANSSON_FAILURE);
> > rc = json_add_bool(&object, "name", true);
> >
> > assert_false(json_is_invalid(&object));
> > @@ -351,7 +351,7 @@ static void test_json_add_string(void **state)
> > * Test json object set new failure
> > */
> > will_return(__wrap_json_string, false);
> > - will_return(__wrap_json_object_set_new, JANNASON_FAILURE);
> > + will_return(__wrap_json_object_set_new, JANSSON_FAILURE);
> > rc = json_add_string(&object, "name", "value");
> >
> > assert_false(json_is_invalid(&object));
> > @@ -360,7 +360,7 @@ static void test_json_add_string(void **state)
> > /*
> > * Test json object set new failure for a NULL string
> > */
> > - will_return(__wrap_json_object_set_new, JANNASON_FAILURE);
> > + will_return(__wrap_json_object_set_new, JANSSON_FAILURE);
> > rc = json_add_string(&object, "null", NULL);
> >
> > assert_false(json_is_invalid(&object));
> > @@ -384,7 +384,7 @@ static void test_json_add_object(void **state)
> > /*
> > * Test json object set new failure
> > */
> > - will_return(__wrap_json_object_set_new, JANNASON_FAILURE);
> > + will_return(__wrap_json_object_set_new, JANSSON_FAILURE);
> > rc = json_add_object(&object, "name", &value);
> >
> > assert_false(json_is_invalid(&object));
> > @@ -394,7 +394,7 @@ static void test_json_add_object(void **state)
> > /*
> > * Test json object set new failure for a NULL value
> > */
> > - will_return(__wrap_json_object_set_new, JANNASON_FAILURE);
> > + will_return(__wrap_json_object_set_new, JANSSON_FAILURE);
> > rc = json_add_object(&object, "null", NULL);
> >
> > assert_false(json_is_invalid(&object));
> > @@ -419,7 +419,7 @@ static void test_json_add_to_array(void **state)
> > /*
> > * Test json array append new failure
> > */
> > - will_return(__wrap_json_array_append_new, JANNASON_FAILURE);
> > + will_return(__wrap_json_array_append_new, JANSSON_FAILURE);
> > rc = json_add_object(&array, "name", &value);
> >
> > assert_false(json_is_invalid(&array));
> > @@ -429,7 +429,7 @@ static void test_json_add_to_array(void **state)
> > /*
> > * Test json append new failure with a NULL value
> > */
> > - will_return(__wrap_json_array_append_new, JANNASON_FAILURE);
> > + will_return(__wrap_json_array_append_new, JANSSON_FAILURE);
> > rc = json_add_object(&array, "null", NULL);
> >
> > assert_false(json_is_invalid(&array));
> > @@ -461,7 +461,7 @@ static void test_json_add_timestamp(void **state)
> > will_return(__wrap_gettimeofday, 0);
> > will_return(__wrap_localtime, false);
> > will_return(__wrap_json_string, false);
> > - will_return(__wrap_json_object_set_new, JANNASON_FAILURE);
> > + will_return(__wrap_json_object_set_new, JANSSON_FAILURE);
> > rc = json_add_timestamp(&object);
> >
> > assert_false(json_is_invalid(&object));
> > @@ -511,7 +511,7 @@ static void test_json_add_stringn(void **state)
> > * Test json object set new failure
> > */
> > will_return(__wrap_json_string, false);
> > - will_return(__wrap_json_object_set_new, JANNASON_FAILURE);
> > + will_return(__wrap_json_object_set_new, JANSSON_FAILURE);
> > rc = json_add_stringn(&object, "name", "value", 3);
> >
> > assert_false(json_is_invalid(&object));
> > @@ -520,7 +520,7 @@ static void test_json_add_stringn(void **state)
> > /*
> > * Test json object set new failure for a NULL string
> > */
> > - will_return(__wrap_json_object_set_new, JANNASON_FAILURE);
> > + will_return(__wrap_json_object_set_new, JANSSON_FAILURE);
> > rc = json_add_stringn(&object, "null", NULL, 2);
> >
> > assert_false(json_is_invalid(&object));
> > @@ -529,7 +529,7 @@ static void test_json_add_stringn(void **state)
> > /*
> > * Test json object set new failure for a zero string size
> > */
> > - will_return(__wrap_json_object_set_new, JANNASON_FAILURE);
> > + will_return(__wrap_json_object_set_new, JANSSON_FAILURE);
> > rc = json_add_stringn(&object, "zero", "no value", 0);
> >
> > assert_false(json_is_invalid(&object));
> > @@ -567,7 +567,7 @@ static void test_json_add_version(void **state)
> >
> > will_return(__wrap_json_object, false);
> > will_return(__wrap_json_integer, false);
> > - will_return(__wrap_json_object_set_new, JANNASON_FAILURE);
> > + will_return(__wrap_json_object_set_new, JANSSON_FAILURE);
> > rc = json_add_version(&object, 2, 12);
> >
> > assert_false(json_is_invalid(&object));
> > @@ -588,7 +588,7 @@ static void test_json_add_version(void **state)
> > will_return(__wrap_json_integer, false);
> > will_return(__wrap_json_object_set_new, CALL_ORIG);
> > will_return(__wrap_json_integer, false);
> > - will_return(__wrap_json_object_set_new, JANNASON_FAILURE);
> > + will_return(__wrap_json_object_set_new, JANSSON_FAILURE);
> > rc = json_add_version(&object, 3, 13);
> >
> > assert_false(json_is_invalid(&object));
> > @@ -609,7 +609,7 @@ static void test_json_add_version(void **state)
> > will_return(__wrap_json_integer, false);
> > will_return(__wrap_json_object_set_new, CALL_ORIG);
> > will_return(__wrap_json_integer, false);
> > - will_return(__wrap_json_object_set_new, JANNASON_FAILURE);
> > + will_return(__wrap_json_object_set_new, JANSSON_FAILURE);
> > rc = json_add_version(&object, 4, 14);
> >
> > assert_false(json_is_invalid(&object));
> > @@ -635,7 +635,7 @@ static void test_json_add_address(void **state)
> > will_return(__wrap_json_object, false);
> > object = json_new_object();
> >
> > - will_return(__wrap_json_object_set_new, JANNASON_FAILURE);
> > + will_return(__wrap_json_object_set_new, JANSSON_FAILURE);
> > rc = json_add_address(&object, "name", NULL);
> >
> > assert_false(json_is_invalid(&object));
> > @@ -650,7 +650,7 @@ static void test_json_add_address(void **state)
> > will_return(__wrap_talloc_named_const, REAL_TALLOC);
> > will_return(__wrap_tsocket_address_string, false);
> > will_return(__wrap_json_string, false);
> > - will_return(__wrap_json_object_set_new, JANNASON_FAILURE);
> > + will_return(__wrap_json_object_set_new, JANSSON_FAILURE);
> > rc = json_add_address(&object, "name", ip);
> >
> > assert_false(json_is_invalid(&object));
> > @@ -698,7 +698,7 @@ static void test_json_add_sid(void **state)
> > will_return(__wrap_json_object, false);
> > object = json_new_object();
> >
> > - will_return(__wrap_json_object_set_new, JANNASON_FAILURE);
> > + will_return(__wrap_json_object_set_new, JANSSON_FAILURE);
> > rc = json_add_sid(&object, "null", NULL);
> > assert_int_equal(JSON_ERROR, rc);
> >
> > @@ -707,7 +707,7 @@ static void test_json_add_sid(void **state)
> > */
> > assert_true(string_to_sid(&sid, SID));
> > will_return(__wrap_json_string, false);
> > - will_return(__wrap_json_object_set_new, JANNASON_FAILURE);
> > + will_return(__wrap_json_object_set_new, JANSSON_FAILURE);
> > rc = json_add_sid(&object, "sid", &sid);
> > assert_int_equal(JSON_ERROR, rc);
> >
> > @@ -728,7 +728,7 @@ static void test_json_add_guid(void **state)
> > will_return(__wrap_json_object, false);
> > object = json_new_object();
> >
> > - will_return(__wrap_json_object_set_new, JANNASON_FAILURE);
> > + will_return(__wrap_json_object_set_new, JANSSON_FAILURE);
> > rc = json_add_guid(&object, "null", NULL);
> > assert_int_equal(JSON_ERROR, rc);
> >
> > @@ -738,7 +738,7 @@ static void test_json_add_guid(void **state)
> > status = GUID_from_string(GUID, &guid);
> > assert_true(NT_STATUS_IS_OK(status));
> > will_return(__wrap_json_string, false);
> > - will_return(__wrap_json_object_set_new, JANNASON_FAILURE);
> > + will_return(__wrap_json_object_set_new, JANSSON_FAILURE);
> > rc = json_add_guid(&object, "guid", &guid);
> > assert_int_equal(JSON_ERROR, rc);
> >
> > @@ -818,7 +818,7 @@ static void test_json_get_object(void **state)
> > {
> > struct json_object object;
> > struct json_object stored;
> > - struct json_object retreived;
> > + struct json_object retrieved;
> >
> > int rc;
> >
> > @@ -839,8 +839,8 @@ static void test_json_get_object(void **state)
> > */
> > will_return(__wrap_json_object, false);
> > will_return(__wrap_json_object_update, true);
> > - retreived = json_get_object(&object, "stored");
> > - assert_true(json_is_invalid(&retreived));
> > + retrieved = json_get_object(&object, "stored");
> > + assert_true(json_is_invalid(&retrieved));
> >
> > json_free(&object);
> > }
> --
> Andrew Bartlett http://samba.org/~abartlet/
> Authentication Developer, Samba Team http://samba.org
> Samba Developer, Catalyst IT http://catalyst.net.nz/services/samba
>
>
>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <http://lists.samba.org/pipermail/samba-technical/attachments/20181025/438893f5/signature.sig>
More information about the samba-technical
mailing list