write some tests, add some coverage

[drazisil]

No description provided.

[codecov]

Codecov Report

Merging #1112 (7048972) into main (b34aae7) will increase coverage by 27.70%.
The diff coverage is 33.65%.

Impacted Files	Coverage Δ
app.js	0.00% <0.00%> (ø)
packages/mcos-persona/index.js	28.29% <13.04%> (ø)
packages/mcos-login/index.js	28.32% <17.33%> (ø)
packages/mcos-lobby/index.js	28.32% <20.45%> (ø)
packages/mcos-shard/index.js	38.30% <20.65%> (ø)
packages/mcos-core/tcpConnection.js	32.95% <23.20%> (ø)
packages/mcos-database/index.js	29.29% <23.21%> (ø)
packages/mcos-core/listener-thread.js	43.03% <26.66%> (ø)
packages/mcos-core/nps-packet-manager.js	51.66% <26.66%> (ø)
packages/mcos-core/connection-mgr.js	41.75% <30.76%> (ø)
... and 37 more

[lgtm-com]

This pull request fixes 4 alerts when merging 118f5a9 into cd13960 - view on LGTM.com

fixed alerts:

• 4 for Unused variable, import, function or class

[lgtm-com]

This pull request fixes 10 alerts when merging e10fb8a into cd13960 - view on LGTM.com

fixed alerts:

• 10 for Unused variable, import, function or class

[lgtm-com]

This pull request fixes 24 alerts when merging fd074ae into cd13960 - view on LGTM.com

fixed alerts:

• 24 for Unused variable, import, function or class

[lgtm-com]

This pull request fixes 24 alerts when merging 2e44dfc into cd13960 - view on LGTM.com

fixed alerts:

• 24 for Unused variable, import, function or class

[lgtm-com]

This pull request fixes 24 alerts when merging fc20cb3 into cd13960 - view on LGTM.com

fixed alerts:

• 24 for Unused variable, import, function or class

[lgtm-com]

This pull request fixes 24 alerts when merging 05cdaf9 into cd13960 - view on LGTM.com

fixed alerts:

• 24 for Unused variable, import, function or class

[lgtm-com]

This pull request fixes 24 alerts when merging af57c24 into cd13960 - view on LGTM.com

fixed alerts:

• 24 for Unused variable, import, function or class

[lgtm-com]

This pull request fixes 24 alerts when merging c44c6ec into cd13960 - view on LGTM.com

fixed alerts:

• 24 for Unused variable, import, function or class

[lgtm-com]

This pull request fixes 24 alerts when merging e9ad414 into cd13960 - view on LGTM.com

fixed alerts:

• 24 for Unused variable, import, function or class

[lgtm-com]

This pull request fixes 24 alerts when merging 1e34a69 into cd13960 - view on LGTM.com

fixed alerts:

• 24 for Unused variable, import, function or class

[lgtm-com]

This pull request fixes 24 alerts when merging 13c4e80 into cd13960 - view on LGTM.com

fixed alerts:

• 24 for Unused variable, import, function or class

[lgtm-com]

This pull request fixes 24 alerts when merging 7444682 into b11cb10 - view on LGTM.com

fixed alerts:

• 24 for Unused variable, import, function or class

[lgtm-com]

This pull request fixes 24 alerts when merging 7048972 into b11cb10 - view on LGTM.com

fixed alerts:

• 24 for Unused variable, import, function or class