Skip to content

Unit tests for QML Cards, Workflow, System, and Visualization components - #382

Merged
suraj-yadav0 merged 2 commits into
CITOpenRep:release/v1.3.4from
parvathyabnair:test-scripts
Oct 22, 2026
Merged

suraj-yadav0 merged 2 commits into
CITOpenRep:release/v1.3.4from
parvathyabnair:test-scripts

Conversation

@parvathyabnair

Copy link
Copy Markdown
Contributor

unit tests in tests/qml/ for cards, workflow handlers, system widgets, and charts:

ProjectDetailsCard.qml (project info, status badge, progress)
ActivityDetailsCard.qml (activity details, assignee, due date)
UpdatesDetailsCard.qml (update summary and timestamp)
FormDraftHandler.qml (draft autosave and change tracking)
AttachmentManager.qml (attachment list and file deletion signal)
VoiceTimerWidget.qml (voice recording states and timer controls)
ModelDownloadTimerWidget.qml (download progress and cancel button)
ImagePreviewer.qml (image zoom, pan, close action)
BackendBridge.qml (signal dispatch between QML and Python)
EHowerIndicator.qml (dot coordinate placement in Eisenhower matrix)
EHower.qml (matrix 2x2 grid layout)
ProjectPieChart.qml (pie slice angles and empty state)
ProjectList.qml (list model population and selection)

Comment thread tests/qml/tst_activitydetailscard.qml
Comment thread tests/qml/QtCharts/PieSeries.qml Outdated
angleSpan: 45
});

slices.push(sliceObj);

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In QML JavaScript engine (V4), array in-place mutations do not emit slicesChanged notifications. As
a result, property bindings depending on slices.length (such as readonly property int count:
slices.length) do not trigger reactivity on append.

Suggested Fix: Assign a new array or reassign slices after appending:
slices = slices.concat([sliceObj]);

Comment thread tests/qml/tst_projectpiechart.qml
Comment thread tests/qml/README.md

@suraj-yadav0 suraj-yadav0 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM ...!!

@suraj-yadav0
suraj-yadav0 merged commit c7d13d9 into CITOpenRep:release/v1.3.4 Oct 22, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants