Re: [asterisk-dev] [Code Review] 3145: res_ari: Add event tests for blind and attended transfers

2014-02-01 Thread Joshua Colp

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

(Updated Feb. 1, 2014, 4:29 p.m.)


Status
--

This change has been marked as submitted.


Review request for Asterisk Developers.


Repository: testsuite


Description
---

This adds tests for the blind and attended transfer events that are going to be 
added to ARI.


Diffs
-

  /asterisk/trunk/tests/rest_api/bridges/tests.yaml 4610 
  /asterisk/trunk/tests/rest_api/bridges/blind_transfer/test-config.yaml 
PRE-CREATION 
  /asterisk/trunk/tests/rest_api/bridges/blind_transfer/configs/ast1/pjsip.conf 
PRE-CREATION 
  
/asterisk/trunk/tests/rest_api/bridges/blind_transfer/configs/ast1/extensions.conf
 PRE-CREATION 
  /asterisk/trunk/tests/rest_api/bridges/blind_transfer/blind_transfer.py 
PRE-CREATION 
  /asterisk/trunk/tests/rest_api/bridges/attended_transfer/test-config.yaml 
PRE-CREATION 
  
/asterisk/trunk/tests/rest_api/bridges/attended_transfer/configs/ast1/pjsip.conf
 PRE-CREATION 
  
/asterisk/trunk/tests/rest_api/bridges/attended_transfer/configs/ast1/extensions.conf
 PRE-CREATION 
  /asterisk/trunk/tests/rest_api/bridges/attended_transfer/attended_transfer.py 
PRE-CREATION 

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


Testing
---

Ran test, fails using branches. Applied patch for events and it passes!


Thanks,

Joshua Colp

-- 
_
-- 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

Re: [asterisk-dev] [Code Review] 3145: res_ari: Add event tests for blind and attended transfers

2014-01-30 Thread Mark Michelson

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

Ship it!


Ship It!

- Mark Michelson


On Jan. 28, 2014, 3:55 p.m., Joshua Colp wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviewboard.asterisk.org/r/3145/
 ---
 
 (Updated Jan. 28, 2014, 3:55 p.m.)
 
 
 Review request for Asterisk Developers.
 
 
 Repository: testsuite
 
 
 Description
 ---
 
 This adds tests for the blind and attended transfer events that are going to 
 be added to ARI.
 
 
 Diffs
 -
 
   /asterisk/trunk/tests/rest_api/bridges/tests.yaml 4610 
   /asterisk/trunk/tests/rest_api/bridges/blind_transfer/test-config.yaml 
 PRE-CREATION 
   
 /asterisk/trunk/tests/rest_api/bridges/blind_transfer/configs/ast1/pjsip.conf 
 PRE-CREATION 
   
 /asterisk/trunk/tests/rest_api/bridges/blind_transfer/configs/ast1/extensions.conf
  PRE-CREATION 
   /asterisk/trunk/tests/rest_api/bridges/blind_transfer/blind_transfer.py 
 PRE-CREATION 
   /asterisk/trunk/tests/rest_api/bridges/attended_transfer/test-config.yaml 
 PRE-CREATION 
   
 /asterisk/trunk/tests/rest_api/bridges/attended_transfer/configs/ast1/pjsip.conf
  PRE-CREATION 
   
 /asterisk/trunk/tests/rest_api/bridges/attended_transfer/configs/ast1/extensions.conf
  PRE-CREATION 
   
 /asterisk/trunk/tests/rest_api/bridges/attended_transfer/attended_transfer.py 
 PRE-CREATION 
 
 Diff: https://reviewboard.asterisk.org/r/3145/diff/
 
 
 Testing
 ---
 
 Ran test, fails using branches. Applied patch for events and it passes!
 
 
 Thanks,
 
 Joshua Colp
 


-- 
_
-- 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

Re: [asterisk-dev] [Code Review] 3145: res_ari: Add event tests for blind and attended transfers

