Add tests to improve coverage

.github/workflows/linux-x64-build-and-test.yml

@@ -76,21 +76,8 @@ jobs:
         npm run test-idl
         npm run clean
 
-    - name: Coveralls Parallel
+    - name: Coveralls
       if: ${{ matrix.ros_distribution == 'rolling' && matrix['node-version'] == '24.X' }}
       uses: coverallsapp/github-action@v2
       with:
         github-token: ${{ secrets.GITHUB_TOKEN }}
-        flag-name: run-${{ inputs.trigger_type }}-${{ matrix.node-version }}-${{ matrix.architecture }}
-        parallel: true
-
-  finish:
-      needs: build
-      if: always()
-      runs-on: ubuntu-latest
-      steps:
-      - name: Coveralls Finished
-        uses: coverallsapp/github-action@v2
-        with:
-          github-token: ${{ secrets.GITHUB_TOKEN }}
-          parallel-finished: true

lib/message_serialization.js

@@ -78,7 +78,7 @@ function toPlainArrays(obj) {
  * @returns {*} The JSON-safe converted object
  */
 function toJSONSafe(obj) {
-  if (obj === null || obj === undefined) {
+  if (obj === null) {
     return obj;
   }
 

lib/native_loader.js

@@ -175,4 +175,14 @@ function loadNativeAddon() {
   }
 }
 
-module.exports = loadNativeAddon();
+const addon = loadNativeAddon();
+
+// Export internal functions for testing purposes
+if (process.env.NODE_ENV === 'test') {
+  addon.TestHelpers = {
+    customFallbackLoader,
+    loadNativeAddon,
+  };
+}
+
+module.exports = addon;

lib/time_source.js

@@ -102,7 +102,7 @@ class TimeSource {
    * @return {undefined}
    */
   attachNode(node) {
-    if ((!node) instanceof rclnodejs.ShadowNode) {
+    if (!(node instanceof rclnodejs.ShadowNode)) {
       throw new TypeValidationError('node', node, 'Node', {
         entityType: 'time source',
       });