search

AppiumTestDistribution / appium-reporter-plugin

Appium2 plugin to generate html report with screenshots.
Apache License 2.0
30 stars 12 forks source link

version 1.1.0-beta.01 breaks the javascript flow (1.0.0-beta.13 works fine) #135

Closed isaac-tribal closed 1 year ago

isaac-tribal commented 1 year ago

GET /getReport 2023-05-05 22:02:26:922 [HTTP] {} 2023-05-05 22:02:26:943 uncaughtException: Cannot read properties of undefined (reading 'map') TypeError: Cannot read properties of undefined (reading 'map') at Object. (/Users/isaactribal/.appium/node_modules/appium-reporter-plugin/lib/reporter.js:91:40) at Generator.next () at fulfilled (/Users/isaactribal/.appium/node_modules/appium-reporter-plugin/lib/reporter.js:5:58) at runMicrotasks () at processTicksAndRejections (node:internal/process/task_queues:96:5)

Attaching log from iOS:

appium-with-plugin-1.1.0-beta.01.log

Dileep17 commented 1 year ago

Hi @isaac-tribal version 1.1.0-beta.01 has a contract change and doesn't have backward compatibility.

Based on logs attached /setTestInfo is still using old bindings which is supported only upto 1.0.0-beta-x 

2023-05-05 22:02:26:906 [HTTP] --> POST /session/46a7491b-47d3-4606-9e79-3ca4b8f6ed1b/setTestInfo
2023-05-05 22:02:26:906 [HTTP] {"testName":"should display first screen when app opens","testStatus":"passed","error":""}

From version 1.1.x /setTestInfo expects sessionId also. Check latest contact @ https://github.com/AppiumTestDistribution/appium-reporter-plugin#settestinfo

I should have made it more clear in release note or readme.

Dileep17 commented 1 year ago

Please re-open if you still face issue after following new contract. Thanks!