Replace Minitest with RSpec: bigbluebutton_api_controller specs (1)

[scouillard]

Description

• Fix RSpec configs
• Remove redis_helper
• Add bbb_api_controller specs
• Add Server Factory
• Add Meeting Factory

Design considerations

• Reverse the logic for checksum check: Before, we had to disable the checksum check in (almost) every test. But, considering most of the tests are not testing the checksum specifically, the checksum check is now disabled by default for all the specs and has to be enabled when testing the checksum specifically.
• In #create, a voiceBridge attribute is provided in every #create request to bypass the voiceBridge generator. This is because the stubbed request will never match the outside request as a voiceBridge generated and added to the uri by Scalelite.
• Removed the redis_helper file as we will be using fakeredis and we don't need to configure a redis database for the test environment

Notes

• The order of the parameters in a URL query string should not affect the functioning of the API request. For example, in #create, override_create_params adds the new params at the end of the uri, and default_create_params adds them at the beginning of the uri. Not following the order will return a different checksum value.

[scouillard]

Upon reviewing, I support using fakeredis for our testing environment

fakeredis ensures that the tests are isolated from each other. Hence, redis_helper was removed as it seems it was used strictly to configure a redis database for the test environment.

However, there might be complex Redis usage implementation in the future that makes fakeredis not suitable.

Everything seems to be working as it should on our testing environment for now.

[sonarcloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
No Duplication information