This PR updates testing and dependencies to the currently required level fro ioBroker adapters.
Node 18 is now required as minimum
node 16 has been removed from test-and-release workflow
@iobroker/testing and adapter-core have been updated
general dependencies have been uopdated
tier attribute has been added to io-pacakge.json
This PR updates main.js to satisfy linter standards
indentions and mixed space/tabs have been fixed to satisfy linter rules
a few single quotes have been changed to double quotes as this adapter has choosen to use double quote style
test scrips (using singel quotes) have been excluded from linter tests
Linter tests and general tests now pass
Integration tests still fail with adapter already running. I will investigate the reason
This PR fixes duplicate class problem at main.js
adapter is written using class mode (class Notificationforandroidtv extends utils.Adapter {...}). Thats ok and current style.
BUT adapter also created a second adapter class by utils.adapter. This method is an alternative method for older legacy code not using classes. The existence of two classes collided.
So the following changes have been performed:
//const adapter = utils.adapter("notificationforandroidtv"); -> use this has been deactivated. Feel free to completly remove the line
all usages of adapter.xxx have been replace by this.xxx to reference and use the adapter class created
ALL tests now pass
Please check functionality as the last fix changed your code and I do not have the hardware to verify functionality.
The following code sequences are suspect and possibly still erronous.
This PR updates testing and dependencies to the currently required level fro ioBroker adapters.
This PR updates main.js to satisfy linter standards
Linter tests and general tests now pass Integration tests still fail with adapter already running. I will investigate the reason
This PR fixes duplicate class problem at main.js
class Notificationforandroidtv extends utils.Adapter {...}
). Thats ok and current style.So the following changes have been performed:
//const adapter = utils.adapter("notificationforandroidtv"); -> use this
has been deactivated. Feel free to completly remove the lineALL tests now pass Please check functionality as the last fix changed your code and I do not have the hardware to verify functionality.
The following code sequences are suspect and possibly still erronous.
Line 576::
event is an array (slice retruns an array) which should not be compared with a string.
Line 666:
is for sure erronous
Please check PR and merge.