Moving around the chef rspec tests allows developers to
more easily discover and maintain them. Additionally, this
restructuring clears the path to use projectionist and more
clearly see if any test coverage is missing.
Related https://gitlab.com/gitlab-org/omnibus-gitlab/-/issues/6721
Signed-off-by: Balasankar "Balu" C <balasankar@gitlab.com>