kde/dolphin
kde/dolphin
1
0
Fork 0
mirror of https://invent.kde.org/system/dolphin synced 2024-10-28 03:21:56 +00:00
Code Issues Packages Projects Releases Wiki Activity

[PlacesItemModelTest] Rename variable

Browse source 
This variable is not the expected size of the groups list, but it's the
expected index of the 'Remote' group in the model.
This commit is contained in:
Elvis Angelaccio 2019-05-05 17:05:46 +02:00
parent afbef7d894
commit 957a6f4f31
1 changed files with 8 additions and 8 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

16
src/tests/placesitemmodeltest.cpp
View file

@ -292,12 +292,12 @@ void PlacesItemModelTest::testModelSort()
void PlacesItemModelTest::testGroups()
{
const auto groups = m_model->groups();
int expectedGroupSize = 3;
int expectedRemoteIndex = 3;
if (m_hasDesktopFolder) {
expectedGroupSize++;
expectedRemoteIndex++;
}
if (m_hasDownloadsFolder) {
expectedGroupSize++;
expectedRemoteIndex++;
}
QCOMPARE(groups.size(), 6);
@ -305,19 +305,19 @@ void PlacesItemModelTest::testGroups()
QCOMPARE(groups.at(0).first, 0);
QCOMPARE(groups.at(0).second.toString(), QStringLiteral("Places"));
QCOMPARE(groups.at(1).first, expectedGroupSize);
QCOMPARE(groups.at(1).first, expectedRemoteIndex);
QCOMPARE(groups.at(1).second.toString(), QStringLiteral("Remote"));
QCOMPARE(groups.at(2).first, expectedGroupSize + 2);
QCOMPARE(groups.at(2).first, expectedRemoteIndex + 2);
QCOMPARE(groups.at(2).second.toString(), QStringLiteral("Recently Saved"));
QCOMPARE(groups.at(3).first, expectedGroupSize + 4);
QCOMPARE(groups.at(3).first, expectedRemoteIndex + 4);
QCOMPARE(groups.at(3).second.toString(), QStringLiteral("Search For"));
QCOMPARE(groups.at(4).first, expectedGroupSize + 8);
QCOMPARE(groups.at(4).first, expectedRemoteIndex + 8);
QCOMPARE(groups.at(4).second.toString(), QStringLiteral("Devices"));
QCOMPARE(groups.at(5).first, expectedGroupSize + 9);
QCOMPARE(groups.at(5).first, expectedRemoteIndex + 9);
QCOMPARE(groups.at(5).second.toString(), QStringLiteral("Removable Devices"));
}