Exchange also parameters in tests

Author: vstinner

Lib/test/test_capi/test_tuple.py

@@ -67,22 +67,22 @@ def test_tuple_fromarray(self):
         tuple_fromarray = _testcapi.tuple_fromarray
 
         tup = tuple([i] for i in range(5))
-        copy = tuple_fromarray(tup)
+        copy = tuple_fromarray(len(tup), tup)
         self.assertEqual(copy, tup)
 
         tup = ()
-        copy = tuple_fromarray(tup)
+        copy = tuple_fromarray(0, tup)
         self.assertIs(copy, tup)
 
-        copy = tuple_fromarray(NULL, 0)
+        copy = tuple_fromarray(0, NULL)
         self.assertIs(copy, ())
 
         with self.assertRaises(SystemError):
-            tuple_fromarray(NULL, -1)
+            tuple_fromarray(-1, NULL)
         with self.assertRaises(SystemError):
-            tuple_fromarray(NULL, PY_SSIZE_T_MIN)
+            tuple_fromarray(PY_SSIZE_T_MIN, NULL)
         with self.assertRaises(MemoryError):
-            tuple_fromarray(NULL, PY_SSIZE_T_MAX)
+            tuple_fromarray(PY_SSIZE_T_MAX, NULL)
 
     def test_tuple_pack(self):
         # Test PyTuple_Pack()

Modules/_testcapi/tuple.c

@@ -107,9 +107,9 @@ _check_tuple_item_is_NULL(PyObject *Py_UNUSED(module), PyObject *args)
 static PyObject *
 tuple_fromarray(PyObject* Py_UNUSED(module), PyObject *args)
 {
+    Py_ssize_t size;
     PyObject *src;
-    Py_ssize_t size = UNINITIALIZED_SIZE;
-    if (!PyArg_ParseTuple(args, "O|n", &src, &size)) {
+    if (!PyArg_ParseTuple(args, "nO", &size, &src)) {
         return NULL;
     }
     if (src != Py_None && !PyTuple_Check(src)) {