Refactoring NCCO Action to have get only property for action type

Removal of NccoConverter
Tests that use NCCO refactored

README.md

@@ -55,7 +55,7 @@ Changes in version 6.x
 
 * Enum values are now caplitalised in alignment with accepted coding practices
 * All classes in the Nexmo namespace that were marked as deprecated in 5.x are now removed
-* 
+* Ncco now inherits from List, it no longer has the `Actions` property, to add an action use `ncco.Add(action);`
 
 ## Targeted frameworks
 

Vonage.Test.Unit/Data/MultiInputTests/TestSerializeNccoAllProperties.json

@@ -1,5 +1,6 @@
 [
   {
+    "action": "input",
     "dtmf": {
       "timeOut": 3,
       "maxDigits": 1,
@@ -18,7 +19,6 @@
       ],
       "startTimeout": 5,
       "maxDuration": 30
-    },
-    "action": "input"
+    }
   }
 ]

Vonage.Test.Unit/Data/MultiInputTests/TestSerializeNccoAllPropertiesEmpty.json

@@ -1,12 +1,12 @@
 [
   {
+    "action": "input",
     "dtmf": {
     },
     "speech": {
       "uuid": [
         "aaaaaaaa-bbbb-cccc-dddd-0123456789ab"
       ]
-    },
-    "action": "input"
+    }
   }
 ]

Vonage.Test.Unit/Data/NccoTests/TestAppEndpoint-request.json

@@ -0,0 +1,4 @@
+{
+  "user": "steve",
+  "type": "app"
+}

Vonage.Test.Unit/Data/NccoTests/TestConnect-request.json

@@ -0,0 +1,26 @@
+[
+  {
+    "action": "connect",
+    "endpoint": [
+      {
+        "number": "447700900001",
+        "dtmfAnswer": "2p02p",
+        "onAnswer": {
+          "url": "https://example.com/answer",
+          "ringbackTone": "http://example.com/ringbackTone.wav"
+        },
+        "type": "phone"
+      }
+    ],
+    "from": "447700900000",
+    "eventType": "synchronous",
+    "timeout": "60",
+    "limit": "7200",
+    "machineDetection": "continue",
+    "eventUrl": [
+      "https://exampe.com/webhooks/events"
+    ],
+    "eventMethod": "POST",
+    "ringbackTone": "http://example.com/ringbackTone.wav"
+  }
+]

Vonage.Test.Unit/Data/NccoTests/TestConversation-request.json

@@ -0,0 +1,18 @@
+[
+  {
+    "action": "conversation",
+    "name": "vonage-conference-standard",
+    "musicOnHoldUrl": [
+      "https://example.com/music.mp3"
+    ],
+    "startOnEnter": "true",
+    "endOnExit": "false",
+    "record": "true",
+    "canSpeak": [
+      "6a4d6af0-55a6-4667-be90-8614e4c8e83c"
+    ],
+    "canHear": [
+      "6a4d6af0-55a6-4667-be90-8614e4c8e83c"
+    ]
+  }
+]

Vonage.Test.Unit/Data/NccoTests/TestNotify-request.json

@@ -0,0 +1,12 @@
+[
+  {
+    "action": "notify",
+    "payload": {
+      "Bar": "foo"
+    },
+    "eventUrl": [
+      "https://example.com/webhooks/events"
+    ],
+    "eventMethod": "POST"
+  }
+]

Vonage.Test.Unit/Data/NccoTests/TestRecord-request.json

@@ -0,0 +1,17 @@
+[
+  {
+    "action": "record",
+    "format": $format$,
+    "split": "conversation",
+    "channels": 2,
+    "endOnSilence": "3",
+    "endOnKey": "#",
+    "timeOut": "60",
+    "beepStart": "true",
+    "eventUrl": [
+      "https://example.com/record",
+      "https://test.com/record"
+    ],
+    "eventMethod": "POST"
+  }
+]

Vonage.Test.Unit/Data/NccoTests/TestRecordMinimalist-request.json

@@ -0,0 +1,5 @@
+[
+  {
+    "action": "record"
+  }
+]

Vonage.Test.Unit/Data/NccoTests/TestSipEndpoint-request.json

@@ -0,0 +1,7 @@
+{
+  "uri": "sip:rebekka@sip.example.com",
+  "headers": {
+    "Bar": "foo"
+  },
+  "type": "sip"
+}

Vonage.Test.Unit/Data/NccoTests/TestStream-request.json

@@ -0,0 +1,11 @@
+[
+  {
+    "action": "stream",
+    "streamUrl": [
+      "https://acme.com/streams/music.mp3"
+    ],
+    "level": "0",
+    "bargeIn": "true",
+    "loop": "2"
+  }
+]

Vonage.Test.Unit/Data/NccoTests/TestTalk-request.json

@@ -0,0 +1,12 @@
+[
+  {
+    "action": "talk",
+    "text": "Hello World",
+    "bargeIn": "true",
+    "loop": "2",
+    "level": "0",
+    "voiceName": "kimberly",
+    "language": "en-US",
+    "style": 0
+  }
+]

Vonage.Test.Unit/Data/NccoTests/TestTalkBareBones-request.json

@@ -0,0 +1,6 @@
+[
+  {
+    "action": "talk",
+    "text": "Hello World"
+  }
+]

Vonage.Test.Unit/Data/NccoTests/TestVbcEndpoint-request.json

@@ -0,0 +1,4 @@
+{
+  "extension": "4567",
+  "type": "vbc"
+}

Vonage.Test.Unit/Data/NccoTests/TestWebsocketEndpoint-request.json

@@ -0,0 +1,8 @@
+{
+  "uri": "wss://example.com/ws",
+  "content-type": "audio/l16;rate=16000",
+  "headers": {
+    "Bar": "foo"
+  },
+  "type": "websocket"
+}

Vonage.Test.Unit/Data/VoiceTests/CreateCall.json

@@ -13,8 +13,8 @@
   },
   "ncco": [
     {
-      "text": "Hello World",
-      "action": "talk"
+      "action": "talk",
+      "text": "Hello World"
     }
   ],
   "answer_url": [

Vonage.Test.Unit/Data/VoiceTests/CreateCallAsync.json

@@ -13,8 +13,8 @@
   },
   "ncco": [
     {
-      "text": "Hello World",
-      "action": "talk"
+      "action": "talk",
+      "text": "Hello World"
     }
   ],
   "answer_url": [

Vonage.Test.Unit/Data/VoiceTests/TestCreateCallWithRandomFromNumber.json

@@ -8,8 +8,8 @@
   ],
   "ncco": [
     {
-      "text": "Hello World",
-      "action": "talk"
+      "action": "talk",
+      "text": "Hello World"
     }
   ],
   "answer_url": [

Vonage.Test.Unit/MultiInputTests.cs

@@ -35,7 +35,7 @@ public void TestSerializeNccoAllProperties()
 
             //act
             var ncco = new Ncco(inputAction);
-            var actual = ncco.ToString();
+            var actual = JsonConvert.SerializeObject(ncco, Serialization.VonageSerialization.SerializerSettings);
 
             //assert
             Assert.Equal(expected, actual);
@@ -54,7 +54,7 @@ public void TestSerializeNccoAllPropertiesEmpty()
 
             //act
             var ncco = new Ncco(inputAction);
-            var actual = ncco.ToString();
+            var actual = JsonConvert.SerializeObject(ncco, Serialization.VonageSerialization.SerializerSettings);
 
             //assert
             Assert.Equal(expected, actual);