Commit c5aa11c3 authored by Luis Javier Merino's avatar Luis Javier Merino Committed by Tomaz Canabrava
Browse files

HistoryTest: don't test things that make no sense

testHistoryScroll() was trying to get the line length of lines that
didn't exist.
parent 2f7a6d9b
......@@ -93,8 +93,6 @@ void HistoryTest::testHistoryScroll()
historyScroll = new HistoryScrollNone();
QVERIFY(!historyScroll->hasScroll());
QCOMPARE(historyScroll->getLines(), 0);
QCOMPARE(historyScroll->getLineLen(0), 0);
QCOMPARE(historyScroll->getLineLen(10), 0);
const HistoryType &historyTypeNone = historyScroll->getType();
QCOMPARE(historyTypeNone.isEnabled(), false);
......@@ -107,8 +105,6 @@ void HistoryTest::testHistoryScroll()
historyScroll = new HistoryScrollFile();
QVERIFY(historyScroll->hasScroll());
QCOMPARE(historyScroll->getLines(), 0);
QCOMPARE(historyScroll->getLineLen(0), 0);
QCOMPARE(historyScroll->getLineLen(10), 0);
const HistoryType &historyTypeFile = historyScroll->getType();
QCOMPARE(historyTypeFile.isEnabled(), true);
......@@ -121,8 +117,6 @@ void HistoryTest::testHistoryScroll()
historyScroll = new CompactHistoryScroll(42);
QVERIFY(historyScroll->hasScroll());
QCOMPARE(historyScroll->getLines(), 0);
QCOMPARE(historyScroll->getLineLen(0), 0);
QCOMPARE(historyScroll->getLineLen(10), 0);
const HistoryType &compactHistoryType = historyScroll->getType();
QCOMPARE(compactHistoryType.isEnabled(), true);
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment