Add search password feature (equivalent to pass find)

plugins/Pass/pass.cpp

@@ -1,7 +1,8 @@
 #include <QUrl>
 #include <QtCore/QStandardPaths>
 #include <QtCore/QDir>
-
+#include <QDirIterator>
+#include <QtConcurrent/QtConcurrent>
 #include "jobs/decryptjob.h"
 #include "jobs/deletekeyjob.h"
 #include "jobs/getkeysjob.h"
@@ -61,8 +62,35 @@ void Pass::initPasswordStore()
     qInfo() << "[Pass] Password Store is :" << m_password_store;
 }
 
+void Pass::lsJob()
+{
+    QDirIterator it(this->m_password_store, QStringList() << "*.gpg", QDir::Files, QDirIterator::Subdirectories);
+    QList<QString> ret;
+    while (it.hasNext()) {
+        QFile f(it.next());
+        QString fname = f.fileName();
+        fname.remove(0, this->m_password_store.length() + 1); // remove system path
+        ret.append(fname);
+    }
+    qInfo() << "[Pass] ls Succeed";
+    emit lsSucceed(ret);
+    this->m_sem->release(1);
+}
+
+bool Pass::ls()
+{
+    qInfo() << "[Pass] ls";
+    if (!this->m_sem->tryAcquire(1, 500)) {
+        qInfo() << "[Pass] A command is already running";
+        return false;
+    }
+    QtConcurrent::run(this, &Pass::lsJob );
+    return true;
+}
+
 bool Pass::show(QUrl url)
 {
+    qInfo() << "[Pass] Show";
     if (!this->m_sem->tryAcquire(1, 500)) {
         qInfo() << "[Pass] A command is already running";
         return false;
@@ -111,7 +139,7 @@ void Pass::slotDeletePasswordStoreResult(bool err)
 {
     this->initPasswordStore(); // reinit an empty password-store
     if (err) {
-        qInfo() << "[Pass]  delete Password Store Failed";
+        qInfo() << "[Pass] Delete Password Store Failed";
         emit deletePasswordStoreFailed("failed to delete password store");
     } else {
         qInfo() << "[Pass] Delete Password Store Succeed";

plugins/Pass/pass.h

@@ -118,6 +118,9 @@ signals:
      */
     void showSucceed(QString name, QString text);
 
+
+    void lsSucceed(QList<QString>);
+
     /**
      * @brief Emitted when showing a password fails.
      * @param message The error message describing the failure.
@@ -160,6 +163,8 @@ private:
      */
     void initPasswordStore();
 
+    void lsJob();
+
 public:
     /**
      * @brief Constructs the Pass object.
@@ -229,6 +234,12 @@ public:
 
     // Password store-related methods
 
+    /**
+     * @brief Get the list of password.
+     * @return The list of password in the password store.
+     */
+    Q_INVOKABLE bool ls();
+
     /**
      * @brief Launch the job to shows the password associated with the specified URL.
      * @param url The URL pointing to the password store entry.

qml/components/FileDir.qml

@@ -6,15 +6,33 @@ import Lomiri.Components.Themes 1.3
 import Pass 1.0
 import QtQuick 2.4
 
-Component {
+Item {
+    //property string folder
+
+    id: fileDir
+
+    property string fName
+    property bool fIsDir
+    property bool commonBorder: true
+    property int lBorderwidth: 0
+    property int rBorderwidth: 0
+    property int tBorderwidth: 0
+    property int bBorderwidth: 0
+    property int commonBorderWidth: 0
+    property string borderColor: LomiriColors.warmGrey
+
+    signal clicked()
+
+    anchors.right: parent.right
+    anchors.left: parent.left
+    height: units.gu(5)
+
     Rectangle {
-        anchors.right: parent.right
+        anchors.fill: parent
-        anchors.left: parent.left
-        height: units.gu(5)
         color: theme.palette.normal.background
 
         Text {
-            text: fileIsDir ? fileName : fileName.slice(0, -4) // remove .gpg if it's a file
+            text: fileDir.fIsDir ? fileDir.fName : fileDir.fName.slice(0, -4) // remove .gpg if it's a file
             anchors.left: parent.left
             anchors.leftMargin: units.gu(2)
             anchors.verticalCenter: parent.verticalCenter
@@ -26,32 +44,36 @@ Component {
             anchors.verticalCenter: parent.verticalCenter
             anchors.rightMargin: units.gu(2)
             height: units.gu(4)
-            name: fileIsDir ? "go-next" : "lock"
+            name: fileDir.fIsDir ? "go-next" : "lock"
             color: LomiriColors.orange
         }
 
         MouseArea {
+            // onClicked: {
+            //     var path = fileDir.fdfolder + "/" + fileName;
+            //     if (fileIsDir) {
+            //         fileDir.fdfolder = path;
+            //         //backAction.visible = true;
+            //         // passwordListHeader.title = fileName;
+            //     } else {
+            //         console.debug("pass show %1".arg(path));
+            //         Pass.show(path);
+            //     }
+            // }
+
             anchors.fill: parent
-            onClicked: {
+            onClicked: fileDir.clicked()
-                var path = folderModel.folder + "/" + fileName;
-                if (fileIsDir) {
-                    folderModel.folder = path;
-                    backAction.visible = true;
-                    passwordListHeader.title = fileName;
-                } else {
-                    console.debug("pass show %1".arg(path));
-                    Pass.show(path);
-                }
-            }
         }
 
         CustomBorder {
-            commonBorder: false
+            id: cb
-            lBorderwidth: 0
+
-            rBorderwidth: 0
+            commonBorder: fileDir.commonBorder
-            tBorderwidth: 0
+            lBorderwidth: fileDir.lBorderwidth
-            bBorderwidth: 1
+            rBorderwidth: fileDir.rBorderwidth
-            borderColor: LomiriColors.warmGrey
+            tBorderwidth: fileDir.tBorderwidth
+            bBorderwidth: fileDir.bBorderwidth
+            borderColor: fileDir.borderColor
         }
 
     }

qml/pages/PasswordList.qml

@@ -10,11 +10,36 @@ import "headers"
 Page {
     id: passwordListPage
 
-    property string passwordStorePath
+    property string __passwordStorePath
+    property var __passwords
+
+    function __searchPasswords(filter) {
+        var ret = [];
+        if (__passwords) {
+            for (var i = 0; i < __passwords.length; i++) {
+                if (__passwords[i].toUpperCase().indexOf(filter.toUpperCase()) > -1)
+                    ret.push(__passwords[i]);
+
+            }
+        }
+        return ret;
+    }
+
+    function __searchUpdateModel(text) {
+        var ret = __searchPasswords(text);
+        passwordListSearch.model.clear();
+        for (var i = 0; i < ret.length; i++) {
+            if (ret[i])
+                passwordListSearch.model.append({
+                    "fileName": ret[i]
+                });
+
+        }
+    }
 
     anchors.fill: parent
     Component.onCompleted: {
-        passwordStorePath = "file:" + Pass.password_store;
+        __passwordStorePath = "file:" + Pass.password_store;
         Pass.onShowSucceed.connect(function(filename, text) {
             pageStack.push(Qt.resolvedUrl("../pages/Password.qml"), {
                 "plainText": text,
@@ -24,16 +49,22 @@ Page {
         Pass.onShowFailed.connect(function(message) {
             PopupUtils.open(passwordPageDecryptError);
         });
+        Pass.onLsSucceed.connect(function(passwords) {
+            __passwords = passwords;
+        });
+        Pass.ls();
     }
 
     Column {
+        id: passwordListEmpty
+
         anchors.top: passwordListHeader.bottom
         anchors.bottom: parent.bottom
         anchors.right: parent.right
         anchors.left: parent.left
         anchors.leftMargin: units.gu(2)
         anchors.rightMargin: units.gu(2)
-        visible: folderModel.count == 0
+        visible: passwordListNav.model.count === 0
 
         Rectangle {
             width: parent.width
@@ -71,24 +102,66 @@ Page {
     }
 
     ListView {
+        id: passwordListNav
+
         anchors.top: passwordListHeader.bottom
         anchors.bottom: parent.bottom
         anchors.right: parent.right
         anchors.left: parent.left
         spacing: 1
-        visible: folderModel.count != 0
+        visible: passwordListNav.model.count !== 0 && passwordListHeader.searchBarIsActive
 
         model: FolderListModel {
-            id: folderModel
-
             nameFilters: ["*.gpg"]
-            rootFolder: passwordStorePath
+            rootFolder: __passwordStorePath
-            folder: passwordStorePath
+            folder: __passwordStorePath
             showDirs: true
         }
 
-        delegate: FileDir {
+        delegate: Component {
-            id: fileDelegate
+            FileDir {
+                fName: fileName
+                fIsDir: fileIsDir
+                onClicked: {
+                    var path = passwordListNav.model.folder + "/" + fileName;
+                    if (fileIsDir) {
+                        passwordListNav.model.folder = path;
+                        backAction.visible = true;
+                        passwordListHeader.title = fileName;
+                    } else {
+                        console.debug("pass show %1".arg(path));
+                        Pass.show(path);
+                    }
+                }
+            }
+
+        }
+
+    }
+
+    ListView {
+        id: passwordListSearch
+
+        anchors.top: passwordListHeader.bottom
+        anchors.bottom: parent.bottom
+        anchors.right: parent.right
+        anchors.left: parent.left
+        visible: passwordListNav.model.count !== 0 && !passwordListHeader.searchBarIsActive
+
+        model: ListModel {
+        }
+
+        delegate: Component {
+            FileDir {
+                fName: fileName
+                fIsDir: false
+                onClicked: {
+                    var path = __passwordStorePath + "/" + fileName;
+                    console.debug("pass show %1".arg(path));
+                    Pass.show(path);
+                }
+            }
+
         }
 
     }
@@ -105,6 +178,8 @@ Page {
     header: MainHeader {
         id: passwordListHeader
 
+        onSearchBarActived: __searchUpdateModel("")
+        onSearchBarTextChanged: __searchUpdateModel(text)
         leadingActionBar.height: units.gu(4)
         leadingActionBar.actions: [
             Action {
@@ -114,9 +189,9 @@ Page {
                 text: i18n.tr("Back")
                 visible: false
                 onTriggered: {
-                    folderModel.folder = folderModel.parentFolder;
+                    passwordListNav.model.folder = passwordListNav.model.parentFolder;
-                    console.debug(folderModel.folder);
+                    console.debug(passwordListNav.model.folder);
-                    if (folderModel.rootFolder === folderModel.folder) {
+                    if (passwordListNav.model.rootFolder === passwordListNav.model.folder) {
                         backAction.visible = false;
                         passwordListHeader.title = i18n.tr("UTPass");
                     } else {

qml/pages/headers/MainHeader.qml

@@ -4,23 +4,29 @@ import QtQuick 2.4
 PageHeader {
     id: mainHeader
 
+    readonly property bool searchBarIsActive: !searchBar.visible
+
+    signal searchBarActived()
+    signal searchBarTextChanged(string text)
+
     width: parent.width
     height: units.gu(6)
     title: i18n.tr("UTPass")
     trailingActionBar.height: units.gu(4)
     trailingActionBar.numberOfSlots: 2
     trailingActionBar.actions: [
-        /*Action { TODO
+        Action {
-            iconName: "search"
+            iconName: searchBarIsActive ? "search" : "close"
             text: i18n.tr("Search")
             onTriggered: {
-                searchBar.visible = !searchBar.visible
+                searchBar.visible = !searchBar.visible;
-                labelTitle.visible = !searchBar.visible
+                labelTitle.visible = !searchBar.visible;
                 if (searchBar.visible === true) {
-                    searchBar.focus = true
+                    searchBar.focus = true;
+                    searchBarActived();
                 }
             }
-        },*/
+        },
         Action {
             iconName: "settings"
             text: i18n.tr("Settings")
@@ -58,8 +64,7 @@ PageHeader {
             height: units.gu(4)
             visible: false
             anchors.verticalCenter: parent.verticalCenter
-            onFocusChanged: {
+            onContentWidthChanged: searchBarTextChanged(searchBar.text)
-            }
         }
 
     }

tests.in.cpp

@@ -16,5 +16,5 @@ int main(int argc, char *argv[])
 
     QGuiApplication::setApplicationName("utpass.qrouland");
 
-    return quick_test_main(argc, argv, @TESTS_PATH@, @TESTS_PATH@);
+    return quick_test_main(argc, argv, "@TESTS_PATH@", "@TESTS_PATH@");
 }

tests/plugins/TestsUtils/CMakeLists.txt

@@ -5,7 +5,6 @@ set(
         SRC
         plugin.cpp
         utils.cpp
-        passphraseprovider.h
 )
 
 set(CMAKE_AUTOMOC ON)
@@ -23,30 +22,6 @@ endif()
 add_library(${PLUGIN} MODULE ${SRC})
 set_target_properties(${PLUGIN} PROPERTIES LIBRARY_OUTPUT_DIRECTORY ${PLUGIN})
 qt5_use_modules(${PLUGIN} Qml Quick DBus)
-set(RNP_BUILD_DIR "${CMAKE_SOURCE_DIR}/build/${ARCH_TRIPLET}/rnp/install")
-
-INCLUDE_DIRECTORIES(${RNP_BUILD_DIR}/include)
-
-add_library(rnp STATIC IMPORTED)
-set_property(TARGET rnp PROPERTY IMPORTED_LOCATION "${RNP_BUILD_DIR}/lib/librnp.a")
-
-add_library(gpgerror SHARED IMPORTED)
-set_property(TARGET gpgerror PROPERTY IMPORTED_LOCATION "/usr/lib/${ARCH_TRIPLET}/libgpg-error.so.0.28.0")
-
-add_library(libassuan SHARED IMPORTED)
-set_property(TARGET libassuan PROPERTY IMPORTED_LOCATION "/usr/lib/${ARCH_TRIPLET}/libassuan.so")
-
-add_library(libgpgme SHARED IMPORTED)
-set_property(TARGET libgpgme PROPERTY IMPORTED_LOCATION "/usr/lib/${ARCH_TRIPLET}/libgpgme.so")
-
-add_library(libgpgmepp SHARED IMPORTED)
-set_property(TARGET libgpgmepp PROPERTY IMPORTED_LOCATION "/usr/lib/${ARCH_TRIPLET}/libgpgmepp.so")
-
-add_library(libqgpgme SHARED IMPORTED)
-set_property(TARGET libqgpgme PROPERTY IMPORTED_LOCATION "/usr/lib/${ARCH_TRIPLET}/libqgpgme.so")
-
-
-target_link_libraries(${PLUGIN} rnp gpgerror libassuan libgpgme libgpgmepp libqgpgme)
 
 
 set(QT_IMPORTS_DIR "/lib/${ARCH_TRIPLET}")

tests/plugins/TestsUtils/passphraseprovider.h

@@ -3,9 +3,9 @@
 
 #include <QObject>
 #include <gpg-error.h>
-extern "C" {
+// extern "C" {
-#include <rnp/rnp.h>
+// #include <rnp/rnp.h>
-}
+// }
 
 class TesTPassphraseProvider : public QObject
 {

tests/plugins/TestsUtils/utils.cpp

@@ -6,11 +6,11 @@
 #include <memory>
 #include <quazip5/JlCompress.h>
 
-#include "passphraseprovider.h"
+//#include "passphraseprovider.h"
 #include "utils.h"
 
-TestsUtils::TestsUtils():
+TestsUtils::TestsUtils()
-    m_passphrase_povider(std::unique_ptr<TesTPassphraseProvider>(new TesTPassphraseProvider()))
+//m_passphrase_povider(std::unique_ptr<TesTPassphraseProvider>(new TesTPassphraseProvider()))
 {}
 
 
@@ -68,9 +68,9 @@ void TestsUtils::copyFolder(QUrl sourceFolderUrl, QUrl destFolderUrl)
     }
 }
 
-QObject *TestsUtils::getTestPassphraseProvider()
+// QObject *TestsUtils::getTestPassphraseProvider()
-{
+// {
-    return &TesTPassphraseProvider::instance();
+//     return &TesTPassphraseProvider::instance();
-}
+// }
 
 

tests/plugins/TestsUtils/utils.h

@@ -1,7 +1,7 @@
 #ifndef TESTSUTILS_H
 #define TESTSUTILS_H
 
-#include "passphraseprovider.h"
+//#include "passphraseprovider.h"
 #include <QObject>
 #include <QUrl>
 #include <QQuickWindow>
@@ -18,7 +18,7 @@ public:
     Q_INVOKABLE QString getTempPath();
     Q_INVOKABLE bool fileExists(QUrl path);
     Q_INVOKABLE void copyFolder(QUrl sourceFolder, QUrl destFolder);
-    Q_INVOKABLE QObject *getTestPassphraseProvider();
+    //Q_INVOKABLE QObject *getTestPassphraseProvider();
 
 };
 

