This is done by switching from the QStandardItemModel, which requires
QtGui, to QStringListModel, which is in QtCore. The result is a much
leaner test, and it's all done by adding a couple of helper functions
which add the simplistic data addition functions from QSIM, except
locked to QStrings (the way it was being used anyway).
Details
Details
- Reviewers
bshah apol - Group Reviewers
Discover Software Store - Commits
- R134:120ee5cbb571: Don't require QtGui for the PaginateModel test
Run the tests, which still pass
Diff Detail
Diff Detail
- Repository
- R134 Discover Software Store
- Branch
- dont-require-qtgui-for-paginatemodel-test (branched from master)
- Lint
No Linters Available - Unit
No Unit Test Coverage - Build Status
Buildable 16907 Build 16925: arc lint + arc unit
Comment Actions
It allows the tests to be run on e.g. postmarketOS' CI - https://gitlab.com/postmarketOS/pmaports/merge_requests/639#note_220402081 shows the specific problem :) (there's other issues, as seen in the comments further down from there, but yeah, it's really just a "why require gui when we don't need it" sort of situation... especially when the fix is as simple as this :) )