Use Transfer because Forward doesn't work once answered

Stephen Paul Weber created 3 years ago

Change summary

test/test_web.rb            | 12 ++++++------
views/outbound/connect.slim |  3 ++-
2 files changed, 8 insertions(+), 7 deletions(-)

Detailed changes

test/test_web.rb

@@ -106,8 +106,8 @@ class WebTest < Minitest::Test
 		assert last_response.ok?
 		assert_equal(
 			"<?xml version=\"1.0\" encoding=\"utf-8\" ?><Response>" \
-			"<Forward from=\"[phone]\" to=\"[phone]\" />" \
-			"</Response>",
+			"<Transfer transferCallerId=\"[phone]\">" \
+			"<PhoneNumber>[phone]</PhoneNumber></Transfer></Response>",
 			last_response.body
 		)
 	end
@@ -181,8 +181,8 @@ class WebTest < Minitest::Test
 		assert last_response.ok?
 		assert_equal(
 			"<?xml version=\"1.0\" encoding=\"utf-8\" ?><Response>" \
-			"<Forward from=\"[phone]\" to=\"[phone]\" />" \
-			"</Response>",
+			"<Transfer transferCallerId=\"[phone]\">" \
+			"<PhoneNumber>[phone]</PhoneNumber></Transfer></Response>",
 			last_response.body
 		)
 		assert_mock ExpiringLock::REDIS
@@ -251,8 +251,8 @@ class WebTest < Minitest::Test
 		assert last_response.ok?
 		assert_equal(
 			"<?xml version=\"1.0\" encoding=\"utf-8\" ?><Response>" \
-			"<Forward from=\"[phone]\" to=\"[phone]\" />" \
-			"</Response>",
+			"<Transfer transferCallerId=\"[phone]\">" \
+			"<PhoneNumber>[phone]</PhoneNumber></Transfer></Response>",
 			last_response.body
 		)
 	end

views/outbound/connect.slim

@@ -1,3 +1,4 @@
 doctype xml
 Response
-	Forward from=from to=to /
+	Transfer transferCallerId=from
+		PhoneNumber= to