rfb: add unittests for nom7 parsers

[hsadia538]

• I have read the contributing guide lines at https://redmine.openinfosecfoundation.org/projects/suricata/wiki/Contributing
• I have signed the Open Information Security Foundation contribution agreement at https://suricata.io/about/contribution-agreement/
• I have updated the user guide (in doc/userguide/) to reflect the changes made (if applicable)

Link to redmine ticket:

Describe changes:

• Add unit tests for rfb protocol

[hsadia538]

Currently, the unit tests are written for the following parse functions:

More parse functions can be incorporated but some of them would be testing the same value e.g parse_security_type_selection , parse_server_security_type , parse_supported_security_types are quite similar as all of them return the security type selected or asked (the difference is only in context).

The packet buffers I have used are actually coming from an rfb pcap that I found in suricata verify. Should I add the reference somewhere in the file?

Apart from this, the main intention of this PR is to gather feedback on the approach and whether should I try doing the same for other tests that I will write, etc (including formatting, readability, and other factors that matter)

[victorjulien]

what is this comment?

[hsadia538]

Did not mean to commit these comments. Will remove them now( I added this for parsers whose unit tests are there)

[victorjulien]

git tip: git add -p <path to file> to manually select which changes should be staged for commit

[victorjulien]

please split formatting changes from the test additions into separate commits

[hsadia538]

Ok.Will address in next PR

[hsadia538]

Followed by #8276