Fix ast_event_new unit test.
authorRussell Bryant <russell@russellbryant.com>
Mon, 30 Jul 2012 13:45:42 +0000 (13:45 +0000)
committerRussell Bryant <russell@russellbryant.com>
Mon, 30 Jul 2012 13:45:42 +0000 (13:45 +0000)
One of my recent commits broke this test.  The error was:

[test_event.c:event_new_test:214]: Events expected to be identical
have different size: 69 != 59

The difference in size occurred because the first event had
the EID IE added to the event twice.  ast_event_new() now always
adds it automatically.  Previously it only added it if there
were no IEs specified, which was kind of weird.

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@370541 65c4cc65-6c06-0410-ace0-fbb531ad65f3

tests/test_event.c

index 92fb4ec..9a4c241 100644 (file)
@@ -178,12 +178,6 @@ AST_TEST_DEFINE(event_new_test)
                goto return_cleanup;
        }
 
-       if (ast_event_append_eid(&event)) {
-               ast_test_status_update(test, "Failed to append EID\n");
-               res = AST_TEST_FAIL;
-               goto return_cleanup;
-       }
-
        if (check_event(event, test, type, "Custom", str, uint, bitflags)) {
                ast_test_status_update(test, "Dynamically generated event broken\n");
                res = AST_TEST_FAIL;