summaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authoraqua <aqua@iserlohn-fortress.net>2022-08-20 16:37:42 +0300
committeraqua <aqua@iserlohn-fortress.net>2022-08-28 09:48:45 +0300
commitf7ccea7bca79f1dc21e497872a419446f20d211e (patch)
tree87ad85a6e402908ce5c7f3439e0e25b889333c1c /src
parentAdd rekonf script to generate SettingsWidgets (diff)
downloadrekonq-f7ccea7bca79f1dc21e497872a419446f20d211e.tar.xz
Add Setting and MockSettings
- rename Settings to RekonqSettings
Diffstat (limited to 'src')
-rw-r--r--src/settings/CMakeLists.txt16
-rw-r--r--src/settings/settings.cpp28
-rw-r--r--src/settings/settings.hpp30
-rw-r--r--src/settings/settingsdialog.cpp10
-rw-r--r--src/settings/settingsdialog.h5
-rw-r--r--src/settings/settingswidgets.hpp11
-rw-r--r--src/settings/test/dialog.cpp12
-rw-r--r--src/settings/test/fonts.cpp31
-rw-r--r--src/settings/test/settings_mock.hpp18
-rw-r--r--src/settings/test/test_settings.cpp71
-rw-r--r--src/settings/test/test_settingsdialog.cpp20
11 files changed, 193 insertions, 59 deletions
diff --git a/src/settings/CMakeLists.txt b/src/settings/CMakeLists.txt
index 91d8e5d6..f597cfb9 100644
--- a/src/settings/CMakeLists.txt
+++ b/src/settings/CMakeLists.txt
@@ -16,6 +16,7 @@ add_custom_command(OUTPUT shortcutssettingswidget.cpp DEPENDS ${PROJECT_SOURCE_D
${PROJECT_SOURCE_DIR}/src/rekonq.kcfg)
add_library(settings STATIC
+ ${PROJECT_SOURCE_DIR}/include/rsettings.hpp settings.cpp settings.hpp
settingsdialog.cpp settingsdialog.h settingsdialog.ui
settingswidgets.hpp helpers.hpp
generalsettingswidget.cpp appearancesettingswidget.cpp networksettingswidget.cpp shortcutssettingswidget.cpp
@@ -23,10 +24,15 @@ add_library(settings STATIC
target_link_libraries(settings PUBLIC Qt6::Widgets)
IF(TESTING)
- add_executable(test_settingsdialog test/dialog.cpp)
- target_link_libraries(test_settingsdialog settings)
+ add_executable(test_settingsdialog test/test_settingsdialog.cpp test/settings_mock.hpp)
+ target_link_libraries(test_settingsdialog GTest::gmock settings)
- add_executable(test_fonts test/fonts.cpp)
- target_link_libraries(test_fonts GTest::gtest Qt6::Gui)
- gtest_discover_tests(test_fonts)
+ # test config file
+ add_custom_command(OUTPUT rekonqrc DEPENDS ${PROJECT_SOURCE_DIR}/src/rekonq.kcfg
+ COMMAND python3 ${PROJECT_SOURCE_DIR}/scripts/rekonf.py
+ --group=all --output=rekonqrc ${PROJECT_SOURCE_DIR}/src/rekonq.kcfg)
+
+ add_executable(test_settings test/test_settings.cpp test/settings_mock.hpp rekonqrc)
+ target_link_libraries(test_settings GTest::gtest GTest::gmock settings)
+ gtest_discover_tests(test_settings EXTRA_ARGS rekonqrc)
endif()
diff --git a/src/settings/settings.cpp b/src/settings/settings.cpp
new file mode 100644
index 00000000..1ffa14cb
--- /dev/null
+++ b/src/settings/settings.cpp
@@ -0,0 +1,28 @@
+/* ============================================================
+ * The rekonq project
+ * ============================================================
+ * SPDX-License-Identifier: GPL-3.0-only
+ * Copyright (C) 2022 aqua <aqua@iserlohn-fortress.net>
+ * ============================================================
+ * Description: Application Settings
+ * ============================================================ */
+
+#include "settings.hpp"
+#include <QSettings>
+
+Settings::Settings(const QString &path, QObject *parent) : RekonqSettings(parent)
+{
+ d = new QSettings(path, QSettings::IniFormat, this);
+}
+
+void Settings::beginGroup(const QString &prefix) { d->beginGroup(prefix); }
+
+void Settings::endGroup() { d->endGroup(); }
+
+void Settings::setValue(const QString &key, const QVariant &value)
+{
+ d->setValue(key, value);
+ emit changed(key, value);
+}
+
+QVariant Settings::value(const QString &key, const QVariant &defaultValue) const { return d->value(key, defaultValue); }
diff --git a/src/settings/settings.hpp b/src/settings/settings.hpp
new file mode 100644
index 00000000..c595e981
--- /dev/null
+++ b/src/settings/settings.hpp
@@ -0,0 +1,30 @@
+/* ============================================================
+ * The rekonq project
+ * ============================================================
+ * SPDX-License-Identifier: GPL-3.0-only
+ * Copyright (C) 2022 aqua <aqua@iserlohn-fortress.net>
+ * ============================================================
+ * Description: Application Settings
+ * ============================================================ */
+
+#pragma once
+
+#include <rsettings.hpp>
+
+class QSettings;
+class Settings final : public RekonqSettings {
+ Q_OBJECT
+
+public:
+ explicit Settings(const QString &path, QObject *parent = nullptr);
+ ~Settings() = default;
+
+ void beginGroup(const QString &prefix) override;
+ void endGroup() override;
+
+ void setValue(const QString &key, const QVariant &value) override;
+ QVariant value(const QString &key, const QVariant &defaultValue = QVariant()) const override;
+
+private:
+ QSettings *d;
+};
diff --git a/src/settings/settingsdialog.cpp b/src/settings/settingsdialog.cpp
index e869fe10..0c54856e 100644
--- a/src/settings/settingsdialog.cpp
+++ b/src/settings/settingsdialog.cpp
@@ -14,14 +14,14 @@
#include "settingswidgets.hpp"
#include "ui_settingsdialog.h"
-SettingsDialog::SettingsDialog(QWidget *parent) : QDialog(parent), ui(new Ui::SettingsDialog)
+SettingsDialog::SettingsDialog(RekonqSettings *settings, QWidget *parent) : QDialog(parent), ui(new Ui::SettingsDialog)
{
ui->setupUi(this);
- ui->stackedWidget->addWidget(new GeneralSettingsWidget);
- ui->stackedWidget->addWidget(new AppearanceSettingsWidget);
- ui->stackedWidget->addWidget(new NetworkSettingsWidget);
- ui->stackedWidget->addWidget(new ShortcutsSettingsWidget);
+ ui->stackedWidget->addWidget(new GeneralSettingsWidget(settings, this));
+ ui->stackedWidget->addWidget(new AppearanceSettingsWidget(settings, this));
+ ui->stackedWidget->addWidget(new NetworkSettingsWidget(settings, this));
+ ui->stackedWidget->addWidget(new ShortcutsSettingsWidget(settings, this));
connect(ui->listWidget, &QListWidget::currentRowChanged, ui->stackedWidget, &QStackedWidget::setCurrentIndex);
}
diff --git a/src/settings/settingsdialog.h b/src/settings/settingsdialog.h
index 3f191e1e..0c793f9e 100644
--- a/src/settings/settingsdialog.h
+++ b/src/settings/settingsdialog.h
@@ -14,14 +14,17 @@
#include <QDialog>
+class RekonqSettings;
+
namespace Ui {
class SettingsDialog;
}
+
class SettingsDialog : public QDialog {
Q_OBJECT
public:
- explicit SettingsDialog(QWidget *parent = nullptr);
+ explicit SettingsDialog(RekonqSettings *settings, QWidget *parent = nullptr);
~SettingsDialog() override;
private:
diff --git a/src/settings/settingswidgets.hpp b/src/settings/settingswidgets.hpp
index 31cd4580..27ebe8a2 100644
--- a/src/settings/settingswidgets.hpp
+++ b/src/settings/settingswidgets.hpp
@@ -10,12 +10,13 @@
#pragma once
#include <QWidget>
+#include <rsettings.hpp>
class SettingsWidget : public QWidget {
Q_OBJECT
public:
- explicit SettingsWidget(QWidget *parent = nullptr) : QWidget(parent) {}
+ explicit SettingsWidget(RekonqSettings *, QWidget *parent = nullptr) : QWidget(parent) {}
signals:
void changed();
@@ -27,7 +28,7 @@ public slots:
class GeneralSettingsWidget final : public SettingsWidget {
Q_OBJECT
public:
- explicit GeneralSettingsWidget(QWidget *parent = nullptr);
+ explicit GeneralSettingsWidget(RekonqSettings *, QWidget *parent = nullptr);
public slots:
void save() override;
@@ -37,7 +38,7 @@ public slots:
class AppearanceSettingsWidget final : public SettingsWidget {
Q_OBJECT
public:
- explicit AppearanceSettingsWidget(QWidget *parent = nullptr);
+ explicit AppearanceSettingsWidget(RekonqSettings *, QWidget *parent = nullptr);
public slots:
void save() override;
@@ -47,7 +48,7 @@ public slots:
class NetworkSettingsWidget final : public SettingsWidget {
Q_OBJECT
public:
- explicit NetworkSettingsWidget(QWidget *parent = nullptr);
+ explicit NetworkSettingsWidget(RekonqSettings *, QWidget *parent = nullptr);
public slots:
void save() override;
@@ -57,7 +58,7 @@ public slots:
class ShortcutsSettingsWidget final : public SettingsWidget {
Q_OBJECT
public:
- explicit ShortcutsSettingsWidget(QWidget *parent = nullptr);
+ explicit ShortcutsSettingsWidget(RekonqSettings *, QWidget *parent = nullptr);
public slots:
void save() override;
diff --git a/src/settings/test/dialog.cpp b/src/settings/test/dialog.cpp
deleted file mode 100644
index 2bc9ffe0..00000000
--- a/src/settings/test/dialog.cpp
+++ /dev/null
@@ -1,12 +0,0 @@
-#include "../settingsdialog.h"
-#include <QApplication>
-
-int main(int argc, char **argv)
-{
- QApplication app(argc, argv);
-
- SettingsDialog dlg;
- dlg.show();
-
- return QApplication::exec();
-} \ No newline at end of file
diff --git a/src/settings/test/fonts.cpp b/src/settings/test/fonts.cpp
deleted file mode 100644
index 65cc7229..00000000
--- a/src/settings/test/fonts.cpp
+++ /dev/null
@@ -1,31 +0,0 @@
-#include "../helpers.hpp"
-#include <QFontInfo>
-#include <QGuiApplication>
-#include <gtest/gtest.h>
-#include <vector>
-
-TEST(Settings, getFont)
-{
- const auto serif = getFont(QFont::Serif);
- EXPECT_EQ(serif.styleHint(), QFont::Serif) << qUtf8Printable(serif.toString());
-
- const auto fixed = getFont(QFont::Monospace);
- EXPECT_EQ(fixed.styleHint(), QFont::Monospace) << qUtf8Printable(fixed.toString());
-}
-
-int main(int argc, char **argv)
-{
- ::testing::InitGoogleTest(&argc, argv);
-
- // handling fonts requires a QGuiApplication
- // The proper platform name needs to be added to the argument list before the QGuiApplication constructor is called
- // This needs to be done here for gtest_discover_tests to work
- QList<char *> args;
- for (int i = 0; i < argc; ++i) args.append(argv[i]);
- args.append({"-platform", "offscreen"});
- int args_count = args.count();
-
- QGuiApplication app(args_count, args.data());
-
- return RUN_ALL_TESTS();
-}
diff --git a/src/settings/test/settings_mock.hpp b/src/settings/test/settings_mock.hpp
new file mode 100644
index 00000000..ec792ff5
--- /dev/null
+++ b/src/settings/test/settings_mock.hpp
@@ -0,0 +1,18 @@
+#pragma once
+
+#include <gmock/gmock.h>
+#include <rsettings.hpp>
+
+class MockSettings : public RekonqSettings {
+ Q_OBJECT
+
+public:
+ explicit MockSettings() : RekonqSettings(nullptr) {}
+ ~MockSettings() = default;
+
+ MOCK_METHOD(void, beginGroup, (const QString &), (override));
+ MOCK_METHOD(void, endGroup, (), (override));
+
+ MOCK_METHOD(void, setValue, (const QString &, const QVariant &), (override));
+ MOCK_METHOD(QVariant, value, (const QString &, const QVariant &), (const, override));
+};
diff --git a/src/settings/test/test_settings.cpp b/src/settings/test/test_settings.cpp
new file mode 100644
index 00000000..72582bc7
--- /dev/null
+++ b/src/settings/test/test_settings.cpp
@@ -0,0 +1,71 @@
+#include <QApplication>
+#include <gtest/gtest.h>
+
+#include <iostream>
+
+// Software under Test
+#include "../helpers.hpp"
+#include "../settings.hpp"
+#include "../settingsdialog.h"
+#include "settings_mock.hpp"
+
+using ::testing::AtLeast;
+using ::testing::ReturnArg;
+
+const char *settingsFile = nullptr;
+
+TEST(settings, getFont)
+{
+ const auto serif = getFont(QFont::Serif);
+ EXPECT_EQ(serif.styleHint(), QFont::Serif) << qUtf8Printable(serif.toString());
+
+ const auto fixed = getFont(QFont::Monospace);
+ EXPECT_EQ(fixed.styleHint(), QFont::Monospace) << qUtf8Printable(fixed.toString());
+}
+
+TEST(settings, Settings)
+{
+ Settings settings(settingsFile);
+
+ const auto FirstRun = settings.value("FirstRun");
+ EXPECT_TRUE(FirstRun.isValid());
+ EXPECT_TRUE(FirstRun.toBool());
+
+ settings.beginGroup("Network");
+ const auto downloadPathAsk = settings.value("downloadPathAsk");
+ EXPECT_TRUE(downloadPathAsk.isValid());
+ EXPECT_TRUE(downloadPathAsk.toBool());
+ settings.endGroup();
+
+ const auto searchUrl = settings.value("searchUrl");
+ EXPECT_TRUE(searchUrl.isValid());
+ EXPECT_EQ(searchUrl.toString(), QString("https://duckduckgo.com/?q=%1"));
+}
+
+TEST(settings, SettingsDialog)
+{
+ MockSettings settings;
+ EXPECT_CALL(settings, beginGroup).Times(4);
+ EXPECT_CALL(settings, endGroup).Times(4);
+ EXPECT_CALL(settings, value).Times(AtLeast(4)).WillRepeatedly(ReturnArg<1>());
+
+ SettingsDialog dlg(&settings);
+}
+
+int main(int argc, char **argv)
+{
+ ::testing::InitGoogleTest(&argc, argv);
+ if (argc > 1) settingsFile = argv[argc - 1];
+
+ // handling fonts requires a QGuiApplication
+ // The proper platform name needs to be added to the argument list before the QGuiApplication constructor is called
+ // This needs to be done here for gtest_discover_tests to work
+ QList<char *> args;
+ for (int i = 0; i < argc; ++i) args.append(argv[i]);
+ args.append({"-platform", "offscreen"});
+ int args_count = args.count();
+
+ QApplication app(args_count, args.data());
+
+ return RUN_ALL_TESTS();
+}
diff --git a/src/settings/test/test_settingsdialog.cpp b/src/settings/test/test_settingsdialog.cpp
new file mode 100644
index 00000000..b70c6dda
--- /dev/null
+++ b/src/settings/test/test_settingsdialog.cpp
@@ -0,0 +1,20 @@
+#include "../settingsdialog.h"
+#include "settings_mock.hpp"
+#include <QApplication>
+
+using ::testing::AtLeast;
+using ::testing::NiceMock;
+using ::testing::ReturnArg;
+
+int main(int argc, char **argv)
+{
+ QApplication app(argc, argv);
+
+ NiceMock<MockSettings> settings;
+ EXPECT_CALL(settings, value).WillRepeatedly(ReturnArg<1>());
+
+ SettingsDialog dlg(&settings);
+ dlg.show();
+
+ return QApplication::exec();
+}