summaryrefslogtreecommitdiff
path: root/tests
Commit message (Collapse)AuthorAgeFilesLines
* Tests: Fix flamegraphview test for Qt6Christian Stenger2022-08-041-5/+5
| | | | | Change-Id: I38271f05c615290cddb4dba5579193a68fd32370 Reviewed-by: Ulf Hermann <ulf.hermann@qt.io>
* Tests: Fix dumper expectationChristian Stenger2022-08-041-1/+1
| | | | | | | Amends c1d75d8df715b. Change-Id: Ib69c095a9f94fa618f5c9791d4c908fbefa5ecb3 Reviewed-by: hjk <hjk@qt.io>
* Debugger: Fix bitfield dumping with Python 3hjk2022-08-031-0/+23
| | | | | Change-Id: I2959a2e14c6f25e29460de74ea95a16a5226d48e Reviewed-by: Christian Stenger <christian.stenger@qt.io>
* QmlDesigner: Implement support for SignalDeclarationPropertyThomas Hartmann2022-07-202-1/+141
| | | | | | | | | | | This allows implementing a signal with a signature. e.g. signal mySignal(int i) Task-number: QDS-7319 Change-Id: I23cb000a218d709218322e7f31c86076d3ad949b Reviewed-by: Qt CI Bot <qt_ci_bot@qt-project.org> Reviewed-by: Marco Bubke <marco.bubke@qt.io> Reviewed-by: <github-actions-qt-creator@cristianadam.eu>
* deviceshell: Fix racecondition and long running testsMarcus Tillmanns2022-07-181-4/+20
| | | | | | | | | | | | | | | When writing to stdout and stderr from two processes, their output could become interleaved. To work around that, we write stdout and stderr to different files and later combine them together in the shell script. Since tst_deviceshell tests could run for a long time if /usr folder is too big, added a check that first tests the runtime once. Since we currently only support linux containers, limit the tests to only run if the container platform is linux as well. Change-Id: I4b313596cdf9acc839d54d7cc77c66fd53ac23bf Reviewed-by: hjk <hjk@qt.io>
* QmlDesigner: Add support for BehavioursThomas Hartmann2022-07-132-0/+98
| | | | | | | | | | | | | | A Behavior will be added as a normal ModelNode to the default property, but we store the property name in behaviorPropertyName. The value of behaviorPropertyName cannot be changed after the ModelNode was created, since I do not see any use case and it keeps things simple. Change-Id: I69ba1d4d706432cfbbd35b001238f623e6e0b4fd Reviewed-by: Qt CI Bot <qt_ci_bot@qt-project.org> Reviewed-by: Marco Bubke <marco.bubke@qt.io> Reviewed-by: <github-actions-qt-creator@cristianadam.eu>
* QmlDesigner: Fix testsThomas Hartmann2022-07-132-6/+6
| | | | | | | | We do not allow rect as id, since it is also a QML type. Change-Id: I348071b9b1f6c3e72d3934770196421ce72d36b5 Reviewed-by: <github-actions-qt-creator@cristianadam.eu> Reviewed-by: Thomas Hartmann <thomas.hartmann@qt.io>
* tst_QtcProcess: Increase the timeoutJarek Kobus2022-07-121-1/+1
| | | | | | | | Fix recursiveCrashingProcess() flakiness. Change-Id: I0d62368ccb5b67be691a787b6fa69717a4f024d1 Reviewed-by: Qt CI Bot <qt_ci_bot@qt-project.org> Reviewed-by: Eike Ziller <eike.ziller@qt.io>
* COIN: Arm auto testsEike Ziller2022-07-059-1/+12
| | | | | | | | | | | | | | | | | | | Some of our auto tests fail, so add an EXCLUDE_FROM_PRECHECK option for add_qtc_test, which sets a corresponding label on the ctest, and exclude these when running ctest on COIN. Disable the Googletest based unittests for now, they are inconsistently failing on CI, e.g. sometimes timing out after 60 seconds for tests that usually take only a fraction of a second. tst_perfdata from the perfparser submodule also fails. Temporarily disable it explicitly, until the EXCLUDE_FROM_PRECHECK change can be merged there. Change-Id: I7e9d7aded75bbe8800f82f7aa125c181271a2a1f Reviewed-by: Qt CI Bot <qt_ci_bot@qt-project.org> Reviewed-by: Jarek Kobus <jaroslaw.kobus@qt.io> Reviewed-by: <github-actions-qt-creator@cristianadam.eu>
* tst_QtcProcess::crashAfterOneSecond(): Increase the timeoutJarek Kobus2022-07-051-2/+2
| | | | | | | | | | | | Increase the timeout to default 30 seconds when waiting for finished. Apparently 2 seconds were sometimes too short. We still ensure that waitForFinished didn't timeout but finished eariler. Change-Id: I12ca647b45537174c7c66d271e7c8a477184f2ff Reviewed-by: Eike Ziller <eike.ziller@qt.io> Reviewed-by: <github-actions-qt-creator@cristianadam.eu> Reviewed-by: Qt CI Bot <qt_ci_bot@qt-project.org>
* Merge "Merge remote-tracking branch 'origin/7.0' into 8.0" into 8.0The Qt Project2022-07-012-0/+97
|\
| * Merge remote-tracking branch 'origin/7.0' into 8.0Tim Jenssen2022-07-012-0/+97
| |\ | | | | | | | | | Change-Id: I7cd5d3808007ef739212f4347ba9b16e7b298943
| | * QmlDesigner: Only remove node with positionThomas Hartmann2022-06-241-2/+25
| | | | | | | | | | | | | | | | | | | | | | | | Extending the test case. Change-Id: I37255de763262e069c2f9d30b1ce584a0347fbcf Reviewed-by: <github-actions-qt-creator@cristianadam.eu> Reviewed-by: Thomas Hartmann <thomas.hartmann@qt.io>
| | * QmlDesigner: Support reparenting to new nodes in transactionsThomas Hartmann2022-06-222-0/+74
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When we reparent to a new node that was created during the same transaction, then this node has no position. In this case we have to delete the reparented nodes and they will be created as part of the creation of the new node, since they are children. Change-Id: Icd1d02f29f529fc0f00809f7ecebf3eabfdc9a5c Reviewed-by: Miikka Heikkinen <miikka.heikkinen@qt.io>
* | | Fix pluginspec test for Windows release buildsEike Ziller2022-07-011-1/+1
|/ / | | | | | | | | | | | | Used a wrong name for the plugin library Change-Id: Ic1cff07387b660a641c2f43bfb913bd8b8668c5c Reviewed-by: Christian Stenger <christian.stenger@qt.io>
* | qmldesigner tests: remove unnecessary definesTim Jenssen2022-07-011-1/+0
| | | | | | | | | | Change-Id: I1363e7e2ac174b51236644dd37fe184e711e96d3 Reviewed-by: Thomas Hartmann <thomas.hartmann@qt.io>
* | tst_qml_testcore: fix spellingTim Jenssen2022-07-011-1/+1
| | | | | | | | | | Change-Id: I76345be2a8837c4371abf2f4a4695d1edebf2770 Reviewed-by: Thomas Hartmann <thomas.hartmann@qt.io>
* | Squish: Adapt to changed UI textChristian Stenger2022-07-011-1/+1
| | | | | | | | | | Change-Id: I02d178646ef04893f8d839b2a30085798f1c5fa7 Reviewed-by: hjk <hjk@qt.io>
* | Squish: Skip a testChristian Stenger2022-06-301-0/+4
| | | | | | | | | | | | | | | | Wizard nowadays expects a Qt version we do not provide by default. Change-Id: If0f2b3aa5755a549bd63fc258fd78b1a40337a8a Reviewed-by: Robert Löhning <robert.loehning@qt.io>
* | Squish: Adapt to changed uiChristian Stenger2022-06-302-3/+8
| | | | | | | | | | | | | | ...and modified minimum requirements inside wizards. Change-Id: I32d2dea0dc0ccff273ebf32f5aa713829330eb01 Reviewed-by: Robert Löhning <robert.loehning@qt.io>
* | Squish: Do not expect clang shipped by QC anymoreChristian Stenger2022-06-301-40/+0
| | | | | | | | | | Change-Id: I2f09835bde52aa2f0cd90def7f6d8d628736ff56 Reviewed-by: Robert Löhning <robert.loehning@qt.io>
* | deviceshell: fix hang on multithreaded accessMarcus Tillmanns2022-06-291-0/+58
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When outputting a commands stdout / stderr / exitcode, the shell was free to interlace different outputs. This could trip the parsing of the output, leading to the shell missing the exit code of a command. When that happened, the caller might wait indefinitely for a result that would never come. To workaround, the shell script now writes each command output to a temporary file first, which is then written to stdout by a single subshell, that way guaranteeing that the output of multiple commands will not interleave Change-Id: I9d8e7e788f5922c612ff533e5ba063f61a22aa8c Reviewed-by: hjk <hjk@qt.io>
* | deviceshell: Disable tests on old macos versionsMarcus Tillmanns2022-06-281-1/+12
| | | | | | | | | | | | | | | | | | The bundled "base64" tool of macos < 10.15 did not support the "-d" option ( only -D ). We disable the tests running on the local shell to keep them from failing. Change-Id: I6af353869033ed15cefedc03e2562621d1ea9904 Reviewed-by: Eike Ziller <eike.ziller@qt.io>
* | Squish: Update location of Options dialogRobert Löhning2022-06-289-16/+16
| | | | | | | | | | Change-Id: Ie9df4f897621520744998c3dbd9a3c44600ba844 Reviewed-by: Christian Stenger <christian.stenger@qt.io>
* | Adapt to move of options to Edit > PreferencesEike Ziller2022-06-283-3/+3
| | | | | | | | | | Change-Id: I6881f658db9eec5059b434bce12093667535d5ac Reviewed-by: Leena Miettinen <riitta-leena.miettinen@qt.io>
* | Tests: Disable SSE dumper test on arm macOSChristian Stenger2022-06-271-0/+19
| | | | | | | | | | | | Change-Id: Idc6f769a7967fa6afb27e419238a10d631bca0fd Reviewed-by: <github-actions-qt-creator@cristianadam.eu> Reviewed-by: hjk <hjk@qt.io>
* | CPlusPlus: Inline some simple central functionshjk2022-06-241-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | Depending on context, callgrind sees contributions of >8% to the total cost of project parsing for these functions. The functional are actualy executed executed out-of-line, often for a function body of one "payload" instruction only. Inlining removes the call/endbr64/ret overhead. Change-Id: I6886f08e322fcaa4e0f54d424279e0a8c24e4718 Reviewed-by: <github-actions-qt-creator@cristianadam.eu> Reviewed-by: Christian Kandeler <christian.kandeler@qt.io>
* | Unit test: Remove libclang test remainsChristian Kandeler2022-06-2468-2355/+12
| | | | | | | | | | Change-Id: Ifa2cd2727ecf80d3b29dc2bc2a10af11baa2f1dc Reviewed-by: Christian Stenger <christian.stenger@qt.io>
* | ClangTools: Move over a test from unittestChristian Kandeler2022-06-2415-561/+0
| | | | | | | | | | Change-Id: Id61a0bdb8e46ead4de8ad0298e62afb7b9fbcfb4 Reviewed-by: Christian Stenger <christian.stenger@qt.io>
* | ClangCodeModel: Move a test over from unittestChristian Kandeler2022-06-233-212/+0
| | | | | | | | | | | | Change-Id: Iac295216fff274fbe4a109477b1a4c2bd2c98d5d Reviewed-by: <github-actions-qt-creator@cristianadam.eu> Reviewed-by: Christian Stenger <christian.stenger@qt.io>
* | Unit test: Remove unused filesChristian Kandeler2022-06-233-97/+0
| | | | | | | | | | | | Change-Id: Ia8d8f2014f8180d5c5fa89538d5aa7fc8821bc28 Reviewed-by: <github-actions-qt-creator@cristianadam.eu> Reviewed-by: Christian Stenger <christian.stenger@qt.io>
* | tst_QtcProcess: Don't use errorOccurred() in testsJarek Kobus2022-06-233-13/+12
| | | | | | | | | | | | Change-Id: I71657e111ab09b599480874ed22e714142697c19 Reviewed-by: <github-actions-qt-creator@cristianadam.eu> Reviewed-by: David Schulz <david.schulz@qt.io>
* | tst_QtcProcess: Connect to done() signal instead of finished()Jarek Kobus2022-06-211-1/+1
| | | | | | | | | | Change-Id: Ie01349b1f9ec8189d6503df3ed0b2e1e1252d0f6 Reviewed-by: hjk <hjk@qt.io>
* | Test: Make valgrind modeldemo test compile with cmakehjk2022-06-215-32/+80
| | | | | | | | | | | | | | | | | | As on it, correct the handling of the launcher interface. This still does not work due to wrong call of valgrind-fake. Change-Id: Id835e84eee29720579ac2947906bb95f84377185 Reviewed-by: hjk <hjk@qt.io> Reviewed-by: Christian Stenger <christian.stenger@qt.io>
* | Debugger: Make one manual test deployable to remote linuxhjk2022-06-201-0/+7
| | | | | | | | | | Change-Id: Ia61926353e93ae394d3eb2082bb92769967cd958 Reviewed-by: Eike Ziller <eike.ziller@qt.io>
* | qbs build: Fix building with MSVC and Qt >= 6.3Christian Kandeler2022-06-177-0/+7
| | | | | | | | | | Change-Id: I59439217575bdcc170f38382129fa5f13edb23e2 Reviewed-by: Christian Stenger <christian.stenger@qt.io>
* | Tests: Provide valgrind testapps in cmake buildChristian Stenger2022-06-1716-1/+59
| | | | | | | | | | | | | | | | Change-Id: I4ec94504e3d2265d066434e118ab2eae6279a42f Reviewed-by: hjk <hjk@qt.io> Reviewed-by: <github-actions-qt-creator@cristianadam.eu> Reviewed-by: Qt CI Bot <qt_ci_bot@qt-project.org> Reviewed-by: Eike Ziller <eike.ziller@qt.io>
* | Increase ctest timeout for tst_qtcprocessEike Ziller2022-06-141-0/+1
| | | | | | | | | | | | | | | | Default is 5 seconds, which is by far not enough. Change-Id: I4a5851de3ec2268bd68b6a386efa8dfa181e0af3 Reviewed-by: <github-actions-qt-creator@cristianadam.eu> Reviewed-by: Jarek Kobus <jaroslaw.kobus@qt.io>
* | qbs build: Require Qt 6.2 for QmlDesignerChristian Kandeler2022-06-131-2/+3
| | | | | | | | | | | | | | As in the cmake build. Change-Id: Iedd7032093682e7cf6f423909724201f6c5840d0 Reviewed-by: Christian Stenger <christian.stenger@qt.io>
* | Use QTEST_GUILESS_MAIN where applicableEike Ziller2022-06-1347-50/+47
| | | | | | | | | | | | | | | | | | | | instead of QTEST_MAIN. Reduces the initialization that is done by the Qt test applications, and can also reduce interference with normal OS operations like the current window loosing focus. Change-Id: If88f289281aa1c8703ac7d4dbe0799d067c16588 Reviewed-by: Christian Stenger <christian.stenger@qt.io> Reviewed-by: Qt CI Bot <qt_ci_bot@qt-project.org>
* | DeviceShell: Don't use 'command' to test for base64Marcus Tillmanns2022-06-092-2/+2
| | | | | | | | | | | | | | | | On (debian based) linux the "command -v base64" did fail to find the base64 exectuable. Instead we now use "which". Change-Id: I9b57527d93952b08bcb9abc7484a7e6891aa664c Reviewed-by: Christian Stenger <christian.stenger@qt.io>
* | QtcProcess: Add a test for various quit methodsJarek Kobus2022-06-083-9/+96
| | | | | | | | | | | | | | | | | | | | The quitBlockingProcess() test examines different types of process quitting and illustrates differences in behavior. Change-Id: I9209f00576e03eef66fbdf5665351138ed437ac9 Reviewed-by: hjk <hjk@qt.io> Reviewed-by: Qt CI Bot <qt_ci_bot@qt-project.org>
* | DeviceShell: Fix compile issuesEike Ziller2022-06-072-2/+2
| | | | | | | | | | | | | | | | | | | | | | - fix warning deviceshell.cpp:222:59: warning: suggest braces around initialization of subobject - add missing dependency to app_version to tests Change-Id: I6d61f26cbc6f51b28175257ab7d0f4d2b491fecb Reviewed-by: Marcus Tillmanns <marcus.tillmanns@qt.io> Reviewed-by: Jarek Kobus <jaroslaw.kobus@qt.io> Reviewed-by: <github-actions-qt-creator@cristianadam.eu>
* | Merge "Merge remote-tracking branch 'origin/7.0' into 8.0" into 8.0The Qt Project2022-06-078-0/+0
|\ \
| * \ Merge remote-tracking branch 'origin/7.0' into 8.0Eike Ziller2022-06-078-0/+0
| |\ \ | | |/ | | | | | | Change-Id: Ib669e2e993c4675a7382f721ccb7d6ecff11fc20
| | * Squish: Remove exec permission where not neededRobert Löhning2022-06-038-0/+0
| | | | | | | | | | | | | | | | | | | | | These python scripts cannot be executed directly anyway. Change-Id: I9ac3170450c4f8be4063b02ee8ec9df28f90cec5 Reviewed-by: Christian Stenger <christian.stenger@qt.io>
* | | device: Use multiplex script to allow multithread supportMarcus Tillmanns2022-06-0710-0/+568
|/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously the runInShell and outputForRunInShell methods were exclusively processed single threaded, meaning all calls were processed sequentially. With the multiplexed helper script we can now run multiple processes simultaneously. ( see tst_manual_deviceshell ) Additionally the new script allows us to capture both stdout and stderr from commands which was not possible previously. Change-Id: I52f4fb46d872dc274edb9c11872d2f6543741b34 Reviewed-by: David Schulz <david.schulz@qt.io> Reviewed-by: Jarek Kobus <jaroslaw.kobus@qt.io> Reviewed-by: <github-actions-qt-creator@cristianadam.eu>
* | ClangCodeModel: Switch to LSP-based UI header approachChristian Kandeler2022-06-022-3/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Generating ui headers in a well-known path and then including that one in the compilation database does not work in the presence of multiple ui files with the same name. As it turns out, we don't have to generate any files at all; instead, we pass the file contents directly to clangd, which then uses them when parsing includes of the respective header. User-visible behavior change apart from the abovementioned bug fix: Tooltips and "follow symbol" on the include directive now always use the actual location of the header provided by the build system. Fixes: QTCREATORBUG-27584 Change-Id: I6b13e12cb3a365199567b0bc824d12b373117697 Reviewed-by: <github-actions-qt-creator@cristianadam.eu> Reviewed-by: David Schulz <david.schulz@qt.io>
* | Valgrind: Inline CallGrindController into CallgrindToolRunnerhjk2022-05-311-1/+0
| | | | | | | | | | | | | | No functional changes. Change-Id: I3b73da69e20e4d0324c582544c67a84eac8dbdab Reviewed-by: Jarek Kobus <jaroslaw.kobus@qt.io>
* | Utils: More forward declarations / drop unused headers [T-Z]Jarek Kobus2022-05-251-0/+1
| | | | | | | | | | | | | | | | | | | | | | Round 1 - focus on headers. For classes with initial in range [T-Z]. Replace QT_FORWARD_DECLARE_CLASS with QT_BEGIN_NAMESPACE and QT_END_NAMESPACE and forward declare inside. Change-Id: I4ac3a8391e6167aa2db3973a9f94a45ac3c8ebd1 Reviewed-by: hjk <hjk@qt.io>