quipucords / camayoc

Test automation framework that facilitates functional testing of quipucords.
https://camayoc.readthedocs.io/
GNU General Public License v3.0
5 stars 4 forks source link

fix: DROP ALL CHECKS FOR NOW 'optional' MESSAGES #398

Closed bruno-fs closed 1 year ago

bruno-fs commented 1 year ago

With the changes proposed here, all CLI tests will pass.

Also requires changes proposed on quipucords/qpc#230 and quipucords/quipucords#2331

NOTE: we should choose between this, #394 or #397

codecov[bot] commented 1 year ago

Codecov Report

Merging #398 (a7b8f56) into master (7396c68) will not change coverage. The diff coverage is n/a.

:exclamation: Current head a7b8f56 differs from pull request most recent head 388049b. Consider uploading reports for the commit 388049b to get more accurate results

@@           Coverage Diff           @@
##           master     #398   +/-   ##
=======================================
  Coverage   80.73%   80.73%           
=======================================
  Files           5        5           
  Lines         244      244           
=======================================
  Hits          197      197           
  Misses         47       47           

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

bruno-fs commented 1 year ago

closing for now in favor to #397

let's not go crazy removing asserts during code freeze 😅 🥶