Addressed PR comments

Author: ezilber-akamai

test/fixtures/nodebalancers_123456_configs.json

@@ -40,7 +40,7 @@
       "ssl_commonname": "",
       "port": 80,
       "nodebalancer_id": 123456,
-      "cipher_suite": "recommended",
+      "cipher_suite": "none",
       "ssl_key": null,
       "nodes_status": {
         "up": 0,

test/unit/objects/nodebalancers_test.py

@@ -46,6 +46,23 @@ def test_get_config(self):
         self.assertEqual(config_udp.protocol, "udp")
         self.assertEqual(config_udp.udp_check_port, 12345)
 
+    def test_update_config_udp(self):
+        """
+        Tests that a config with a protocol of udp can be updated and that cipher suite is properly excluded in save()
+        """
+        with self.mock_put(
+            "nodebalancers/123456/configs/65431"
+        ) as m:
+            config = self.client.load(NodeBalancerConfig, 65431, 123456)
+            config.udp_check_port = 54321
+            config.save()
+
+            self.assertEqual(
+                m.call_url, "/nodebalancers/123456/configs/65431"
+            )
+            self.assertEqual(m.call_data["udp_check_port"], 54321)
+            self.assertNotIn("cipher_suite", m.call_data)
+
 
 class NodeBalancerNodeTest(ClientBaseCase):
     """