[asterisk-dev] [Code Review] Use spaces to format XML and create SIPpVersion class

[Paul Belanger]

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

(Updated 2010-12-09 17:05:55.453341)


Review request for Asterisk Developers.


Changes
-------

v2: Fix logic issues.


Summary
-------

After running into some issue getting the testsuite running under Ubuntu 8.04, it seemed older versions of SIPp fail to parse tabs (/t) properly. So, I replaced them with 4 spaces.

I then ran into an issue of failing tests because my SIPp version did not have the appropriate compile time flags enabled, so I created a new SIPpVersion parser.  I used the AsteriskVersion class as a template, also creating various unit tests to test the parser (very cool and fun BTW).  I've defaulted our SIPp based tests to 'v3.0' however I know some will require TLS, PCAP or both.  Will need help to confirm which will use it.

Feedback is required :)


Diffs (updated)
-----

  asterisk/trunk/lib/python/sipp/version.py PRE-CREATION 
  asterisk/trunk/runtests.py 1127 
  asterisk/trunk/tests/cdr/app_dial_G_flag/sipp/call.xml 1127 
  asterisk/trunk/tests/cdr/app_dial_G_flag/sipp/wait-for-call.xml 1127 
  asterisk/trunk/tests/cdr/app_dial_G_flag/test-config.yaml 1127 
  asterisk/trunk/tests/cdr/app_queue/sipp/call-then-hangup.xml 1127 
  asterisk/trunk/tests/cdr/app_queue/sipp/call.xml 1127 
  asterisk/trunk/tests/cdr/app_queue/sipp/wait-for-call-hangup.xml 1127 
  asterisk/trunk/tests/cdr/app_queue/test-config.yaml 1127 
  asterisk/trunk/tests/cdr/blind-transfer-accountcode/sipp/call-then-blind-transfer.xml 1127 
  asterisk/trunk/tests/cdr/blind-transfer-accountcode/sipp/register.xml 1127 
  asterisk/trunk/tests/cdr/blind-transfer-accountcode/sipp/wait-for-call-do-hangup.xml 1127 
  asterisk/trunk/tests/cdr/blind-transfer-accountcode/sipp/wait-for-call.xml 1127 
  asterisk/trunk/tests/cdr/blind-transfer-accountcode/test-config.yaml 1127 
  asterisk/trunk/tests/cdr/originate-cdr-disposition/sipp/wait-for-call-busy.xml 1127 
  asterisk/trunk/tests/cdr/originate-cdr-disposition/sipp/wait-for-call-congestion.xml 1127 
  asterisk/trunk/tests/cdr/originate-cdr-disposition/sipp/wait-for-call-timeout.xml 1127 
  asterisk/trunk/tests/cdr/originate-cdr-disposition/sipp/wait-for-call.xml 1127 
  asterisk/trunk/tests/cdr/originate-cdr-disposition/test-config.yaml 1127 
  asterisk/trunk/tests/queues/position_priority_maxlen/test-config.yaml 1127 
  asterisk/trunk/tests/queues/ringinuse_and_pause/sipp/uas.xml 1127 
  asterisk/trunk/tests/queues/ringinuse_and_pause/test-config.yaml 1127 
  asterisk/trunk/tests/rfc2833_dtmf_detect/sipp/broken_dtmf.xml 1127 
  asterisk/trunk/tests/rfc2833_dtmf_detect/sipp/dtmf_baseline.xml 1127 
  asterisk/trunk/tests/rfc2833_dtmf_detect/sipp/dtmf_noend.xml 1127 
  asterisk/trunk/tests/rfc2833_dtmf_detect/sipp/register.xml 1127 
  asterisk/trunk/tests/rfc2833_dtmf_detect/test-config.yaml 1127 
  asterisk/trunk/tests/sip/handle_response_refer/sipp/wait-refer-200-notify.xml 1127 
  asterisk/trunk/tests/sip/handle_response_refer/sipp/wait-refer-202-error.xml 1127 
  asterisk/trunk/tests/sip/handle_response_refer/sipp/wait-refer-202-notify-provisional.xml 1127 
  asterisk/trunk/tests/sip/handle_response_refer/sipp/wait-refer-202-notify.xml 1127 
  asterisk/trunk/tests/sip/handle_response_refer/sipp/wait-refer-400.xml 1127 
  asterisk/trunk/tests/sip/handle_response_refer/sipp/wait-refer-500.xml 1127 
  asterisk/trunk/tests/sip/handle_response_refer/sipp/wait-refer-603.xml 1127 
  asterisk/trunk/tests/sip/handle_response_refer/test-config.yaml 1127 
  asterisk/trunk/tests/sip/options/sipp/options.xml 1127 
  asterisk/trunk/tests/sip/options/sipp/options2.xml 1127 
  asterisk/trunk/tests/sip/options/test-config.yaml 1127 
  asterisk/trunk/tests/sip_channel_params/sipp/call.xml 1127 
  asterisk/trunk/tests/sip_channel_params/sipp/register.xml 1127 
  asterisk/trunk/tests/sip_channel_params/test-config.yaml 1127 
  asterisk/trunk/tests/sip_outbound_address/sipp/uas1.xml 1127 
  asterisk/trunk/tests/sip_outbound_address/sipp/uas2.xml 1127 
  asterisk/trunk/tests/sip_outbound_address/test-config.yaml 1127 
  asterisk/trunk/tests/sip_register/sipp/registerv4.xml 1127 
  asterisk/trunk/tests/sip_register/sipp/registerv6.xml 1127 
  asterisk/trunk/tests/sip_register/test-config.yaml 1127 

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


Testing
-------

local Ubuntu, FreeBSD and CentOS system.


Thanks,

Paul

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.digium.com/pipermail/asterisk-dev/attachments/20101209/b5f3d77a/attachment.htm