tests/units/pass/PassTestCase.qml

@@ -4,6 +4,8 @@ import QtTest 1.2
 import TestsUtils 1.0
 
 TestCase {
+    //Pass.passphrase_provider = TestsUtils.getTestPassphraseProvider();
+
     property string password_store
     property string gpg_home
 
@@ -13,7 +15,6 @@ TestCase {
         Pass.gpg_home = gpg_home;
         password_store = TestsUtils.getTempPath();
         Pass.password_store = password_store;
-        Pass.passphrase_provider = TestsUtils.getTestPassphraseProvider();
     }
 
 }

tests/units/pass/tst_ls.qml

@@ -0,0 +1,44 @@
+import Pass 1.0
+import QtQuick 2.9
+import QtTest 1.2
+import TestsUtils 1.0
+
+PassTestCase {
+    //TODO some additionanl error test
+
+    function init_data() {
+        return [{
+            "spy": lsSucceed,
+            "add_home_gpg_data": true,
+            "passwords": ["test.gpg"]
+        }, {
+            "spy": lsSucceed,
+            "add_home_gpg_data": false,
+            "passwords": []
+        }];
+    }
+
+    function test_ls(data) {
+        if (data.add_home_gpg_data === true)
+            TestsUtils.copyFolder(Qt.resolvedUrl("../../assets/password-store"), Qt.resolvedUrl(password_store));
+
+        var passwords;
+        Pass.lsSucceed.connect(function(ret) {
+            passwords = ret;
+        });
+        Pass.ls();
+        data.spy.wait();
+        verify(passwords.length === data.passwords.length, "Should return %1 password(s) but return %2 password(s)".arg(data.nb_password).arg(passwords.length));
+        for (var i = 0; data.passwords.length; i++) {
+            verify(passwords[i] === data.passwords[i], "%1 name should be %2 but is %3".arg(i).arg(data.passwords[i]).arg(passwords[i]));
+        }
+    }
+
+    SignalSpy {
+        id: lsSucceed
+
+        target: Pass
+        signalName: "lsSucceed"
+    }
+
+}

tests/units/pass/tst_show.qml

@@ -4,24 +4,24 @@ import QtTest 1.2
 import TestsUtils 1.0
 
 PassTestCase {
+    // TODO This test need to fixed by providing custom stub password provider to the pass plugin for succeed tests
     //TODO some additionanl error test
-
     function init_data() {
         return [{
             "spy": showFailed,
             "err_msg": "Bad password",
-            "add_home_gpg_data": true,
+            "add_password_store_data": true,
             "file": "../../assets/gpg/clear_text.txt.gpg"
         }, {
             "spy": showFailed,
             "err_msg": "No suitable key",
-            "add_home_gpg_data": false,
+            "add_password_store_data": false,
             "file": "../../assets/gpg/clear_text.txt.gpg"
         }];
     }
 
     function test_pass_show(data) {
-        if (data.add_home_gpg_data === true)
+        if (data.add_password_store_data === true)
             TestsUtils.copyFolder(Qt.resolvedUrl("../../assets/gpghome"), Qt.resolvedUrl(gpg_home));
 
         var fname, ctext;