Fix tests (#120)

Author: Wulf

.github/workflows/Test.yml

@@ -14,7 +14,7 @@ jobs:
     strategy:
       matrix:
         os: [ubuntu-latest, macos-latest]
-        node-version: ["lts/*", "node"]
+        node-version: ["lts/*"]
 
     steps:
       - name: Checkout code
@@ -58,7 +58,7 @@ jobs:
     strategy:
       matrix:
         node-version: ["lts/*"]
-        arch: ["x64", "arm64"]
+        arch: ["x64"]
         include:
           - node-version: "22"
             arch: "x86"

src/Async/AddDictionaryWorker.cc

@@ -1,19 +1,20 @@
 #include <napi.h>
 #include <hunspell.hxx>
+#include <string>
 #include "Worker.cc"
 
 class AddDictionaryWorker : public Worker {
     public:
         AddDictionaryWorker(
             HunspellContext* context,
             Napi::Promise::Deferred d,
-            const char* dictionary)
-        : Worker(context, d), dictionary(dictionary) {}
+            std::string dictionary)
+        : Worker(context, d), dictionary(std::move(dictionary)) {}
 
     void Execute() {
         // Worker thread; don't use N-API here
         context->lockWrite();
-        context->instance->add_dic(dictionary);
+        context->instance->add_dic(dictionary.c_str());
         context->unlockWrite();
     }
 
@@ -24,5 +25,5 @@ class AddDictionaryWorker : public Worker {
     }
 
     private:
-        const char* dictionary;
+        std::string dictionary;
 };

src/Async/AddWithAffixWorker.cc

@@ -9,7 +9,7 @@ class AddWithAffixWorker : public Worker {
             Napi::Promise::Deferred d,
             std::string word,
             std::string example)
-        : Worker(context, d), word(word), example(example) {}
+        : Worker(context, d), word(std::move(word)), example(std::move(example)) {}
 
     void Execute() {
         // Worker thread; don't use N-API here

src/Async/AddWorker.cc

@@ -8,7 +8,7 @@ class AddWorker : public Worker {
             HunspellContext* context,
             Napi::Promise::Deferred d,
             std::string word)
-        : Worker(context, d), word(word) {}
+        : Worker(context, d), word(std::move(word)) {}
 
     void Execute() {
         // Worker thread; don't use N-API here

src/Async/AnalyzeWorker.cc

@@ -8,7 +8,7 @@ class AnalyzeWorker : public Worker {
             HunspellContext* context,
             Napi::Promise::Deferred d,
             std::string word)
-        : Worker(context, d), word(word) {}
+        : Worker(context, d), word(std::move(word)) {}
 
     void Execute() {
         // Worker thread; don't use N-API here

src/Async/GenerateWorker.cc

@@ -9,7 +9,7 @@ class GenerateWorker : public Worker {
             Napi::Promise::Deferred d,
             std::string word,
             std::string example)
-        : Worker(context, d), word(word), example(example) {}
+        : Worker(context, d), word(std::move(word)), example(std::move(example)) {}
 
     void Execute() {
         // Worker thread; don't use N-API here

src/Async/RemoveWorker.cc

@@ -8,7 +8,7 @@ class RemoveWorker : public Worker {
             HunspellContext* context,
             Napi::Promise::Deferred d,
             std::string word)
-        : Worker(context, d), word(word) {}
+        : Worker(context, d), word(std::move(word)) {}
 
     void Execute() {
         // Worker thread; don't use N-API here

src/Async/SpellWorker.cc

@@ -8,7 +8,7 @@ class SpellWorker : public Worker {
             HunspellContext* context,
             Napi::Promise::Deferred d,
             std::string word)
-        : Worker(context, d), word(word) {}
+        : Worker(context, d), word(std::move(word)) {}
 
     void Execute() {
         // Worker thread; don't use N-API here

src/Async/StemWorker.cc

@@ -8,7 +8,7 @@ class StemWorker : public Worker {
             HunspellContext* context,
             Napi::Promise::Deferred d,
             std::string word)
-        : Worker(context, d), word(word) {}
+        : Worker(context, d), word(std::move(word)) {}
 
     void Execute() {
         // Worker thread; don't use N-API here

src/Async/SuggestWorker.cc

@@ -8,7 +8,7 @@ class SuggestWorker : public Worker {
             HunspellContext* context,
             Napi::Promise::Deferred d,
             std::string word)
-        : Worker(context, d), word(word) {}
+        : Worker(context, d), word(std::move(word)) {}
 
     void Execute() {
         // Worker thread; don't use N-API here