search

eclipse-uprotocol / up-tck

uProtocol Test Compatibility Kit
Apache License 2.0
4 stars 16 forks source link

Port up-tck C++ socket transport to 1.5.8 API #60

Closed gregmedd closed 3 months ago

gregmedd commented 4 months ago

C++ transport implementations have fewer responsibilities after the 1.5.8 API update in up-cpp. The socket transport will need to be updated to conform to the new API.

debruce commented 4 months ago

I would like to work on this.

debruce commented 4 months ago

I have code compiling for all of this, but the callback part. I need to figure out how to use Dispatcher and python to test this part, before moving on to callback implementation.

debruce commented 4 months ago

Since the break, I got some instructions from Matthew or Agosh about how to run the dispatcher as a standalone script. I'm currently using that in testing sendImpl() from a standalone C++ test. The message goes out and comes back from dispatcher. This is enough to start using the callback data structure, which was already written before the break.

debruce commented 4 months ago

I've got this working. I've converted the PR to review status.

debruce commented 3 months ago

The prior request for review didn't have anything for wildcard testing, and didn't include 1.0.1-rc0 changes. That has been addressed. Furthermore, this PR has a (non gtest) test program exercising pub->sub, rpc client->server, and rpc server->client. Attempts to get a notification test working have been unsuccessful to date.