fix: allow read-only "auth_type" attribute on credential API response

[infinitewarp]

This change supports quipucords/quipucords#2735

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 78.33%. Comparing base (ae26f6b) to head (56a1ef4).

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #540   +/-   ##
=======================================
  Coverage   78.33%   78.33%           
=======================================
  Files           7        7           
  Lines         683      683           
=======================================
  Hits          535      535           
  Misses        148      148           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[infinitewarp]

Standalone test run with updated quipucords image and camayoc:
https://stage-jenkins-csb-smqe.apps.ocp-c1.prod.psi.redhat.com/view/Discovery/job/discovery-standalone/198/

[mirekdlugosz]

Would it make sense to include actual expected value and make assertions stronger this way?

Most of the tests pass password or ssh_keyfile, so they do know what auth_type value they expect to see in response.

[infinitewarp]

I updated the regex pattern and surrounding logic to match on the specified data["auth_type"] if provided else anything from the current list of valid auth_type values. Here is another successful standalone CI run after pushing those changes:

https://stage-jenkins-csb-smqe.apps.ocp-c1.prod.psi.redhat.com/view/Discovery/job/discovery-standalone/199/

[mirekdlugosz]

That default is more than I asked for 😃 . Thanks!