I'm confused about how this can be? Unless I'm missing something a module dependency doesn't effect how the test runs, just if it runs. If chan_sip is not required then shouldn't we remove sip.conf from each of these tests config dirs?
---------- Forwarded message ---------- From: SVN commits to the Asterisk project <asterisk-comm...@lists.digium.com> Date: Thu, Oct 30, 2014 at 1:28 PM Subject: [asterisk-commits] mjordan: testsuite/asterisk/trunk r5811 - in /asterisk/trunk/tests/fax/pjsip:... To: asterisk-comm...@lists.digium.com, svn-comm...@lists.digium.com Author: mjordan Date: Thu Oct 30 12:28:43 2014 New Revision: 5811 URL: http://svnview.digium.com/svn/testsuite?view=rev&rev=5811 Log: tests/fax/pjsip: Fix failing PJSIP fax tests The tests were failing due to having an erroneous dependency on chan_sip. That has now been corrected in each test's test-config.yaml. Modified: asterisk/trunk/tests/fax/pjsip/gateway_native_t38/test-config.yaml asterisk/trunk/tests/fax/pjsip/gateway_t38_g711/test-config.yaml asterisk/trunk/tests/fax/pjsip/t38/test-config.yaml Modified: asterisk/trunk/tests/fax/pjsip/gateway_native_t38/test-config.yaml URL: http://svnview.digium.com/svn/testsuite/asterisk/trunk/tests/fax/pjsip/gateway_native_t38/test-config.yaml?view=diff&rev=5811&r1=5810&r2=5811 ============================================================================== --- asterisk/trunk/tests/fax/pjsip/gateway_native_t38/test-config.yaml (original) +++ asterisk/trunk/tests/fax/pjsip/gateway_native_t38/test-config.yaml Thu Oct 30 12:28:43 2014 @@ -21,7 +21,6 @@ - python : 'twisted' - python : 'starpy' - custom : 'fax' - - asterisk : 'chan_sip' - asterisk : 'chan_pjsip' - asterisk : 'res_pjsip_t38' tags: Modified: asterisk/trunk/tests/fax/pjsip/gateway_t38_g711/test-config.yaml URL: http://svnview.digium.com/svn/testsuite/asterisk/trunk/tests/fax/pjsip/gateway_t38_g711/test-config.yaml?view=diff&rev=5811&r1=5810&r2=5811 ============================================================================== --- asterisk/trunk/tests/fax/pjsip/gateway_t38_g711/test-config.yaml (original) +++ asterisk/trunk/tests/fax/pjsip/gateway_t38_g711/test-config.yaml Thu Oct 30 12:28:43 2014 @@ -23,7 +23,6 @@ - python : 'twisted' - python : 'starpy' - custom : 'fax' - - asterisk : 'chan_sip' - asterisk : 'chan_pjsip' - asterisk : 'res_pjsip_t38' tags: Modified: asterisk/trunk/tests/fax/pjsip/t38/test-config.yaml URL: http://svnview.digium.com/svn/testsuite/asterisk/trunk/tests/fax/pjsip/t38/test-config.yaml?view=diff&rev=5811&r1=5810&r2=5811 ============================================================================== --- asterisk/trunk/tests/fax/pjsip/t38/test-config.yaml (original) +++ asterisk/trunk/tests/fax/pjsip/t38/test-config.yaml Thu Oct 30 12:28:43 2014 @@ -10,7 +10,6 @@ - python : 'twisted' - python : 'starpy' - custom : 'fax' - - asterisk : 'chan_sip' - asterisk : 'chan_pjsip' - asterisk : 'res_pjsip_t38' tags: -- _____________________________________________________________________ -- Bandwidth and Colocation Provided by http://www.api-digital.com -- asterisk-commits mailing list To UNSUBSCRIBE or update options visit: http://lists.digium.com/mailman/listinfo/asterisk-commits -- _____________________________________________________________________ -- Bandwidth and Colocation Provided by http://www.api-digital.com -- asterisk-dev mailing list To UNSUBSCRIBE or update options visit: http://lists.digium.com/mailman/listinfo/asterisk-dev