From 252ead3b132f62c1e97bc7657b4a720c4a33ec5a Mon Sep 17 00:00:00 2001 From: Matthew Jordan Date: Fri, 15 Aug 2014 15:36:44 +0000 Subject: [PATCH] app_voicemail/app: Remove test events that were duplicated by r421059 Moving the test event raised when a file is played back (which occurred in r421059) broke the ever loving snot out of the voicemail tests. This caused duplicate test events to get raised, as app_voicemail and main/app were raising events prior to call ast_streamfile. The voicemail tests did not enjoy getting multiple events. Since raising the playback event in ast_streamfile is far more useful to the vast majority of tests, this patch keeps the call there and simply removes the extraneous calls that duplicated the event. ........ Merged revisions 421125 from http://svn.asterisk.org/svn/asterisk/branches/1.8 git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/11@421164 65c4cc65-6c06-0410-ace0-fbb531ad65f3 --- apps/app_voicemail.c | 5 ----- main/app.c | 1 - 2 files changed, 6 deletions(-) diff --git a/apps/app_voicemail.c b/apps/app_voicemail.c index 4015866eba..2dd3615f41 100644 --- a/apps/app_voicemail.c +++ b/apps/app_voicemail.c @@ -7981,7 +7981,6 @@ static int forward_message(struct ast_channel *chan, char *context, struct vm_st } } else { /* Ask for an extension */ - ast_test_suite_event_notify("PLAYBACK", "Message: vm-extension"); res = ast_streamfile(chan, "vm-extension", ast_channel_language(chan)); /* "extension" */ prompt_played++; if (res || prompt_played > 4) @@ -10533,7 +10532,6 @@ static int vm_authenticate(struct ast_channel *chan, char *mailbox, int mailbox_ adsi_begin(chan, &useadsi); if (!skipuser && useadsi) adsi_login(chan); - ast_test_suite_event_notify("PLAYBACK", "Message: vm-login"); if (!silent && !skipuser && ast_streamfile(chan, "vm-login", ast_channel_language(chan))) { ast_log(AST_LOG_WARNING, "Couldn't stream login file\n"); return -1; @@ -10581,7 +10579,6 @@ static int vm_authenticate(struct ast_channel *chan, char *mailbox, int mailbox_ /* saved password is blank, so don't bother asking */ password[0] = '\0'; } else { - ast_test_suite_event_notify("PLAYBACK", "Message: %s", vm_password); if (ast_streamfile(chan, vm_password, ast_channel_language(chan))) { ast_log(AST_LOG_WARNING, "Unable to stream password file\n"); return -1; @@ -10618,13 +10615,11 @@ static int vm_authenticate(struct ast_channel *chan, char *mailbox, int mailbox_ logretries++; if (!valid) { if (skipuser || logretries >= max_logins) { - ast_test_suite_event_notify("PLAYBACK", "Message: vm-incorrect"); if (ast_streamfile(chan, "vm-incorrect", ast_channel_language(chan))) { ast_log(AST_LOG_WARNING, "Unable to stream incorrect message\n"); return -1; } } else { - ast_test_suite_event_notify("PLAYBACK", "Message: vm-incorrect-mailbox"); if (useadsi) adsi_login(chan); if (ast_streamfile(chan, "vm-incorrect-mailbox", ast_channel_language(chan))) { diff --git a/main/app.c b/main/app.c index 8900fdadac..1179233536 100644 --- a/main/app.c +++ b/main/app.c @@ -1085,7 +1085,6 @@ int ast_play_and_wait(struct ast_channel *chan, const char *fn) { int d = 0; - ast_test_suite_event_notify("PLAYBACK", "Message: %s\r\nChannel: %s", fn, ast_channel_name(chan)); if ((d = ast_streamfile(chan, fn, ast_channel_language(chan)))) { return d; }