chore(linux): Fix bug and update documentation

keymanapp · Sep 20, 2023 · 704ba55 · 704ba55
1 parent e6017e8
commit 704ba55
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 2 deletions.
diff --git a/linux/ibus-keyman/src/KeymanSystemServiceClient.cpp b/linux/ibus-keyman/src/KeymanSystemServiceClient.cpp
@@ -78,7 +78,7 @@ gint32 KeymanSystemServiceClient::GetCapsLockIndicator() {
     KEYMAN_INTERFACE_NAME, "GetCapsLockIndicator", error, &msg, "");
   if (result < 0) {
     g_error("%s: Failed to call method GetCapsLockIndicator: %s. %s. %s.",
-      __FUNCTION__, strerror(-result), error->name, error->message);
+      __FUNCTION__, strerror(-result), error ? error->name : "-", error ? error->message : "-");
     return -1;
   }
 

diff --git a/linux/ibus-keyman/tests/KmDbusTestServer.cpp b/linux/ibus-keyman/tests/KmDbusTestServer.cpp
@@ -139,6 +139,11 @@ void KmDbusTestServer::Loop()
 
 int
 main(int argc, char *argv[]) {
+  if (!g_file_test(KEYMAN_TEST_SERVICE_PATH, G_FILE_TEST_IS_DIR)) {
+    std::cerr << "ERROR: Directory " << KEYMAN_TEST_SERVICE_PATH << " doesn't exist! Exiting." << std::endl;
+    return 1;
+  }
+
   KmDbusTestServer testServer;
   testServer.Loop();
 

diff --git a/linux/ibus-keyman/tests/README.md b/linux/ibus-keyman/tests/README.md
@@ -6,7 +6,7 @@ are stored in `~/.config/glib-2.0/settings/keyfile`.
 
 ## Running tests
 
-The tests get run as part of building `ibus-keyman`, more specifically when running `make check`.
+The tests get run as part of building `ibus-keyman`, more specifically when running `build.sh test`.
 
 ### Run all tests