ably-java icon indicating copy to clipboard operation
ably-java copied to clipboard

Published 20 hours ago • verified publisher icon ably

Enable and fix tests in RealtimeMessageTest

Open qsdigor opened this issue 2 years ago • 3 comments

There are 15 of 20 tests that are ignored in RealtimeMessageTest class. This is way too many and ignoring tests leaves parts of code untested. As first step we should enable all ignored tests in this class. If there are failing tests we need to fix them and add new ones if necessary.

┆Issue is synchronized with this Jira Task by Unito

qsdigor avatar Nov 24 '22 12:11 qsdigor

➤ Automation for Jira commented:

The link to the corresponding Jira issue is https://ably.atlassian.net/browse/SDK-2976

sync-by-unito[bot] avatar Nov 24 '22 12:11 sync-by-unito[bot]

I have enabled ignored tests and fixed one of them. There are still 3 of 21 ignored tests that need to be solved. These tests depend on ably-common repository for loading encoding JSON files. It seems there is some logic to load ably-common repo as a local path which appears not to be working.

qsdigor avatar Nov 24 '22 13:11 qsdigor

All tests are fixed and enabled.

qsdigor avatar Nov 28 '22 07:11 qsdigor