gador/nixpkgs
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

lomiri.lomiri-indicator-network: Apply patch to hopefully fix Hydra failures (#377984)

Browse Source
This commit is contained in:
Cosima Neidahl
2025-01-31 20:09:04 +01:00
committed by GitHub
parent 7447b0d899 594309958d
commit 466a1cdd62
2 changed files with 244 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files
pkgs/desktops/lomiri/services/lomiri-indicator-network/1001-test-secret-agent-Make-GetServerInformation-not-leak-into-tests.patch
+240
View File
@@ -0,0 +1,240 @@
From 9c2a6a6349f705017e3c8a34daa4ba1805586498 Mon Sep 17 00:00:00 2001
From: OPNA2608 <opna2608@protonmail.com>
Date: Thu, 30 Jan 2025 14:53:02 +0100
Subject: [PATCH] tests/unit/secret-agent/test-secret-agent: Make sure signal
emitted on agent startup doesn't leak into tests
---
tests/unit/secret-agent/test-secret-agent.cpp | 116 ++++++++++--------
1 file changed, 67 insertions(+), 49 deletions(-)
diff --git a/tests/unit/secret-agent/test-secret-agent.cpp b/tests/unit/secret-agent/test-secret-agent.cpp
index 1f1cd7e9..9c72e251 100644
--- a/tests/unit/secret-agent/test-secret-agent.cpp
+++ b/tests/unit/secret-agent/test-secret-agent.cpp
@@ -29,6 +29,16 @@
#include <lomiri/gmenuharness/MatchUtils.h>
#include <lomiri/gmenuharness/MenuMatcher.h>
+#define WAIT_FOR_SIGNALS(signalSpy, signalsExpected)\
+{\
+ while (signalSpy.size() < signalsExpected)\
+ {\
+ ASSERT_TRUE(signalSpy.wait()) << "Waiting for " << signalsExpected << " signals, got " << signalSpy.size();\
+ }\
+ ASSERT_EQ(signalsExpected, signalSpy.size()) << "Waiting for " << signalsExpected << " signals, got " << signalSpy.size();\
+}
+
+
using namespace std;
using namespace testing;
using namespace QtDBusTest;
@@ -49,21 +59,6 @@ protected:
dbusMock.registerTemplate(NM_DBUS_SERVICE, NETWORK_MANAGER_TEMPLATE_PATH, {}, QDBusConnection::SystemBus);
dbusTestRunner.startServices();
- QProcessEnvironment env(QProcessEnvironment::systemEnvironment());
- env.insert("SECRET_AGENT_DEBUG_PASSWORD", "1");
- secretAgent.setProcessEnvironment(env);
- secretAgent.setReadChannel(QProcess::StandardOutput);
- secretAgent.setProcessChannelMode(QProcess::ForwardedErrorChannel);
- secretAgent.start(SECRET_AGENT_BIN, QStringList() << "--print-address");
- secretAgent.waitForStarted();
- secretAgent.waitForReadyRead();
- agentBus = secretAgent.readAll().trimmed();
-
- agentInterface.reset(
- new OrgFreedesktopNetworkManagerSecretAgentInterface(agentBus,
- NM_DBUS_PATH_SECRET_AGENT, dbusTestRunner.systemConnection()));
-
-
notificationsInterface.reset(
new OrgFreedesktopDBusMockInterface(
"org.freedesktop.Notifications",
@@ -72,8 +67,11 @@ protected:
}
virtual ~TestSecretAgentCommon() {
- secretAgent.terminate();
- secretAgent.waitForFinished();
+ if (secretAgent.state() != QProcess::NotRunning)
+ {
+ secretAgent.terminate();
+ secretAgent.waitForFinished();
+ }
}
QVariantDictMap connection(const QString &keyManagement) {
@@ -111,6 +109,32 @@ protected:
return connection;
}
+ void setupSecretAgent (void) {
+ QSignalSpy notificationSpy(notificationsInterface.data(),
+ SIGNAL(MethodCalled(const QString &, const QVariantList &)));
+
+ QProcessEnvironment env(QProcessEnvironment::systemEnvironment());
+ env.insert("SECRET_AGENT_DEBUG_PASSWORD", "1");
+ secretAgent.setProcessEnvironment(env);
+ secretAgent.setReadChannel(QProcess::StandardOutput);
+ secretAgent.setProcessChannelMode(QProcess::ForwardedErrorChannel);
+ secretAgent.start(SECRET_AGENT_BIN, QStringList() << "--print-address");
+ secretAgent.waitForStarted();
+ secretAgent.waitForReadyRead();
+
+ agentBus = secretAgent.readAll().trimmed();
+
+ agentInterface.reset(
+ new OrgFreedesktopNetworkManagerSecretAgentInterface(agentBus,
+ NM_DBUS_PATH_SECRET_AGENT, dbusTestRunner.systemConnection()));
+
+ WAIT_FOR_SIGNALS(notificationSpy, 1);
+ {
+ const QVariantList &call(notificationSpy.at(0));
+ EXPECT_EQ(call.at(0), "GetServerInformation");
+ }
+ }
+
DBusTestRunner dbusTestRunner;
DBusMock dbusMock;
@@ -163,22 +187,21 @@ static void transform(QVariantList &list) {
}
TEST_P(TestSecretAgentGetSecrets, ProvidesPasswordForWpaPsk) {
+ setupSecretAgent();
+
+ QSignalSpy notificationSpy(notificationsInterface.data(),
+ SIGNAL(MethodCalled(const QString &, const QVariantList &)));
+
QDBusPendingReply<QVariantDictMap> reply(
agentInterface->GetSecrets(connection(GetParam().keyManagement),
QDBusObjectPath("/connection/foo"),
SecretAgent::NM_WIRELESS_SECURITY_SETTING_NAME, QStringList(),
5));
- QSignalSpy notificationSpy(notificationsInterface.data(),
- SIGNAL(MethodCalled(const QString &, const QVariantList &)));
- if (notificationSpy.empty())
- {
- ASSERT_TRUE(notificationSpy.wait());
- }
+ WAIT_FOR_SIGNALS(notificationSpy, 1);
- ASSERT_EQ(1, notificationSpy.size());
const QVariantList &call(notificationSpy.at(0));
- EXPECT_EQ("Notify", call.at(0).toString().toStdString());
+ EXPECT_EQ("Notify", call.at(0));
QVariantList args(call.at(1).toList());
transform(args);
@@ -254,6 +277,7 @@ class TestSecretAgent: public TestSecretAgentCommon, public Test {
};
TEST_F(TestSecretAgent, GetSecretsWithNone) {
+ setupSecretAgent();
QDBusPendingReply<QVariantDictMap> reply(
agentInterface->GetSecrets(
@@ -272,6 +296,8 @@ TEST_F(TestSecretAgent, GetSecretsWithNone) {
/* Tests that if we request secrets and then cancel the request
that we close the notification */
TEST_F(TestSecretAgent, CancelGetSecrets) {
+ setupSecretAgent();
+
QSignalSpy notificationSpy(notificationsInterface.data(), SIGNAL(MethodCalled(const QString &, const QVariantList &)));
agentInterface->GetSecrets(
@@ -280,23 +306,19 @@ TEST_F(TestSecretAgent, CancelGetSecrets) {
SecretAgent::NM_WIRELESS_SECURITY_SETTING_NAME, QStringList(),
5);
- notificationSpy.wait();
-
- ASSERT_EQ(1, notificationSpy.size());
- const QVariantList &call(notificationSpy.at(0));
- EXPECT_EQ("Notify", call.at(0).toString().toStdString());
+ WAIT_FOR_SIGNALS(notificationSpy, 1);
+ {
+ const QVariantList &call(notificationSpy.at(0));
+ EXPECT_EQ("Notify", call.at(0));
+ }
notificationSpy.clear();
agentInterface->CancelGetSecrets(QDBusObjectPath("/connection/foo"),
SecretAgent::NM_WIRELESS_SECURITY_SETTING_NAME);
- if (notificationSpy.empty())
- {
- ASSERT_TRUE(notificationSpy.wait());
- }
+ WAIT_FOR_SIGNALS(notificationSpy, 1);
- ASSERT_EQ(1, notificationSpy.size());
const QVariantList &closecall(notificationSpy.at(0));
EXPECT_EQ("CloseNotification", closecall.at(0).toString().toStdString());
}
@@ -304,6 +326,8 @@ TEST_F(TestSecretAgent, CancelGetSecrets) {
/* Ensures that if we request secrets twice we close the notification
for the first request */
TEST_F(TestSecretAgent, MultiSecrets) {
+ setupSecretAgent();
+
QSignalSpy notificationSpy(notificationsInterface.data(), SIGNAL(MethodCalled(const QString &, const QVariantList &)));
agentInterface->GetSecrets(
@@ -312,15 +336,12 @@ TEST_F(TestSecretAgent, MultiSecrets) {
SecretAgent::NM_WIRELESS_SECURITY_SETTING_NAME, QStringList(),
5);
- if (notificationSpy.empty())
+ WAIT_FOR_SIGNALS(notificationSpy, 1);
{
- ASSERT_TRUE(notificationSpy.wait());
+ const QVariantList &call(notificationSpy.at(0));
+ EXPECT_EQ("Notify", call.at(0));
}
- ASSERT_EQ(1, notificationSpy.size());
- const QVariantList &call(notificationSpy.at(0));
- EXPECT_EQ("Notify", call.at(0).toString().toStdString());
-
notificationSpy.clear();
agentInterface->GetSecrets(
@@ -329,14 +350,7 @@ TEST_F(TestSecretAgent, MultiSecrets) {
SecretAgent::NM_WIRELESS_SECURITY_SETTING_NAME, QStringList(),
5);
- if (notificationSpy.empty())
- {
- ASSERT_TRUE(notificationSpy.wait());
- }
- if (notificationSpy.size() == 1)
- {
- ASSERT_TRUE(notificationSpy.wait());
- }
+ WAIT_FOR_SIGNALS(notificationSpy, 2);
ASSERT_EQ(2, notificationSpy.size());
const QVariantList &closecall(notificationSpy.at(1));
@@ -347,11 +361,15 @@ TEST_F(TestSecretAgent, MultiSecrets) {
}
TEST_F(TestSecretAgent, SaveSecrets) {
+ setupSecretAgent();
+
agentInterface->SaveSecrets(QVariantDictMap(),
QDBusObjectPath("/connection/foo")).waitForFinished();
}
TEST_F(TestSecretAgent, DeleteSecrets) {
+ setupSecretAgent();
+
agentInterface->DeleteSecrets(QVariantDictMap(),
QDBusObjectPath("/connection/foo")).waitForFinished();
}
--
2.47.1
pkgs/desktops/lomiri/services/lomiri-indicator-network/default.nix
+4
View File
@@ -49,6 +49,10 @@ stdenv.mkDerivation (finalAttrs: {
"doc"
];
patches = [
./1001-test-secret-agent-Make-GetServerInformation-not-leak-into-tests.patch
];
postPatch = ''
# Override original prefixes
substituteInPlace data/CMakeLists.txt \