aboutsummaryrefslogtreecommitdiffstats
path: root/sources/pyside2/tests
diff options
context:
space:
mode:
authorCristian Maureira-Fredes <cristian.maureira-fredes@qt.io>2019-01-07 12:46:13 +0100
committerCristian Maureira-Fredes <cristian.maureira-fredes@qt.io>2019-01-08 11:47:47 +0000
commitc8970b29d76415d41fa383cbe5eb77eefb191fa1 (patch)
tree3cadb0b721f8bd9778a07589fe693bc6e0a00210 /sources/pyside2/tests
parent1bd279b01d06eba0abbbb5fea0c12a6de2600cdd (diff)
[reg] Remove createIndex method that uses PyObject*
Even though we have support for Void*, having to import it as an additional class to be able to call a separate function, is adding too much complexity so something that should be easy to do, like creating an index for an itemmodel. The regression is a change that replaced the way of doing: QAbstractItemModel.createIndex(0, 0, PyObject*) to QAbstractItemModel.createIndex(0, 0, VoidPtr(PyObject*)) which is not really intuitive, and is generating other issues. Included modification in the registries too. Change-Id: Ie6112c6baeb4fc3b22fc78e7edeb66aa4a17c22b Fixes: PYSIDE-883 Reviewed-by: Christian Tismer <tismer@stackless.com> Reviewed-by: Qt CI Bot <qt_ci_bot@qt-project.org>
Diffstat (limited to 'sources/pyside2/tests')
-rw-r--r--sources/pyside2/tests/QtCore/qmodelindex_internalpointer_test.py12
-rw-r--r--sources/pyside2/tests/registry/exists_darwin_5_12_0_ci.py4
-rw-r--r--sources/pyside2/tests/registry/exists_opensuse423_5_12_0_ci.py4
-rw-r--r--sources/pyside2/tests/registry/exists_redhatenterpriselinuxworkstation74_5_12_0_ci.py4
-rw-r--r--sources/pyside2/tests/registry/exists_win32_5_12_0_ci.py4
5 files changed, 13 insertions, 15 deletions
diff --git a/sources/pyside2/tests/QtCore/qmodelindex_internalpointer_test.py b/sources/pyside2/tests/QtCore/qmodelindex_internalpointer_test.py
index a67bb380a..770600ea3 100644
--- a/sources/pyside2/tests/QtCore/qmodelindex_internalpointer_test.py
+++ b/sources/pyside2/tests/QtCore/qmodelindex_internalpointer_test.py
@@ -33,7 +33,6 @@
import sys
import unittest
from PySide2.QtCore import *
-from PySide2.support import VoidPtr
class MyModel (QAbstractListModel):
pass
@@ -51,21 +50,20 @@ class TestQModelIndexInternalPointer(unittest.TestCase):
def testInternalPointer(self):
#Test QAbstractListModel.createIndex and
- #QModelIndex.internalPointer
+ #QModelIndex.internalPointer with regular Python objects
obj = QObject()
- obj_ptr = VoidPtr(obj)
- idx = self.model.createIndex(0, 0, obj)
+ idx = self.model.createIndex(0, 0, "Hello")
i = idx.internalPointer()
- self.assertEqual(int(obj_ptr), int(i))
+ self.assertEqual(i, "Hello")
def testReferenceCounting(self):
#Test reference counting when retrieving data with
#QModelIndex.internalPointer
- o = QObject()
+ o = [1, 2, 3]
o_refcnt = sys.getrefcount(o)
idx = self.model.createIndex(0, 0, o)
ptr = idx.internalPointer()
- self.assertEqual(sys.getrefcount(o), o_refcnt)
+ self.assertEqual(sys.getrefcount(o), o_refcnt + 1)
def testIndexForDefaultDataArg(self):
diff --git a/sources/pyside2/tests/registry/exists_darwin_5_12_0_ci.py b/sources/pyside2/tests/registry/exists_darwin_5_12_0_ci.py
index a2b6aa14f..7a070abfe 100644
--- a/sources/pyside2/tests/registry/exists_darwin_5_12_0_ci.py
+++ b/sources/pyside2/tests/registry/exists_darwin_5_12_0_ci.py
@@ -157,7 +157,7 @@ if "PySide2.QtCore" in sys.modules:
"QAbstractItemModel.changePersistentIndexList": ('typing.List[int]', 'typing.List[int]'),
"QAbstractItemModel.checkIndex": ('PySide2.QtCore.QModelIndex', 'PySide2.QtCore.QAbstractItemModel.CheckIndexOptions'),
"QAbstractItemModel.columnCount": ('PySide2.QtCore.QModelIndex',),
- "QAbstractItemModel.createIndex": ('int', 'int', 'int'),
+ "QAbstractItemModel.createIndex": [('int', 'int', 'int'), ('int', 'int', 'object')],
"QAbstractItemModel.data": ('PySide2.QtCore.QModelIndex', 'int'),
"QAbstractItemModel.decodeData": ('int', 'int', 'PySide2.QtCore.QModelIndex', 'PySide2.QtCore.QDataStream'),
"QAbstractItemModel.dropMimeData": ('PySide2.QtCore.QMimeData', 'PySide2.QtCore.Qt.DropAction', 'int', 'int', 'PySide2.QtCore.QModelIndex'),
@@ -15202,7 +15202,7 @@ if "PySide2.QtHelp" in sys.modules:
"QHelpEngineCore.unregisterDocumentation": ('str',),
# class PySide2.QtHelp.QHelpIndexModel:
- "QHelpIndexModel.createIndex": [('int', 'int', 'int'), ('str',)],
+ "QHelpIndexModel.createIndex": [('int', 'int', 'int'), ('int', 'int', 'object'), ('str',)],
"QHelpIndexModel.filter": ('str', 'str'),
"QHelpIndexModel.isCreatingIndex": (),
"QHelpIndexModel.linksForKeyword": ('str',),
diff --git a/sources/pyside2/tests/registry/exists_opensuse423_5_12_0_ci.py b/sources/pyside2/tests/registry/exists_opensuse423_5_12_0_ci.py
index 5cb8a2306..6caa0947d 100644
--- a/sources/pyside2/tests/registry/exists_opensuse423_5_12_0_ci.py
+++ b/sources/pyside2/tests/registry/exists_opensuse423_5_12_0_ci.py
@@ -157,7 +157,7 @@ if "PySide2.QtCore" in sys.modules:
"QAbstractItemModel.changePersistentIndexList": ('typing.List[int]', 'typing.List[int]'),
"QAbstractItemModel.checkIndex": ('PySide2.QtCore.QModelIndex', 'PySide2.QtCore.QAbstractItemModel.CheckIndexOptions'),
"QAbstractItemModel.columnCount": ('PySide2.QtCore.QModelIndex',),
- "QAbstractItemModel.createIndex": ('int', 'int', 'int'),
+ "QAbstractItemModel.createIndex": [('int', 'int', 'int'), ('int', 'int', 'object')],
"QAbstractItemModel.data": ('PySide2.QtCore.QModelIndex', 'int'),
"QAbstractItemModel.decodeData": ('int', 'int', 'PySide2.QtCore.QModelIndex', 'PySide2.QtCore.QDataStream'),
"QAbstractItemModel.dropMimeData": ('PySide2.QtCore.QMimeData', 'PySide2.QtCore.Qt.DropAction', 'int', 'int', 'PySide2.QtCore.QModelIndex'),
@@ -15213,7 +15213,7 @@ if "PySide2.QtHelp" in sys.modules:
"QHelpEngineCore.unregisterDocumentation": ('str',),
# class PySide2.QtHelp.QHelpIndexModel:
- "QHelpIndexModel.createIndex": [('int', 'int', 'int'), ('str',)],
+ "QHelpIndexModel.createIndex": [('int', 'int', 'int'), ('int', 'int', 'object'), ('str',)],
"QHelpIndexModel.filter": ('str', 'str'),
"QHelpIndexModel.isCreatingIndex": (),
"QHelpIndexModel.linksForKeyword": ('str',),
diff --git a/sources/pyside2/tests/registry/exists_redhatenterpriselinuxworkstation74_5_12_0_ci.py b/sources/pyside2/tests/registry/exists_redhatenterpriselinuxworkstation74_5_12_0_ci.py
index 969ac36c8..c7b58c3fb 100644
--- a/sources/pyside2/tests/registry/exists_redhatenterpriselinuxworkstation74_5_12_0_ci.py
+++ b/sources/pyside2/tests/registry/exists_redhatenterpriselinuxworkstation74_5_12_0_ci.py
@@ -157,7 +157,7 @@ if "PySide2.QtCore" in sys.modules:
"QAbstractItemModel.changePersistentIndexList": ('typing.List[int]', 'typing.List[int]'),
"QAbstractItemModel.checkIndex": ('PySide2.QtCore.QModelIndex', 'PySide2.QtCore.QAbstractItemModel.CheckIndexOptions'),
"QAbstractItemModel.columnCount": ('PySide2.QtCore.QModelIndex',),
- "QAbstractItemModel.createIndex": ('int', 'int', 'int'),
+ "QAbstractItemModel.createIndex": [('int', 'int', 'int'), ('int', 'int', 'object')],
"QAbstractItemModel.data": ('PySide2.QtCore.QModelIndex', 'int'),
"QAbstractItemModel.decodeData": ('int', 'int', 'PySide2.QtCore.QModelIndex', 'PySide2.QtCore.QDataStream'),
"QAbstractItemModel.dropMimeData": ('PySide2.QtCore.QMimeData', 'PySide2.QtCore.Qt.DropAction', 'int', 'int', 'PySide2.QtCore.QModelIndex'),
@@ -15213,7 +15213,7 @@ if "PySide2.QtHelp" in sys.modules:
"QHelpEngineCore.unregisterDocumentation": ('str',),
# class PySide2.QtHelp.QHelpIndexModel:
- "QHelpIndexModel.createIndex": [('int', 'int', 'int'), ('str',)],
+ "QHelpIndexModel.createIndex": [('int', 'int', 'int'), ('int', 'int', 'object'), ('str',)],
"QHelpIndexModel.filter": ('str', 'str'),
"QHelpIndexModel.isCreatingIndex": (),
"QHelpIndexModel.linksForKeyword": ('str',),
diff --git a/sources/pyside2/tests/registry/exists_win32_5_12_0_ci.py b/sources/pyside2/tests/registry/exists_win32_5_12_0_ci.py
index 189e5c74d..0a3f587f3 100644
--- a/sources/pyside2/tests/registry/exists_win32_5_12_0_ci.py
+++ b/sources/pyside2/tests/registry/exists_win32_5_12_0_ci.py
@@ -159,7 +159,7 @@ if "PySide2.QtCore" in sys.modules:
"QAbstractItemModel.changePersistentIndexList": ('typing.List', 'typing.List'),
"QAbstractItemModel.checkIndex": ('PySide2.QtCore.QModelIndex', 'PySide2.QtCore.QAbstractItemModel.CheckIndexOptions'),
"QAbstractItemModel.columnCount": ('PySide2.QtCore.QModelIndex',),
- "QAbstractItemModel.createIndex": ('int', 'int', 'int'),
+ "QAbstractItemModel.createIndex": [('int', 'int', 'int'), ('int', 'int', 'object')],
"QAbstractItemModel.data": ('PySide2.QtCore.QModelIndex', 'int'),
"QAbstractItemModel.decodeData": ('int', 'int', 'PySide2.QtCore.QModelIndex', 'PySide2.QtCore.QDataStream'),
"QAbstractItemModel.dropMimeData": ('PySide2.QtCore.QMimeData', 'PySide2.QtCore.Qt.DropAction', 'int', 'int', 'PySide2.QtCore.QModelIndex'),
@@ -15335,7 +15335,7 @@ if "PySide2.QtHelp" in sys.modules:
"QHelpEngineCore.unregisterDocumentation": ('str',),
# class PySide2.QtHelp.QHelpIndexModel:
- "QHelpIndexModel.createIndex": [('int', 'int', 'int'), ('str',)],
+ "QHelpIndexModel.createIndex": [('int', 'int', 'int'), ('int', 'int', 'object'), ('str',)],
"QHelpIndexModel.filter": ('str', 'str'),
"QHelpIndexModel.isCreatingIndex": (),
"QHelpIndexModel.linksForKeyword": ('str',),