ManageIQ / manageiq

ManageIQ Open-Source Management Platform
https://manageiq.org
Apache License 2.0
1.35k stars 900 forks source link

Move security test suite code out of rake and into an .rb file #23122

Closed Fryguy closed 4 months ago

Fryguy commented 4 months ago

Straight refactoring. Follow up to #23098

@jrafanie Please review.

miq-bot commented 4 months ago

Some comments on commit https://github.com/Fryguy/manageiq/commit/aa1d804e6e8b6752eb821e4b2bd3bced7b2ffdd2

lib/tasks/test_security_helper.rb

miq-bot commented 4 months ago

Checked commit https://github.com/Fryguy/manageiq/commit/aa1d804e6e8b6752eb821e4b2bd3bced7b2ffdd2 with ruby 3.1.5, rubocop 1.56.3, haml-lint 0.51.0, and yamllint 2 files checked, 17 offenses detected

lib/tasks/test_security.rake

lib/tasks/test_security_helper.rb

jrafanie commented 4 months ago

the cancelled 3.0 vmdb didn't re-run when I kicked the failed tests... restarting it

jrafanie commented 4 months ago

Merging, whitesource common failure is back: Oops! An error occurred while running the Security Check.The Scanner was unable to connect to the repository or branch and clone it.