fablo icon indicating copy to clipboard operation
fablo copied to clipboard

updated test-01-simple to test gateway client

Open iamkalio opened this issue 7 months ago • 7 comments

Addresses #542

iamkalio avatar May 05 '25 05:05 iamkalio

Hello @umegbewe @dzikowski Still in progress, but I would love your feedback.

iamkalio avatar May 05 '25 20:05 iamkalio

thanks @kalio007

the idea is fine, this is the script we want to update with a gateway test. However I have a few remarks:

  • please don't mute -e flag, it's there to prevent from ignoring errors
  • we should grep the output of the node command and check if it contains Put result: {"success":"OK"}
  • please also adjust the file structure to our conventions (spacing, location of env vars on the top, the style of printed messages). When you run the script locally, the output should look consistent

dzikowski avatar May 06 '25 10:05 dzikowski

Hello @dzikowski , I have updated the test script. I would appreciate your review.

iamkalio avatar May 09 '25 14:05 iamkalio

Hi @dzikowski , I have updated the env management. I would like to know if I should change a thing or two, thanks

iamkalio avatar May 09 '25 19:05 iamkalio

please make sure test are working. You can execute e2e-network/docker scripts also on local machine

dzikowski avatar May 10 '25 09:05 dzikowski

Hello @umegbewe @dzikowski , I've been running into an issue during testing:

EndorseError: 14 UNAVAILABLE: No connection established. Last error: Error: 800C480302000000:error:0A00010B:SSL routines:ssl3_get_record:wrong version number:../deps/openssl/openssl/ssl/record/ssl3_record.c:355: It seems related to a TLS mismatch, the test is trying to establish TLS connection with the server but failing. Could you please take a look when you're free?

iamkalio avatar May 10 '25 15:05 iamkalio

Hello @dzikowski, Please check this out when you're free, I'll appreciate your feedback.

iamkalio avatar May 13 '25 14:05 iamkalio

resolved in #587

dzikowski avatar Aug 13 '25 07:08 dzikowski