Review of unit test for 'events'

[ricott1]

Review for: https://github.com/LiskHQ/lisk-sdk/blob/development/elements/lisk-chain/test/unit/event.spec.ts

• Invalid event name: https://github.com/LiskHQ/lisk-sdk/blob/development/elements/lisk-chain/test/unit/event.spec.ts#L25 name and module should be alphanumeric strings as specified in https://github.com/LiskHQ/lips/blob/main/proposals/lip-0065.md#module-name-and-data
• I would still add a test for invalid events, like too many topics or too large data size.

[mosmartin]

@ricott1, @shuse2 the requirements/unit tests requested in this ticket cannot be tested as there is no implementation in the event file being tested. There is no function that checks for the event name nor is there a function that checks for event validity (like too many topics or too large data size). When I was going through the tests, I noticed there are no -ve tests so I added some.