fleet icon indicating copy to clipboard operation
fleet copied to clipboard

Software SS: add CLI support for `self_service`

Open mna opened this issue 1 year ago • 1 comments

#18834 (CLI part)

Checklist for submitter

  • [x] Changes file added for user-visible changes in changes/, orbit/changes/ or ee/fleetd-chrome/changes. See Changes files for more information.
  • [x] Input data is properly validated, SELECT * is avoided, SQL injection is prevented (using placeholders for values in statements)
  • [x] Added/updated tests
  • [x] Manual QA for all new/changed functionality

mna avatar May 22 '24 16:05 mna

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Please upload report for BASE (feat-software-self-service@87c4deb). Learn more about missing BASE report.

:exclamation: Current head 06456bf differs from pull request most recent head 0aa8cd6

Please upload reports for the commit 0aa8cd6 to get more accurate results.

Additional details and impacted files
@@                      Coverage Diff                      @@
##             feat-software-self-service   #19205   +/-   ##
=============================================================
  Coverage                              ?   66.59%           
=============================================================
  Files                                 ?      900           
  Lines                                 ?   111637           
  Branches                              ?        0           
=============================================================
  Hits                                  ?    74348           
  Misses                                ?    31256           
  Partials                              ?     6033           
Flag Coverage Δ
backend 66.59% <100.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

codecov[bot] avatar May 22 '24 19:05 codecov[bot]

@roperzh I also thought it was going to be more involved tbh!

mna avatar May 27 '24 14:05 mna