[SCM] ksystemlog packaging branch, master, updated. debian/16.08.3-1-43-g203fe58

Maximiliano Curia maxy at moszumanska.debian.org
Tue Oct 31 18:10:42 UTC 2017


Gitweb-URL: http://git.debian.org/?p=pkg-kde/applications/ksystemlog.git;a=commitdiff;h=7da0c54

The following commit has been merged in the master branch:
commit 7da0c545b6cef90ded15cd4056866dfb2ea9a1de
Author: Jonathan Riddell <jr at jriddell.org>
Date:   Thu Jul 7 09:54:53 2016 +0100

    remove patch already in branch
---
 debian/patches/maintainer_fix-test.diff | 91 ---------------------------------
 debian/patches/series                   |  1 -
 2 files changed, 92 deletions(-)

diff --git a/debian/patches/maintainer_fix-test.diff b/debian/patches/maintainer_fix-test.diff
deleted file mode 100644
index 593f971..0000000
--- a/debian/patches/maintainer_fix-test.diff
+++ /dev/null
@@ -1,91 +0,0 @@
-commit 8283de282f103501ae8d0108db1c52feddfc331c
-Author: Harald Sitter <sitter at kde.org>
-Date:   Wed Jul 6 16:58:23 2016 +0200
-
-    make kiologfilereader test actually do something
-    
-    previously:
-    - it read from http which would error in the kjob but since
-      the kiologreader class has no facilities to communicate errors this
-      gets ignored
-    - it consturcted a !local! url from an http uri string resulting in kio
-      constructing a file slave for http://localhost...
-    - it had a 100 second wait for no good reason
-    - it didn't do any verification of any sort
-    - all in all this test was a fancy sleep 100
-    
-    new:
-    - replaced excess slot with lambda to compress test case to make it
-      more readable in one go
-    - find fixture on-disk via QFINDTESTDATA
-    - make logfile read that
-    - connect to read lines and line-for-line compare them with the fixture
-      (read via qtextstream)
-    
-    i.e. this test now actually checks that the logfile class can read lines
-    
-    REVIEW: 128328
-
-diff --git a/tests/kioLogFileReaderTest.cpp b/tests/kioLogFileReaderTest.cpp
-index 4c15f34..56a3390 100644
---- a/tests/kioLogFileReaderTest.cpp
-+++ b/tests/kioLogFileReaderTest.cpp
-@@ -2,6 +2,8 @@
-  *   KSystemLog, a system log viewer tool                                  *
-  *   Copyright (C) 2007 by Nicolas Ternisien                               *
-  *   nicolas.ternisien at gmail.com                                           *
-+ *   Copyright (C) 2016 by Harald Sitter                                   *
-+ *   sitter at kde.org                                                        *
-  *                                                                         *
-  *   This program is free software; you can redistribute it and/or modify  *
-  *   it under the terms of the GNU General Public License as published by  *
-@@ -43,8 +45,6 @@ private slots:
- 
-     void testKioLogFileReader();
- 
--    void readLine(const QString &line);
--
- private:
-     TestUtil testUtil;
- };
-@@ -56,27 +56,24 @@ void KioLogFileReaderTest::initTestCase()
- 
- void KioLogFileReaderTest::testKioLogFileReader()
- {
--    /*
--    QList<LogFile> logFiles = testUtil.createLogFiles(":/logs/logFileReader/file.txt");
--    LogFile logFile = logFiles.first();
--    */
--
--    /// home/nicolas/test.txt
--    LogFile logFile(QUrl::fromLocalFile(QStringLiteral("http://localhost/test.txt")),
-+    QString fixturePath = QFINDTESTDATA("testFiles/logFileReader/file.txt");
-+    LogFile logFile(QUrl::fromLocalFile(fixturePath),
-                     Globals::instance().informationLogLevel());
- 
-     KioLogFileReader *logFileReader = new KioLogFileReader(logFile);
--
-     logFileReader->open();
- 
--    connect(logFileReader, &KioLogFileReader::lineRead, this, &KioLogFileReaderTest::readLine);
--
--    QTest::qWait(100000);
--}
--
--void KioLogFileReaderTest::readLine(const QString &line)
--{
--    logDebug() << "Line " << line << endl;
-+    connect(logFileReader, &KioLogFileReader::lineRead, this, [=](const QString &line) {
-+        logDebug() << "Line " << line << endl;
-+        static QFile file(fixturePath);
-+        static bool open = false;
-+        if (!open) {
-+            QVERIFY(file.open(QIODevice::ReadOnly | QIODevice::Text));
-+            open = true;
-+        }
-+        static QTextStream stream(&file);
-+        QCOMPARE(line, stream.readLine());
-+    });
- }
- 
- QTEST_MAIN(KioLogFileReaderTest)
diff --git a/debian/patches/series b/debian/patches/series
index d647521..f2db06b 100644
--- a/debian/patches/series
+++ b/debian/patches/series
@@ -1,2 +1 @@
-maintainer_fix-test.diff
 maintainer_systemd_pkgconfig.diff

-- 
ksystemlog packaging



More information about the pkg-kde-commits mailing list