2014-01-28 Thread Joshua Colp


 On Jan. 24, 2014, 9:25 p.m., Mark Michelson wrote:
  /asterisk/trunk/tests/rest_api/bridges/attended_transfer/configs/ast1/extensions.conf,
   lines 3-6
  https://reviewboard.asterisk.org/r/3145/diff/1/?file=52985#file52985line3
 
  Both tests have an s extension, but they don't appear to be used for 
  anything.

ARI originates a call into them, starting the test kick off.


- Joshua


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


On Jan. 21, 2014, 12:48 p.m., Joshua Colp wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviewboard.asterisk.org/r/3145/
 ---
 
 (Updated Jan. 21, 2014, 12:48 p.m.)
 
 
 Review request for Asterisk Developers.
 
 
 Repository: testsuite
 
 
 Description
 ---
 
 This adds tests for the blind and attended transfer events that are going to 
 be added to ARI.
 
 
 Diffs
 -
 
   /asterisk/trunk/tests/rest_api/bridges/tests.yaml 4574 
   /asterisk/trunk/tests/rest_api/bridges/blind_transfer/test-config.yaml 
 PRE-CREATION 
   
 /asterisk/trunk/tests/rest_api/bridges/blind_transfer/configs/ast1/pjsip.conf 
 PRE-CREATION 
   
 /asterisk/trunk/tests/rest_api/bridges/blind_transfer/configs/ast1/extensions.conf
  PRE-CREATION 
   /asterisk/trunk/tests/rest_api/bridges/blind_transfer/blind_transfer.py 
 PRE-CREATION 
   /asterisk/trunk/tests/rest_api/bridges/attended_transfer/test-config.yaml 
 PRE-CREATION 
   
 /asterisk/trunk/tests/rest_api/bridges/attended_transfer/configs/ast1/pjsip.conf
  PRE-CREATION 
   
 /asterisk/trunk/tests/rest_api/bridges/attended_transfer/configs/ast1/extensions.conf
  PRE-CREATION 
   
 /asterisk/trunk/tests/rest_api/bridges/attended_transfer/attended_transfer.py 
 PRE-CREATION 
 
 Diff: https://reviewboard.asterisk.org/r/3145/diff/
 
 
 Testing
 ---
 
 Ran test, fails using branches. Applied patch for events and it passes!
 
 
 Thanks,
 
 Joshua Colp
 


-- 
_
-- 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

Re: [asterisk-dev] [Code Review] 3145: res_ari: Add event tests for blind and attended transfers

2014-01-28 Thread Joshua Colp

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

(Updated Jan. 28, 2014, 3:55 p.m.)


Review request for Asterisk Developers.


Repository: testsuite


Description
---

This adds tests for the blind and attended transfer events that are going to be 
added to ARI.


Diffs (updated)
-

  /asterisk/trunk/tests/rest_api/bridges/tests.yaml 4610 
  /asterisk/trunk/tests/rest_api/bridges/blind_transfer/test-config.yaml 
PRE-CREATION 
  /asterisk/trunk/tests/rest_api/bridges/blind_transfer/configs/ast1/pjsip.conf 
PRE-CREATION 
  
/asterisk/trunk/tests/rest_api/bridges/blind_transfer/configs/ast1/extensions.conf
 PRE-CREATION 
  /asterisk/trunk/tests/rest_api/bridges/blind_transfer/blind_transfer.py 
PRE-CREATION 
  /asterisk/trunk/tests/rest_api/bridges/attended_transfer/test-config.yaml 
PRE-CREATION 
  
/asterisk/trunk/tests/rest_api/bridges/attended_transfer/configs/ast1/pjsip.conf
 PRE-CREATION 
  
/asterisk/trunk/tests/rest_api/bridges/attended_transfer/configs/ast1/extensions.conf
 PRE-CREATION 
  /asterisk/trunk/tests/rest_api/bridges/attended_transfer/attended_transfer.py 
PRE-CREATION 

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


Testing
---

Ran test, fails using branches. Applied patch for events and it passes!


Thanks,

Joshua Colp

-- 
_
-- 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