presentation/ComposeDraftStore.cpp
| Line | Branch | Exec | Source |
|---|---|---|---|
| 1 | #include "ComposeDraftStore.h" | ||
| 2 | #include "util/SafePathComponent.h" | ||
| 3 | #include <QCryptographicHash> | ||
| 4 | #include <QDir> | ||
| 5 | #include <QFile> | ||
| 6 | #include <QFileInfo> | ||
| 7 | #include <QJsonDocument> | ||
| 8 | #include <QRegularExpression> | ||
| 9 | #include <QSaveFile> | ||
| 10 | #include <utility> | ||
| 11 | |||
| 12 | 1597 | ComposeDraftStore::ComposeDraftStore(QString profileDirectory) | |
| 13 |
3/3✓ Branch 2 → 3 taken 1597 times.
✓ Branch 3 → 4 taken 1597 times.
✓ Branch 4 → 5 taken 1597 times.
|
1597 | : m_directory(QDir(profileDirectory).filePath("drafts")) {} |
| 14 | 1644 | QString ComposeDraftStore::path(const QString &accountId) const { | |
| 15 |
5/5✓ Branch 3 → 4 taken 799 times.
✓ Branch 3 → 11 taken 845 times.
✓ Branch 4 → 5 taken 799 times.
✓ Branch 5 → 6 taken 799 times.
✓ Branch 6 → 7 taken 799 times.
|
1644 | if (accountId.isEmpty()) return QDir(m_directory).filePath("compose-unconfigured.json"); |
| 16 |
2/2✓ Branch 11 → 12 taken 845 times.
✓ Branch 16 → 17 taken 845 times.
|
3380 | return QDir(m_directory).filePath("compose-" + QString::fromLatin1( |
| 17 |
6/6✓ Branch 12 → 13 taken 845 times.
✓ Branch 14 → 15 taken 845 times.
✓ Branch 15 → 16 taken 845 times.
✓ Branch 17 → 18 taken 845 times.
✓ Branch 18 → 19 taken 845 times.
✓ Branch 19 → 20 taken 845 times.
|
2535 | QCryptographicHash::hash(accountId.toUtf8(), QCryptographicHash::Sha256).toHex()) + ".json"); |
| 18 | } | ||
| 19 | 13 | QJsonObject ComposeDraftStore::load(const QString &accountId, const QString &accountName, | |
| 20 | bool ownsUnscopedLegacy) const { | ||
| 21 |
1/1✓ Branch 2 → 3 taken 13 times.
|
13 | const auto scoped = path(accountId); |
| 22 |
8/11✓ Branch 3 → 4 taken 13 times.
✓ Branch 4 → 5 taken 5 times.
✓ Branch 4 → 8 taken 8 times.
✓ Branch 5 → 6 taken 5 times.
✓ Branch 6 → 7 taken 5 times.
✗ Branch 7 → 8 not taken.
✓ Branch 7 → 9 taken 5 times.
✓ Branch 10 → 11 taken 5 times.
✓ Branch 10 → 12 taken 8 times.
✗ Branch 139 → 140 not taken.
✗ Branch 139 → 141 not taken.
|
13 | const bool hasScoped = QFileInfo::exists(scoped) || QFileInfo(scoped).isSymLink(); |
| 23 |
12/16✓ Branch 12 → 13 taken 5 times.
✓ Branch 12 → 19 taken 8 times.
✓ Branch 13 → 14 taken 2 times.
✓ Branch 13 → 19 taken 3 times.
✓ Branch 15 → 16 taken 2 times.
✓ Branch 16 → 17 taken 2 times.
✓ Branch 17 → 18 taken 1 time.
✓ Branch 17 → 19 taken 1 time.
✓ Branch 20 → 21 taken 2 times.
✓ Branch 20 → 22 taken 11 times.
✓ Branch 22 → 23 taken 2 times.
✓ Branch 22 → 24 taken 11 times.
✗ Branch 143 → 144 not taken.
✗ Branch 143 → 145 not taken.
✗ Branch 147 → 148 not taken.
✗ Branch 147 → 149 not taken.
|
13 | const bool useUnconfigured = !hasScoped && ownsUnscopedLegacy && QFileInfo::exists(path({})); |
| 24 |
15/21✓ Branch 24 → 25 taken 8 times.
✓ Branch 24 → 26 taken 5 times.
✓ Branch 26 → 27 taken 1 time.
✓ Branch 26 → 29 taken 4 times.
✓ Branch 28 → 32 taken 1 time.
✓ Branch 29 → 30 taken 4 times.
✓ Branch 30 → 31 taken 4 times.
✓ Branch 31 → 32 taken 4 times.
✓ Branch 32 → 33 taken 13 times.
✓ Branch 34 → 35 taken 4 times.
✓ Branch 34 → 36 taken 9 times.
✓ Branch 36 → 37 taken 4 times.
✓ Branch 36 → 38 taken 9 times.
✓ Branch 38 → 39 taken 1 time.
✓ Branch 38 → 40 taken 12 times.
✗ Branch 153 → 154 not taken.
✗ Branch 153 → 155 not taken.
✗ Branch 157 → 158 not taken.
✗ Branch 157 → 159 not taken.
✗ Branch 161 → 162 not taken.
✗ Branch 161 → 163 not taken.
|
13 | QFile file(hasScoped ? scoped : useUnconfigured ? path({}) : QDir(m_directory).filePath("compose-draft.json")); |
| 25 |
12/17✓ Branch 40 → 41 taken 13 times.
✓ Branch 41 → 42 taken 13 times.
✓ Branch 42 → 43 taken 13 times.
✗ Branch 42 → 48 not taken.
✓ Branch 43 → 44 taken 13 times.
✓ Branch 44 → 45 taken 13 times.
✗ Branch 44 → 48 not taken.
✓ Branch 46 → 47 taken 13 times.
✓ Branch 47 → 48 taken 2 times.
✓ Branch 47 → 49 taken 11 times.
✓ Branch 50 → 51 taken 13 times.
✗ Branch 50 → 52 not taken.
✓ Branch 52 → 53 taken 2 times.
✓ Branch 52 → 55 taken 11 times.
✓ Branch 53 → 54 taken 2 times.
✗ Branch 167 → 168 not taken.
✗ Branch 167 → 169 not taken.
|
13 | if (QFileInfo(file).isSymLink() || file.size() > MaximumBytes || !file.open(QIODevice::ReadOnly)) return {}; |
| 26 |
2/2✓ Branch 55 → 56 taken 11 times.
✓ Branch 56 → 57 taken 11 times.
|
11 | const auto document = QJsonDocument::fromJson(file.read(MaximumBytes + 1)); |
| 27 |
2/4✓ Branch 58 → 59 taken 11 times.
✗ Branch 59 → 60 not taken.
✓ Branch 59 → 62 taken 11 times.
✗ Branch 60 → 61 not taken.
|
11 | if (!document.isObject()) return {}; |
| 28 |
1/1✓ Branch 62 → 63 taken 11 times.
|
11 | const auto object = document.object(); |
| 29 |
2/2✓ Branch 63 → 64 taken 8 times.
✓ Branch 63 → 88 taken 3 times.
|
11 | if (hasScoped) { |
| 30 |
17/33✓ Branch 64 → 65 taken 8 times.
✓ Branch 65 → 66 taken 8 times.
✓ Branch 66 → 67 taken 8 times.
✓ Branch 68 → 69 taken 8 times.
✗ Branch 68 → 73 not taken.
✓ Branch 69 → 70 taken 8 times.
✓ Branch 70 → 71 taken 8 times.
✓ Branch 71 → 72 taken 8 times.
✓ Branch 72 → 73 taken 3 times.
✓ Branch 72 → 74 taken 5 times.
✓ Branch 75 → 76 taken 8 times.
✗ Branch 75 → 77 not taken.
✓ Branch 77 → 78 taken 8 times.
✗ Branch 77 → 79 not taken.
✓ Branch 79 → 80 taken 8 times.
✗ Branch 79 → 81 not taken.
✓ Branch 81 → 82 taken 8 times.
✗ Branch 81 → 83 not taken.
✓ Branch 83 → 84 taken 8 times.
✗ Branch 83 → 85 not taken.
✓ Branch 85 → 86 taken 3 times.
✓ Branch 85 → 132 taken 5 times.
✓ Branch 86 → 87 taken 3 times.
✗ Branch 174 → 175 not taken.
✗ Branch 174 → 176 not taken.
✗ Branch 178 → 179 not taken.
✗ Branch 178 → 180 not taken.
✗ Branch 182 → 183 not taken.
✗ Branch 182 → 184 not taken.
✗ Branch 186 → 187 not taken.
✗ Branch 186 → 188 not taken.
✗ Branch 190 → 191 not taken.
✗ Branch 190 → 192 not taken.
|
8 | if (object.value("accountConfigId").toString() != accountId || object.value("cleared").toBool()) return {}; |
| 31 |
2/2✓ Branch 88 → 89 taken 1 time.
✓ Branch 88 → 113 taken 2 times.
|
3 | } else if (useUnconfigured) { |
| 32 |
14/33✓ Branch 89 → 90 taken 1 time.
✓ Branch 90 → 91 taken 1 time.
✓ Branch 91 → 92 taken 1 time.
✓ Branch 93 → 94 taken 1 time.
✗ Branch 93 → 98 not taken.
✓ Branch 94 → 95 taken 1 time.
✓ Branch 95 → 96 taken 1 time.
✓ Branch 96 → 97 taken 1 time.
✗ Branch 97 → 98 not taken.
✓ Branch 97 → 99 taken 1 time.
✓ Branch 100 → 101 taken 1 time.
✗ Branch 100 → 102 not taken.
✓ Branch 102 → 103 taken 1 time.
✗ Branch 102 → 104 not taken.
✓ Branch 104 → 105 taken 1 time.
✗ Branch 104 → 106 not taken.
✓ Branch 106 → 107 taken 1 time.
✗ Branch 106 → 108 not taken.
✓ Branch 108 → 109 taken 1 time.
✗ Branch 108 → 110 not taken.
✗ Branch 110 → 111 not taken.
✓ Branch 110 → 132 taken 1 time.
✗ Branch 111 → 112 not taken.
✗ Branch 194 → 195 not taken.
✗ Branch 194 → 196 not taken.
✗ Branch 198 → 199 not taken.
✗ Branch 198 → 200 not taken.
✗ Branch 202 → 203 not taken.
✗ Branch 202 → 204 not taken.
✗ Branch 206 → 207 not taken.
✗ Branch 206 → 208 not taken.
✗ Branch 210 → 211 not taken.
✗ Branch 210 → 212 not taken.
|
1 | if (!object.value("accountConfigId").toString().isEmpty() || object.value("cleared").toBool()) return {}; |
| 33 |
11/21✓ Branch 113 → 114 taken 2 times.
✓ Branch 114 → 115 taken 2 times.
✓ Branch 115 → 116 taken 2 times.
✗ Branch 115 → 121 not taken.
✓ Branch 116 → 117 taken 2 times.
✓ Branch 117 → 118 taken 2 times.
✓ Branch 118 → 119 taken 2 times.
✓ Branch 122 → 123 taken 2 times.
✗ Branch 122 → 124 not taken.
✓ Branch 124 → 125 taken 2 times.
✗ Branch 124 → 126 not taken.
✓ Branch 126 → 127 taken 2 times.
✗ Branch 126 → 128 not taken.
✓ Branch 129 → 130 taken 1 time.
✓ Branch 129 → 132 taken 1 time.
✗ Branch 214 → 215 not taken.
✗ Branch 214 → 216 not taken.
✗ Branch 218 → 219 not taken.
✗ Branch 218 → 220 not taken.
✗ Branch 222 → 223 not taken.
✗ Branch 222 → 224 not taken.
|
2 | } else if (object.contains("accountId") ? object.value("accountId").toString() != accountName : !ownsUnscopedLegacy) { |
| 34 |
1/1✓ Branch 130 → 131 taken 1 time.
|
1 | return {}; |
| 35 | } | ||
| 36 | 7 | return object; | |
| 37 | 13 | } | |
| 38 | 15 | bool ComposeDraftStore::save(const QString &accountId, QJsonObject draft) const { | |
| 39 |
1/1✓ Branch 4 → 5 taken 15 times.
|
15 | return saveWithResult(accountId, std::move(draft)) == SaveResult::Saved; |
| 40 | } | ||
| 41 | |||
| 42 | 15 | ComposeDraftStore::SaveResult ComposeDraftStore::saveWithResult( | |
| 43 | const QString &accountId, QJsonObject draft) const { | ||
| 44 |
4/4✓ Branch 2 → 3 taken 15 times.
✓ Branch 3 → 4 taken 15 times.
✓ Branch 4 → 5 taken 15 times.
✓ Branch 5 → 6 taken 15 times.
|
15 | draft["accountConfigId"] = accountId; |
| 45 |
2/2✓ Branch 8 → 9 taken 15 times.
✓ Branch 9 → 10 taken 15 times.
|
15 | const auto bytes = QJsonDocument(draft).toJson(); |
| 46 |
2/2✓ Branch 12 → 13 taken 1 time.
✓ Branch 12 → 14 taken 14 times.
|
15 | if (bytes.size() > MaximumBytes) return SaveResult::TooLarge; |
| 47 |
9/20✓ Branch 14 → 15 taken 14 times.
✓ Branch 15 → 16 taken 14 times.
✓ Branch 16 → 17 taken 14 times.
✗ Branch 16 → 25 not taken.
✓ Branch 18 → 19 taken 14 times.
✓ Branch 19 → 20 taken 14 times.
✓ Branch 20 → 21 taken 14 times.
✗ Branch 20 → 25 not taken.
✓ Branch 33 → 34 taken 14 times.
✗ Branch 33 → 35 not taken.
✓ Branch 35 → 36 taken 14 times.
✗ Branch 35 → 37 not taken.
✗ Branch 37 → 38 not taken.
✓ Branch 37 → 39 taken 14 times.
✗ Branch 79 → 80 not taken.
✗ Branch 79 → 81 not taken.
✗ Branch 83 → 84 not taken.
✗ Branch 83 → 85 not taken.
✗ Branch 87 → 88 not taken.
✗ Branch 87 → 89 not taken.
|
42 | if (QFileInfo(m_directory).isSymLink() || !QDir().mkpath(m_directory) || |
| 48 |
7/15✓ Branch 21 → 22 taken 14 times.
✓ Branch 22 → 23 taken 14 times.
✓ Branch 23 → 24 taken 14 times.
✗ Branch 24 → 25 not taken.
✓ Branch 24 → 26 taken 14 times.
✓ Branch 27 → 28 taken 14 times.
✗ Branch 27 → 29 not taken.
✓ Branch 29 → 30 taken 14 times.
✗ Branch 29 → 31 not taken.
✓ Branch 31 → 32 taken 14 times.
✗ Branch 31 → 33 not taken.
✗ Branch 71 → 72 not taken.
✗ Branch 71 → 73 not taken.
✗ Branch 75 → 76 not taken.
✗ Branch 75 → 77 not taken.
|
28 | QFileInfo(path(accountId)).isSymLink()) return SaveResult::Failed; |
| 49 |
2/2✓ Branch 39 → 40 taken 14 times.
✓ Branch 40 → 41 taken 14 times.
|
14 | QSaveFile file(path(accountId)); |
| 50 |
1/1✓ Branch 42 → 43 taken 14 times.
|
14 | file.setDirectWriteFallback(false); |
| 51 |
1/1✓ Branch 44 → 45 taken 14 times.
|
14 | return file.open(QIODevice::WriteOnly) && |
| 52 |
3/5✓ Branch 47 → 48 taken 14 times.
✓ Branch 48 → 49 taken 14 times.
✗ Branch 48 → 55 not taken.
✓ Branch 51 → 52 taken 14 times.
✗ Branch 51 → 55 not taken.
|
28 | file.setPermissions(QFile::ReadOwner | QFile::WriteOwner) && |
| 53 |
3/4✓ Branch 49 → 50 taken 14 times.
✓ Branch 52 → 53 taken 14 times.
✓ Branch 53 → 54 taken 14 times.
✗ Branch 53 → 55 not taken.
|
28 | file.write(bytes) == bytes.size() && file.commit() |
| 54 |
1/2✓ Branch 45 → 46 taken 14 times.
✗ Branch 45 → 55 not taken.
|
28 | ? SaveResult::Saved : SaveResult::Failed; |
| 55 | 15 | } | |
| 56 | |||
| 57 | // ─── Q06: session slots ─── | ||
| 58 | |||
| 59 | 5 | QString ComposeDraftStore::slotPath(const QString &accountId, | |
| 60 | const QString &slotId) const { | ||
| 61 | // Slot ids are session tokens; harden the file name against traversal. | ||
| 62 |
4/7✓ Branch 2 → 3 taken 2 times.
✓ Branch 2 → 15 taken 3 times.
✓ Branch 4 → 5 taken 2 times.
✗ Branch 4 → 15 not taken.
✓ Branch 9 → 10 taken 2 times.
✗ Branch 53 → 54 not taken.
✗ Branch 53 → 55 not taken.
|
7 | static const QRegularExpression safeId(QStringLiteral("^[A-Za-z0-9-]{1,64}$")); |
| 63 |
3/4✓ Branch 16 → 17 taken 5 times.
✓ Branch 17 → 18 taken 5 times.
✗ Branch 19 → 20 not taken.
✓ Branch 19 → 21 taken 5 times.
|
5 | if (!safeId.match(slotId).hasMatch()) return {}; |
| 64 | // S106-234: the account id is part of the file name too — only a plain | ||
| 65 | // path component (the S106-198 rule) is accepted. Today's ids are slugs | ||
| 66 | // or UUIDs; the existing slot names stay as they are. | ||
| 67 |
2/2✓ Branch 22 → 23 taken 2 times.
✓ Branch 22 → 24 taken 3 times.
|
5 | if (!SafePathComponent::isSafe(accountId)) return {}; |
| 68 |
4/4✓ Branch 28 → 29 taken 3 times.
✓ Branch 31 → 32 taken 3 times.
✓ Branch 32 → 33 taken 3 times.
✓ Branch 33 → 34 taken 3 times.
|
6 | return m_directory + QLatin1Char('/') + accountId + QLatin1String("-session-") + |
| 69 |
2/2✓ Branch 34 → 35 taken 3 times.
✓ Branch 35 → 36 taken 3 times.
|
9 | slotId + QStringLiteral(".json"); |
| 70 | } | ||
| 71 | |||
| 72 | 1 | QJsonObject ComposeDraftStore::loadSlot(const QString &accountId, | |
| 73 | const QString &slotId) const { | ||
| 74 |
1/1✓ Branch 2 → 3 taken 1 time.
|
1 | const QString file = slotPath(accountId, slotId); |
| 75 |
1/3✗ Branch 4 → 5 not taken.
✓ Branch 4 → 7 taken 1 time.
✗ Branch 5 → 6 not taken.
|
1 | if (file.isEmpty()) return {}; |
| 76 |
1/1✓ Branch 7 → 8 taken 1 time.
|
1 | const QFileInfo info(file); |
| 77 |
5/9✓ Branch 8 → 9 taken 1 time.
✓ Branch 9 → 10 taken 1 time.
✗ Branch 9 → 12 not taken.
✓ Branch 10 → 11 taken 1 time.
✗ Branch 11 → 12 not taken.
✓ Branch 11 → 13 taken 1 time.
✗ Branch 14 → 15 not taken.
✓ Branch 14 → 17 taken 1 time.
✗ Branch 15 → 16 not taken.
|
1 | if (info.isSymLink() || info.size() > MaximumBytes) return {}; |
| 78 |
1/1✓ Branch 17 → 18 taken 1 time.
|
1 | QFile input(file); |
| 79 |
2/4✓ Branch 19 → 20 taken 1 time.
✗ Branch 20 → 21 not taken.
✓ Branch 20 → 23 taken 1 time.
✗ Branch 21 → 22 not taken.
|
1 | if (!input.open(QIODevice::ReadOnly)) return {}; |
| 80 |
1/1✓ Branch 23 → 24 taken 1 time.
|
1 | const QByteArray bytes = input.read(MaximumBytes + 1); |
| 81 |
4/8✓ Branch 25 → 26 taken 1 time.
✗ Branch 25 → 28 not taken.
✓ Branch 26 → 27 taken 1 time.
✗ Branch 27 → 28 not taken.
✓ Branch 27 → 29 taken 1 time.
✗ Branch 30 → 31 not taken.
✓ Branch 30 → 33 taken 1 time.
✗ Branch 31 → 32 not taken.
|
1 | if (bytes.size() > MaximumBytes || input.error() != QFileDevice::NoError) return {}; |
| 82 |
2/2✓ Branch 33 → 34 taken 1 time.
✓ Branch 34 → 35 taken 1 time.
|
1 | const auto object = QJsonDocument::fromJson(bytes).object(); |
| 83 |
7/17✓ Branch 36 → 37 taken 1 time.
✓ Branch 37 → 38 taken 1 time.
✓ Branch 38 → 39 taken 1 time.
✓ Branch 40 → 41 taken 1 time.
✗ Branch 40 → 45 not taken.
✓ Branch 53 → 54 taken 1 time.
✗ Branch 53 → 55 not taken.
✓ Branch 55 → 56 taken 1 time.
✗ Branch 55 → 57 not taken.
✗ Branch 57 → 58 not taken.
✓ Branch 57 → 60 taken 1 time.
✗ Branch 80 → 81 not taken.
✗ Branch 80 → 82 not taken.
✗ Branch 84 → 85 not taken.
✗ Branch 84 → 86 not taken.
✗ Branch 88 → 89 not taken.
✗ Branch 88 → 90 not taken.
|
3 | if (object.value("accountConfigId").toString() != accountId || |
| 84 |
7/15✓ Branch 41 → 42 taken 1 time.
✓ Branch 42 → 43 taken 1 time.
✓ Branch 43 → 44 taken 1 time.
✗ Branch 44 → 45 not taken.
✓ Branch 44 → 46 taken 1 time.
✓ Branch 47 → 48 taken 1 time.
✗ Branch 47 → 49 not taken.
✓ Branch 49 → 50 taken 1 time.
✗ Branch 49 → 51 not taken.
✓ Branch 51 → 52 taken 1 time.
✗ Branch 51 → 53 not taken.
✗ Branch 72 → 73 not taken.
✗ Branch 72 → 74 not taken.
✗ Branch 76 → 77 not taken.
✗ Branch 76 → 78 not taken.
|
2 | object.value("cleared").toBool()) |
| 85 | ✗ | return {}; | |
| 86 | 1 | return object; | |
| 87 | 1 | } | |
| 88 | |||
| 89 | 4 | bool ComposeDraftStore::saveSlot(const QString &accountId, const QString &slotId, | |
| 90 | QJsonObject draft) const { | ||
| 91 |
1/1✓ Branch 2 → 3 taken 4 times.
|
4 | const QString file = slotPath(accountId, slotId); |
| 92 |
2/2✓ Branch 4 → 5 taken 2 times.
✓ Branch 4 → 6 taken 2 times.
|
4 | if (file.isEmpty()) return false; |
| 93 |
4/4✓ Branch 6 → 7 taken 2 times.
✓ Branch 7 → 8 taken 2 times.
✓ Branch 8 → 9 taken 2 times.
✓ Branch 9 → 10 taken 2 times.
|
2 | draft["accountConfigId"] = accountId; |
| 94 |
4/4✓ Branch 12 → 13 taken 2 times.
✓ Branch 13 → 14 taken 2 times.
✓ Branch 14 → 15 taken 2 times.
✓ Branch 15 → 16 taken 2 times.
|
2 | draft["sessionId"] = slotId; |
| 95 |
2/2✓ Branch 18 → 19 taken 2 times.
✓ Branch 19 → 20 taken 2 times.
|
2 | const auto bytes = QJsonDocument(draft).toJson(); |
| 96 |
4/8✓ Branch 23 → 24 taken 2 times.
✓ Branch 24 → 25 taken 2 times.
✓ Branch 25 → 26 taken 2 times.
✗ Branch 25 → 33 not taken.
✗ Branch 43 → 44 not taken.
✓ Branch 43 → 45 taken 2 times.
✗ Branch 95 → 96 not taken.
✗ Branch 95 → 97 not taken.
|
4 | if (bytes.size() > MaximumBytes || QFileInfo(m_directory).isSymLink() || |
| 97 |
11/24✓ Branch 22 → 23 taken 2 times.
✗ Branch 22 → 33 not taken.
✓ Branch 26 → 27 taken 2 times.
✓ Branch 27 → 28 taken 2 times.
✓ Branch 28 → 29 taken 2 times.
✗ Branch 28 → 33 not taken.
✓ Branch 30 → 31 taken 2 times.
✓ Branch 31 → 32 taken 2 times.
✗ Branch 32 → 33 not taken.
✓ Branch 32 → 34 taken 2 times.
✓ Branch 35 → 36 taken 2 times.
✗ Branch 35 → 37 not taken.
✓ Branch 37 → 38 taken 2 times.
✗ Branch 37 → 39 not taken.
✓ Branch 39 → 40 taken 2 times.
✗ Branch 39 → 41 not taken.
✓ Branch 41 → 42 taken 2 times.
✗ Branch 41 → 43 not taken.
✗ Branch 83 → 84 not taken.
✗ Branch 83 → 85 not taken.
✗ Branch 87 → 88 not taken.
✗ Branch 87 → 89 not taken.
✗ Branch 91 → 92 not taken.
✗ Branch 91 → 93 not taken.
|
4 | QFileInfo(file).isSymLink() || !QDir().mkpath(m_directory)) return false; |
| 98 |
1/1✓ Branch 45 → 46 taken 2 times.
|
2 | QSaveFile out(file); |
| 99 |
1/1✓ Branch 46 → 47 taken 2 times.
|
2 | out.setDirectWriteFallback(false); |
| 100 |
1/1✓ Branch 48 → 49 taken 2 times.
|
2 | return out.open(QIODevice::WriteOnly) && |
| 101 |
3/5✓ Branch 51 → 52 taken 2 times.
✓ Branch 52 → 53 taken 2 times.
✗ Branch 52 → 59 not taken.
✓ Branch 55 → 56 taken 2 times.
✗ Branch 55 → 59 not taken.
|
4 | out.setPermissions(QFile::ReadOwner | QFile::WriteOwner) && |
| 102 |
4/6✓ Branch 49 → 50 taken 2 times.
✗ Branch 49 → 59 not taken.
✓ Branch 53 → 54 taken 2 times.
✓ Branch 56 → 57 taken 2 times.
✓ Branch 57 → 58 taken 2 times.
✗ Branch 57 → 59 not taken.
|
6 | out.write(bytes) == bytes.size() && out.commit(); |
| 103 | 4 | } | |
| 104 | |||
| 105 | ✗ | bool ComposeDraftStore::clearSlot(const QString &accountId, | |
| 106 | const QString &slotId) const { | ||
| 107 | ✗ | const QString file = slotPath(accountId, slotId); | |
| 108 | ✗ | if (file.isEmpty()) return false; | |
| 109 | // Remove outright: a cleared marker would keep the slot listed after a | ||
| 110 | // finished session; absence is the natural "no session draft" state. | ||
| 111 | ✗ | return !QFile::exists(file) || QFile::remove(file); | |
| 112 | ✗ | } | |
| 113 | |||
| 114 | 7 | bool ComposeDraftStore::clear(const QString &accountId) const { | |
| 115 |
3/5✓ Branch 4 → 5 taken 7 times.
✓ Branch 8 → 9 taken 7 times.
✓ Branch 8 → 10 taken 7 times.
✗ Branch 15 → 16 not taken.
✗ Branch 15 → 17 not taken.
|
21 | return save(accountId, QJsonObject{{"cleared", true}}); |
| 116 | 14 | } | |
| 117 | |||
| 118 | 807 | QStringList ComposeDraftStore::migrationSources(const QString &accountId, bool ownsUnscopedLegacy) const { | |
| 119 | 807 | QStringList sources; | |
| 120 |
1/1✓ Branch 2 → 3 taken 807 times.
|
807 | const QString scoped = path(accountId); |
| 121 |
5/5✓ Branch 3 → 4 taken 807 times.
✓ Branch 4 → 5 taken 807 times.
✓ Branch 6 → 7 taken 1 time.
✓ Branch 6 → 8 taken 806 times.
✓ Branch 7 → 37 taken 1 time.
|
807 | if (QFileInfo(scoped).isFile()) sources.append(scoped); |
| 122 |
2/2✓ Branch 8 → 9 taken 793 times.
✓ Branch 8 → 37 taken 13 times.
|
806 | else if (ownsUnscopedLegacy) |
| 123 |
5/9✓ Branch 10 → 11 taken 793 times.
✓ Branch 11 → 12 taken 793 times.
✓ Branch 15 → 16 taken 793 times.
✓ Branch 29 → 22 taken 1586 times.
✓ Branch 29 → 30 taken 793 times.
✗ Branch 85 → 86 not taken.
✗ Branch 85 → 89 not taken.
✗ Branch 87 → 88 not taken.
✗ Branch 87 → 89 not taken.
|
4758 | for (const QString &legacy : {path({}), QDir(m_directory).filePath(QStringLiteral("compose-draft.json"))}) |
| 124 |
3/4✓ Branch 22 → 23 taken 1586 times.
✓ Branch 23 → 24 taken 1586 times.
✗ Branch 25 → 26 not taken.
✓ Branch 25 → 28 taken 1586 times.
|
1586 | if (QFileInfo(legacy).isFile()) { |
| 125 | ✗ | sources.append(legacy); | |
| 126 | ✗ | break; | |
| 127 | 2379 | } | |
| 128 |
1/2✓ Branch 38 → 39 taken 807 times.
✗ Branch 38 → 67 not taken.
|
807 | if (SafePathComponent::isSafe(accountId)) |
| 129 | 807 | for (const QFileInfo &slot : | |
| 130 |
8/10✓ Branch 39 → 40 taken 807 times.
✓ Branch 47 → 48 taken 807 times.
✓ Branch 50 → 51 taken 807 times.
✓ Branch 50 → 52 taken 807 times.
✓ Branch 55 → 56 taken 807 times.
✓ Branch 56 → 57 taken 807 times.
✓ Branch 64 → 58 taken 1 time.
✓ Branch 64 → 65 taken 807 times.
✗ Branch 104 → 105 not taken.
✗ Branch 104 → 106 not taken.
|
4036 | QDir(m_directory).entryInfoList({accountId + QStringLiteral("-session-*.json")}, QDir::Files, QDir::Name)) |
| 131 |
2/2✓ Branch 59 → 60 taken 1 time.
✓ Branch 60 → 61 taken 1 time.
|
808 | sources.append(slot.absoluteFilePath()); |
| 132 | 807 | return sources; | |
| 133 | 2421 | } | |
| 134 | |||
| 135 | 2 | QJsonObject ComposeDraftStore::readFile(const QString &path) { | |
| 136 |
1/1✓ Branch 2 → 3 taken 2 times.
|
2 | QFile file(path); |
| 137 |
9/17✓ Branch 3 → 4 taken 2 times.
✓ Branch 4 → 5 taken 2 times.
✓ Branch 5 → 6 taken 2 times.
✗ Branch 5 → 11 not taken.
✓ Branch 6 → 7 taken 2 times.
✓ Branch 7 → 8 taken 2 times.
✗ Branch 7 → 11 not taken.
✓ Branch 9 → 10 taken 2 times.
✗ Branch 10 → 11 not taken.
✓ Branch 10 → 12 taken 2 times.
✓ Branch 13 → 14 taken 2 times.
✗ Branch 13 → 15 not taken.
✗ Branch 15 → 16 not taken.
✓ Branch 15 → 18 taken 2 times.
✗ Branch 16 → 17 not taken.
✗ Branch 31 → 32 not taken.
✗ Branch 31 → 33 not taken.
|
2 | if (QFileInfo(file).isSymLink() || file.size() > MaximumBytes || !file.open(QIODevice::ReadOnly)) return {}; |
| 138 |
2/2✓ Branch 18 → 19 taken 2 times.
✓ Branch 19 → 20 taken 2 times.
|
2 | const auto document = QJsonDocument::fromJson(file.read(MaximumBytes + 1)); |
| 139 |
3/5✓ Branch 21 → 22 taken 2 times.
✓ Branch 22 → 23 taken 2 times.
✗ Branch 22 → 24 not taken.
✓ Branch 23 → 25 taken 2 times.
✗ Branch 24 → 25 not taken.
|
2 | return document.isObject() ? document.object() : QJsonObject{}; |
| 140 | 2 | } | |
| 141 | |||
| 142 | 2 | bool ComposeDraftStore::retire(const QString &path) { | |
| 143 |
1/1✓ Branch 5 → 6 taken 2 times.
|
2 | const QString target = path + QStringLiteral(".v1-migrated"); |
| 144 |
1/1✓ Branch 8 → 9 taken 2 times.
|
2 | QFile::remove(target); |
| 145 |
1/1✓ Branch 9 → 10 taken 2 times.
|
4 | return QFile::rename(path, target); |
| 146 | 2 | } | |
| 147 |