slapos proxy/test: Assert integers in request/publish

slapos/tests/test_slapproxy/__init__.py

@@ -616,6 +616,33 @@ class TestRequest(MasterMixin):
     self.assertIsInstance(request, slapos.slap.ComputerPartition)
 
 
+  def test_request_with_int(self):
+    """
+    Verify that request with int parameters is correctly accepted
+    """
+    self.add_free_partition(1)
+    request = self.request('http://sr//', None, 'myinstance', 'slappart0',
+                           partition_parameter_kw={'int': 1})
+    self.assertIsInstance(request, slapos.slap.ComputerPartition)
+
+  def test_request_set_connection_parameters_with_int(self):
+    """
+    Cerify that request int works in connection parameters
+    """
+    self.add_free_partition(1)
+    partition_id = self.request('http://sr//', None, 'myinstance', 'slappart0')._partition_id
+    # Set connection parameter
+    partition = self.getPartitionInformation(partition_id)
+
+    self.setConnectionDict(partition_id=partition_id,
+                           connection_dict={'foo': 1})
+
+    # Get updated information for the partition
+    partition_new = self.request('http://sr//', None, 'myinstance', 'slappart0')
+    self.assertEqual(slave_partition.getConnectionParameter('foo'), 1)
+
+
+
 class TestSlaveRequest(MasterMixin):
   """
   Test requests related to slave instances.

[Bryton Lacquement]

Typo? Certify

[Łukasz Nowak]

Fixed.

[Bryton Lacquement]

Typo? partition_new, not slave_partition

[Łukasz Nowak]

Fixed.

[Bryton Lacquement]

(Related to nexedi/slapos@b5b79483)

int should be supported again, see nexedi/slapos.core@b8bd51d8.

Also, it should test the equality with '1', not 1.

[Łukasz Nowak]

Thanks, it works, I'll merge this MR.