1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
|
/*
* qtmaildir - a Qt6 mail client for notmuch-indexed Maildirs
* Copyright (C) 2026 Danilo M. <danix@danix.xyz>
*
* This program is free software; you can redistribute it and/or modify
* it under the terms of the GNU General Public License version 2 as
* published by the Free Software Foundation.
*
* This program is distributed in the hope that it will be useful,
* but WITHOUT ANY WARRANTY; without even the implied warranty of
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
* GNU General Public License for more details.
*
* You should have received a copy of the GNU General Public License
* along with this program; if not, write to the Free Software
* Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
*/
#include "carddelegate.h"
#include "cardlayout.h"
#include "tagchip.h"
#include "tagcolors.h"
#include "threadlistmodel.h"
#include <QTest>
class TestCardDelegate : public QObject
{
Q_OBJECT
private slots:
void theAccentLiftsAMutedAccountColour();
void theAccentKeepsEachAccountTellableApart();
void anAccountWithNoColourFallsBackToTheNeutralLine();
void theFadeEndsAtSixtyPercentOfTheCard();
void aReplyFadeStartsAtItsOwnSpine();
void theDelegateAsksForAScaledSquircle();
void aRowWithNoSenderFallsBackToTheAccount();
};
namespace {
/// The user's five configured account colours, as a realistic sample. They are
/// all mid-tone because they were chosen as CHIP fills, which is the whole
/// reason the bar has to lift them.
const QList<QColor> &sampleAccounts()
{
static const QList<QColor> colours{
QColor("#2f6fa8"), QColor("#3d7a4a"), QColor("#a83f2f"),
QColor("#8a5cb8"), QColor("#b8862f"),
};
return colours;
}
float saturationOf(const QColor &c)
{
float h = 0, s = 0, l = 0, a = 0;
c.getHslF(&h, &s, &l, &a);
return s;
}
float lightnessOf(const QColor &c)
{
float h = 0, s = 0, l = 0, a = 0;
c.getHslF(&h, &s, &l, &a);
return l;
}
float hueOf(const QColor &c)
{
float h = 0, s = 0, l = 0, a = 0;
c.getHslF(&h, &s, &l, &a);
return h;
}
} // namespace
void TestCardDelegate::theAccentLiftsAMutedAccountColour()
{
// An account colour is chosen to be a chip's fill with legible text on top,
// so it is mid-tone by construction. Three pixels of a mid-tone colour
// beside a card's own background barely register, which is what the user
// reported: "the colours could be a little more vivid".
//
// The green is the weakest of the five (S 0.33, L 0.36) and is the one that
// has to move most.
const QColor muted("#3d7a4a");
const QColor accent = CardDelegate::accentLineColour(muted);
QVERIFY2(saturationOf(accent) > saturationOf(muted),
"the accent is no more saturated than the chip colour it comes "
"from, so a muted account stays muted as a 3px bar");
QVERIFY2(lightnessOf(accent) > lightnessOf(muted),
"the accent is no lighter than the chip colour, so it cannot "
"carry on a dark theme");
// The floor, stated as the numbers that were chosen by rendering all five
// against both themes. Higher pushed the green toward a neon that no
// longer matched its own chip.
QVERIFY(saturationOf(accent) >= 0.65f - 0.01f);
QVERIFY(lightnessOf(accent) >= 0.50f - 0.01f);
// A colour already past the floor is left alone: the lift is a floor, not
// a repaint, or a user who picked a vivid colour would have it changed.
const QColor alreadyVivid = QColor::fromHslF(0.6f, 0.9f, 0.6f);
const QColor untouched = CardDelegate::accentLineColour(alreadyVivid);
QCOMPARE(saturationOf(untouched), saturationOf(alreadyVivid));
QCOMPARE(lightnessOf(untouched), lightnessOf(alreadyVivid));
}
void TestCardDelegate::theAccentKeepsEachAccountTellableApart()
{
// The bar's whole job is saying WHICH account, so the lift must not
// converge two hues. Asserted across the real five rather than one pair:
// a floor applied to saturation and lightness leaves hue untouched, and
// this is what proves it stayed that way.
for (const QColor &configured : sampleAccounts()) {
const QColor accent = CardDelegate::accentLineColour(configured);
QVERIFY2(qAbs(hueOf(accent) - hueOf(configured)) < 0.01f,
qPrintable(QStringLiteral("account %1 changed hue to %2, so "
"it no longer matches its own chip")
.arg(configured.name(), accent.name())));
}
// And no two of them collapse onto each other.
QSet<QRgb> seen;
for (const QColor &configured : sampleAccounts())
seen.insert(CardDelegate::accentLineColour(configured).rgb());
QCOMPARE(seen.size(), sampleAccounts().size());
}
void TestCardDelegate::anAccountWithNoColourFallsBackToTheNeutralLine()
{
// A thread with no account tag has no colour to lift, and must not end up
// with a saturated bar invented out of an invalid QColor.
QCOMPARE(CardDelegate::accentLineColour(QColor()),
ThreadListModel::threadLineColour());
}
void TestCardDelegate::theFadeEndsAtSixtyPercentOfTheCard()
{
const QRect card(0, 0, 500, 60);
const QRect root = CardDelegate::fadeRectFor(card, QRect());
// Anchored at the card's RIGHT edge: the hard stop belongs where the card
// ends, not 60% across it, which read as a slab.
QCOMPARE(root.right(), card.right());
QCOMPARE(root.width(), 300);
}
void TestCardDelegate::aReplyFadeStartsAtItsOwnSpine()
{
const QRect card(0, 0, 500, 60);
// The innermost spine of a nested reply, which is its own coloured border.
const QRect spine(80, 0, 2, 60);
// A spine deep enough to cut into the wash, which starts at 40% here.
const QRect deep(300, 0, 2, 60);
const QRect reply = CardDelegate::fadeRectFor(card, deep);
// Clamped at the spine, so the wash never runs under a reply's own border.
QCOMPARE(reply.left(), deep.left());
// Still anchored at the card's right edge, so a deeper reply's wash is
// shorter rather than displaced.
QCOMPARE(reply.right(), CardDelegate::fadeRectFor(card, QRect()).right());
QVERIFY(reply.width() < CardDelegate::fadeRectFor(card, QRect()).width());
// A shallow spine sits left of where the wash begins and changes nothing.
const QRect shallow(80, 0, 2, 60);
QCOMPARE(CardDelegate::fadeRectFor(card, shallow),
CardDelegate::fadeRectFor(card, QRect()));
}
void TestCardDelegate::theDelegateAsksForAScaledSquircle()
{
// Asserted through the function the PRODUCTION path calls, not through
// Avatar::pixmapFor() directly: a test pointed at the function being
// called into proves what that function does and nothing about whether the
// delegate asks it for the right thing. CLAUDE.md records a mutation that
// survived exactly that mistake.
const QRect card(0, 0, 500, 60);
const QFont font;
const CardLayout layout =
CardLayout::compute(CardLayout::Input(), card, font);
const QPixmap pixmap = CardDelegate::avatarFor(
QStringLiteral("john@example.org"), QStringLiteral("John Doe"),
QStringLiteral("me@example.org"), QStringLiteral("Work"), false,
layout.avatarRect.width(), font);
QCOMPARE(pixmap.size(),
QSize(layout.avatarRect.width(), layout.avatarRect.width()));
}
void TestCardDelegate::aRowWithNoSenderFallsBackToTheAccount()
{
const QFont font;
// No sender address at all: the squircle is still drawn, seeded from the
// account, so a card never shows a hole.
const QPixmap fallback = CardDelegate::avatarFor(
QString(), QString(), QStringLiteral("me@example.org"),
QStringLiteral("Work"), false, 44, font);
QVERIFY(!fallback.isNull());
// And it is the ACCOUNT's identity, not an arbitrary one: seeding from the
// same account twice agrees.
const QPixmap again = CardDelegate::avatarFor(
QString(), QString(), QStringLiteral("me@example.org"),
QStringLiteral("Work"), false, 44, font);
QCOMPARE(fallback.toImage(), again.toImage());
}
QTEST_MAIN(TestCardDelegate)
#include "test_carddelegate.moc"
|
