[asterisk-dev] [Code Review] regression test for r1570 (app_voicemail, bad load_config order)

wdoekes reviewboard at asterisk.org
Tue Dec 6 13:06:10 CST 2011


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/1572/
-----------------------------------------------------------

(Updated Dec. 6, 2011, 1:06 p.m.)


Review request for Asterisk Developers.


Changes
-------

Updated.


Summary
-------

Added a regression test for my fix in r1570. It's not real pretty, but it does the job: show that the config load order is slightly broken.

Regard it as a stub that could be expanded whenever someone wants to fiddle with the app_voicemail configuration options.


Diffs (updated)
-----

  /branches/1.8/apps/app_voicemail.c 347109 

Diff: https://reviewboard.asterisk.org/r/1572/diff


Testing
-------

The test case does this, until r1570 is committed: first run fails, second run succeeds.

*CLI> test execute category /apps/app_voicemail/ name test_voicemail_load_config
Running all available tests matching category /apps/app_voicemail/ and name test_voicemail_load_config

START  /apps/app_voicemail/ - test_voicemail_load_config 
[app_voicemail.c:test_voicemail_load_config:13002]: mailbox 00000002 should have locale 'de_DE.UTF-8', but has ''
Failed to load configuration file.
END    /apps/app_voicemail/ - test_voicemail_load_config Time: <1ms Result: FAIL

1 Test(s) Executed  0 Passed  1 Failed
*CLI> test execute category /apps/app_voicemail/ name test_voicemail_load_config
Running all available tests matching category /apps/app_voicemail/ and name test_voicemail_load_config

START  /apps/app_voicemail/ - test_voicemail_load_config 
Failed to load configuration file.
END    /apps/app_voicemail/ - test_voicemail_load_config Time: <1ms Result: PASS

1 Test(s) Executed  1 Passed  0 Failed*CLI> test execute category /apps/app_voicemail/ name test_voicemail_load_config
Running all available tests matching category /apps/app_voicemail/ and name test_voicemail_load_config

START  /apps/app_voicemail/ - test_voicemail_load_config 
[app_voicemail.c:test_voicemail_load_config:13002]: mailbox 00000002 should have locale 'de_DE.UTF-8', but has ''
Failed to load configuration file.
END    /apps/app_voicemail/ - test_voicemail_load_config Time: <1ms Result: FAIL

1 Test(s) Executed  0 Passed  1 Failed
*CLI> test execute category /apps/app_voicemail/ name test_voicemail_load_config
Running all available tests matching category /apps/app_voicemail/ and name test_voicemail_load_config

START  /apps/app_voicemail/ - test_voicemail_load_config 
Failed to load configuration file.
END    /apps/app_voicemail/ - test_voicemail_load_config Time: <1ms Result: PASS

1 Test(s) Executed  1 Passed  0 Failed


When r1570 is also applied, the first run of test succeeds too, like expected.


Thanks,

wdoekes

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20111206/e68e4c1b/attachment.htm>


More information about the asterisk-dev mailing list