mop icon indicating copy to clipboard operation
mop copied to clipboard

integration tests reproducing MoP proxy issue

Open tsturzl opened this issue 1 year ago • 1 comments

Master Issue: #1212

Motivation

In the issue regarding this issue I was asked to reproduce the issue with an integration tests. This PR provides an integration test which reproduces the issue reliably. Obviously tests will not pass as this provides a test case which reproduces a known issue.

Modifications

Added a test to reproduce MoP proxy keep alive issue as mentioned in the linked github issue.

Verifying this change

  • [ ] Make sure that the change passes the CI checks.

NOTE: CI checks will not pass, as this only provides the reproduction case not the actual fix for the issue.

This change added tests and can be verified as follows:

  • An integration tests to create a Java Paho MQTT client connection to the MoP proxy to reproduce a bug with keep alive.

Documentation

Check the box below.

Need to update docs?

  • [ ] doc-required

    (If you need help on updating docs, create a doc issue)

  • [x] no-need-doc

    (Please explain why)

  • [ ] doc

    (If this PR contains doc changes)

tsturzl avatar Mar 13 '24 18:03 tsturzl

@tsturzl:Thanks for your contribution. For this PR, do we need to update docs? (The PR template contains info about doc, which helps others know more about the changes. Can you provide doc-related info in this and future PR descriptions? Thanks)

github-actions[bot] avatar Mar 13 '24 18:03 github-actions[bot]

Thanks @tsturzl , I'm checking

Technoboy- avatar Mar 14 '24 23:03 Technoboy-

Hi @tsturzl Thanks for find this issue. Could you please help move the test to this package? image

And then fix this issue by image

Then I wil merge this fix

Technoboy- avatar Mar 15 '24 01:03 Technoboy-

Fixed by #1293

Technoboy- avatar Mar 18 '24 09:03 Technoboy-