Fix leaks in test introduced by db6d94f82a0f860131392aa483f4d1e21dc74…

[normanmaurer]

…249.

Motivation:

db6d94f added some testcase which did have a buffer leak and so failed the CI.

Modifications:

Fix leak by proper testing of encoder and decoder using EmbeddedChannel.

Result:

CI is happy again.

[normanmaurer]

@slandelle FYI... this fixes the leaks introduced by your changes.

[slandelle]

Thanks!

[normanmaurer]

@slandelle hmmm ... now the test suite fails because the autobahntestsuite thinks what we do is not correct:

[ERROR] Failed to execute goal me.normanmaurer.maven.autobahntestsuite:autobahntestsuite-maven-plugin:0.1.4:fuzzingclient (default) on project netty-testsuite-autobahn: 
[ERROR] Failed test cases:
[ERROR] 	[7.9.8] behavior: FAILED, behaviorClose: WRONG_CODE, duration: 2ms, remoteCloseCode: 1014, reportFile: target/autobahntestsuite-reports/autobahntestsuite_maven_plugin_case_7_9_8.json
[ERROR] 	[7.9.6] behavior: FAILED, behaviorClose: WRONG_CODE, duration: 2ms, remoteCloseCode: 1012, reportFile: target/autobahntestsuite-reports/autobahntestsuite_maven_plugin_case_7_9_6.json
[ERROR] 	[7.9.7] behavior: FAILED, behaviorClose: WRONG_CODE, duration: 2ms, remoteCloseCode: 1013, reportFile: target/autobahntestsuite-reports/autobahntestsuite_maven_plugin_case_7_9_7.json

[normanmaurer]

Let me revert your commit for now and we can investigate in your PR then (Please incorporate my test changes in the PR).

[slandelle]

@normanmaurer It looks like autobahn follows RFC6455 strictly and wasn't updated to latest IANA code. Do you want me to reach out to them?

[slandelle]

Can be closed (in favor of #8664)

[normanmaurer]

@slandelle yep why not ... thanks!