fix: Test transport layers back to using string, not json

rdkcentral · Jun 18, 2024 · e2e57bf · e2e57bf
1 parent 8ecb0df
commit e2e57bf
Show file tree

Hide file tree

Showing 9 changed files with 17 additions and 10 deletions.
diff --git a/languages/javascript/src/shared/Transport/MockTransport.mjs b/languages/javascript/src/shared/Transport/MockTransport.mjs
@@ -37,7 +37,7 @@ function send(message) {
   const json = JSON.parse(message)
   // handle bulk sends
   if (Array.isArray(json)) {
-    json.forEach(send)
+    json.forEach(json => send(JSON.stringify(json)))
     return
   }
 

diff --git a/test/TransportHarness.js b/test/TransportHarness.js
@@ -52,10 +52,10 @@
              id: id,
              result: result
          }
-         receiver && receiver(response)
+         receiver && receiver(JSON.stringify(response))
      },
      request: function(json) {
-        receiver && receiver(json)
+        receiver && receiver(JSON.stringify(json))
      }
  }
 

diff --git a/test/suite/properties-context.test.js b/test/suite/properties-context.test.js
@@ -31,7 +31,8 @@ let bothContextSentToEvent = false
 
 beforeAll( () => {
 
-    transport.onSend(json => {
+    transport.onSend(message => {
+        const json = JSON.parse(message)
         console.dir(json)
         if (json.method === 'Advanced.propertyWithContext') {
             if (json.params.appId === 'some-app') {

diff --git a/test/suite/properties.test.js b/test/suite/properties.test.js
@@ -28,7 +28,8 @@ let propertySetterWasTriggeredWithValue = false
 
 beforeAll( () => {
 
-    transport.onSend(json => {
+    transport.onSend(message => {
+        const json = JSON.parse(message)
         if (json.method === 'Simple.property') {
             transport.response(json.id, {
                 foo: "here's foo"

diff --git a/test/suite/provider-errors.test.js b/test/suite/provider-errors.test.js
@@ -44,7 +44,8 @@ beforeAll( () => {
         }
     }
 
-    transport.onSend(json => {
+    transport.onSend(message => {
+        const json = JSON.parse(message)
         if (json.method) {
             if (json.method === 'Provider.provideSimple') {
                 providerMethodNotificationRegistered = true

diff --git a/test/suite/provider.as-class.test.js b/test/suite/provider.as-class.test.js
@@ -35,7 +35,8 @@ beforeAll( () => {
         }
     }
 
-    transport.onSend(json => {
+    transport.onSend(message => {
+        const json = JSON.parse(message)
         if (json.method) {
             if (json.method === 'Provider.provideSimple') {
                 providerMethodNotificationRegistered = true

diff --git a/test/suite/provider.as-object.test.js b/test/suite/provider.as-object.test.js
@@ -36,7 +36,8 @@ beforeAll( () => {
     }
 
 
-    transport.onSend(json => {
+    transport.onSend(message => {
+        const json = JSON.parse(message)
         if (json.method) {
             if (json.method === 'Provider.provideSimple') {
                 providerMethodNotificationRegistered = true

diff --git a/test/suite/provider.with-multiple-methods.test.js b/test/suite/provider.with-multiple-methods.test.js
@@ -54,7 +54,8 @@ beforeAll( () => {
         }
     }
 
-    transport.onSend(json => {
+    transport.onSend(message => {
+        const json = JSON.parse(message)
         if (json.method) {
             if (json.method === 'Provider.provideMultipleMethods') {
                 providerMethodOneNotificationRegistered = true

diff --git a/test/suite/provider.with-no-response.test.js b/test/suite/provider.with-no-response.test.js
@@ -38,7 +38,8 @@ beforeAll( () => {
         }
     }
 
-    transport.onSend(json => {
+    transport.onSend(message => {
+        const json = JSON.parse(message)
         if (json.method) {
             if (json.method === 'Provider.provideNoResponseMethod') {
                 providerMethodNotificationRegistered = true