[PR #52] [MERGED] Qml Unit tests improvements #188

New issue

Closed

opened 2026-05-05 11:20:28 -06:00 by gitea-mirror · 0 comments

gitea-mirror commented

2026-05-05 11:20:28 -06:00

Owner

📋 Pull Request Information

Original PR: https://github.com/qmlnet/qmlnet/pull/52
Author: @devmil
Created: 8/6/2018
Status: ✅ Merged
Merged: 8/7/2018
Merged by: @pauldotknopf

Base: develop ← Head: improvements/mvvmbehavior_unit_tests

📝 Commits (10+)

544f009 MvvmBehavior Unit Tests check if they break the manual signal support
52580e7 MvvmBehavior: TestResult is null in the beginning
ac41d23 MvvmBehavior: unit test for checking not breaking custon notify signals
a7eb717 Qml unit tests: method for executing onCompleted code
fa2ae86 migrating UInt tests to RunQmlTest method
59b0ace Qml tests: RunQmlTest doesn't need the generic parameter any more
9a1c8ce Qml tests: converted StringTests
fe1d19e Qml tests: converted SignalTests
7d9d86d Unit tests: run sequentially
f14344f Qml tests: converted ObjectTests

📊 Changes

15 files changed (+423 additions, -719 deletions)

View changed files

➕ src/net/Qml.Net.Tests/Properties.cs (+3 -0)
📝 src/net/Qml.Net.Tests/Qml/AwaitTests.cs (+24 -42)
📝 src/net/Qml.Net.Tests/Qml/BaseQmlTests.cs (+24 -16)
📝 src/net/Qml.Net.Tests/Qml/BoolTests.cs (+18 -54)
📝 src/net/Qml.Net.Tests/Qml/CharTests.cs (+21 -63)
📝 src/net/Qml.Net.Tests/Qml/DateTimeOffsetTests.cs (+12 -30)
📝 src/net/Qml.Net.Tests/Qml/DateTimeTests.cs (+12 -30)
📝 src/net/Qml.Net.Tests/Qml/DoubleTests.cs (+12 -36)
📝 src/net/Qml.Net.Tests/Qml/IntTests.cs (+12 -36)
📝 src/net/Qml.Net.Tests/Qml/JsValueTests.cs (+72 -126)
📝 src/net/Qml.Net.Tests/Qml/MvvmInteropBehaviorTests.cs (+106 -46)
📝 src/net/Qml.Net.Tests/Qml/ObjectTests.cs (+14 -38)
📝 src/net/Qml.Net.Tests/Qml/SignalTests.cs (+65 -125)
📝 src/net/Qml.Net.Tests/Qml/StringTests.cs (+12 -36)
📝 src/net/Qml.Net.Tests/Qml/UIntTests.cs (+16 -41)

📄 Description

MvvmBehavior Tests expanded
Utility method for running Qml unit tests reducing the ceremony for each unit test

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/qmlnet/qmlnet/pull/52 **Author:** [@devmil](https://github.com/devmil) **Created:** 8/6/2018 **Status:** ✅ Merged **Merged:** 8/7/2018 **Merged by:** [@pauldotknopf](https://github.com/pauldotknopf) **Base:** `develop` ← **Head:** `improvements/mvvmbehavior_unit_tests` --- ### 📝 Commits (10+) - [`544f009`](https://github.com/qmlnet/qmlnet/commit/544f0096a9cf3c138957c2747adc49fb6128284a) MvvmBehavior Unit Tests check if they break the manual signal support - [`52580e7`](https://github.com/qmlnet/qmlnet/commit/52580e77b81feda7f2741b3b0c76e203db26f61d) MvvmBehavior: TestResult is null in the beginning - [`ac41d23`](https://github.com/qmlnet/qmlnet/commit/ac41d23dd1d9e4054bec3566c34bcbf95b80e079) MvvmBehavior: unit test for checking not breaking custon notify signals - [`a7eb717`](https://github.com/qmlnet/qmlnet/commit/a7eb717f706f1a1f508cec3d8341de651d2eddf2) Qml unit tests: method for executing onCompleted code - [`fa2ae86`](https://github.com/qmlnet/qmlnet/commit/fa2ae860cf81dec01123cb1b6bb75e2dd629a783) migrating UInt tests to RunQmlTest method - [`59b0ace`](https://github.com/qmlnet/qmlnet/commit/59b0ace2049d181aabb23f57cecfa8870ce4c305) Qml tests: RunQmlTest doesn't need the generic parameter any more - [`9a1c8ce`](https://github.com/qmlnet/qmlnet/commit/9a1c8ce11d63c741c402c010ec13bf27cdc59336) Qml tests: converted StringTests - [`fe1d19e`](https://github.com/qmlnet/qmlnet/commit/fe1d19e729db787c9f19b499f2cebfb1b84de216) Qml tests: converted SignalTests - [`7d9d86d`](https://github.com/qmlnet/qmlnet/commit/7d9d86d084e6e67763ceb072a7b2d98eaacdd554) Unit tests: run sequentially - [`f14344f`](https://github.com/qmlnet/qmlnet/commit/f14344f35be8a16be7dce9f93ba3953e8b7dd8da) Qml tests: converted ObjectTests ### 📊 Changes **15 files changed** (+423 additions, -719 deletions) <details> <summary>View changed files</summary> ➕ `src/net/Qml.Net.Tests/Properties.cs` (+3 -0) 📝 `src/net/Qml.Net.Tests/Qml/AwaitTests.cs` (+24 -42) 📝 `src/net/Qml.Net.Tests/Qml/BaseQmlTests.cs` (+24 -16) 📝 `src/net/Qml.Net.Tests/Qml/BoolTests.cs` (+18 -54) 📝 `src/net/Qml.Net.Tests/Qml/CharTests.cs` (+21 -63) 📝 `src/net/Qml.Net.Tests/Qml/DateTimeOffsetTests.cs` (+12 -30) 📝 `src/net/Qml.Net.Tests/Qml/DateTimeTests.cs` (+12 -30) 📝 `src/net/Qml.Net.Tests/Qml/DoubleTests.cs` (+12 -36) 📝 `src/net/Qml.Net.Tests/Qml/IntTests.cs` (+12 -36) 📝 `src/net/Qml.Net.Tests/Qml/JsValueTests.cs` (+72 -126) 📝 `src/net/Qml.Net.Tests/Qml/MvvmInteropBehaviorTests.cs` (+106 -46) 📝 `src/net/Qml.Net.Tests/Qml/ObjectTests.cs` (+14 -38) 📝 `src/net/Qml.Net.Tests/Qml/SignalTests.cs` (+65 -125) 📝 `src/net/Qml.Net.Tests/Qml/StringTests.cs` (+12 -36) 📝 `src/net/Qml.Net.Tests/Qml/UIntTests.cs` (+16 -41) </details> ### 📄 Description MvvmBehavior Tests expanded Utility method for running Qml unit tests reducing the ceremony for each unit test --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>