diff options
author | Russell Bryant <russell@russellbryant.com> | 2010-04-22 21:06:53 +0000 |
---|---|---|
committer | Russell Bryant <russell@russellbryant.com> | 2010-04-22 21:06:53 +0000 |
commit | 52a8ddba515d5ea2bce69c8ba1d15b926f680f93 (patch) | |
tree | 81248e5756094401a6fee21329af4056f2b72562 /utils/extconf.c | |
parent | 78edf881d551560648730b2e2c439ed3a1ca1b2e (diff) |
Add ast_event subscription unit test and fix some ast_event API bugs.
This patch introduces another test in test_event.c that exercises most of the
subscription related ast_event API calls. I made some minor additions to the
existing event allocation test to increase API coverage by the test code.
Finally, I made a list in a comment of API calls not yet touched by the test
module as a to-do list for future test development.
During the development of this test code, I discovered a number of bugs in
the event API.
1) subscriptions to AST_EVENT_ALL were not handled appropriately in a couple
of different places. The API allows a subscription to all event types,
but with IE parameters, just as if it was a subscription to a specific
event type. However, the parameters were being ignored. This affected
ast_event_check_subscriber() and event distribution to subscribers.
2) Some of the logic in ast_event_check_subscriber() for checking subscriptions
against query parameters was wrong.
Review: https://reviewboard.asterisk.org/r/617/
git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@258632 65c4cc65-6c06-0410-ace0-fbb531ad65f3
Diffstat (limited to 'utils/extconf.c')
0 files changed, 0 insertions, 0 deletions