Use get_flag_names more consistently

[Swatinem]

We are using the .flags property in a lot of places where we only ever wanted the .keys(). So lets use get_flag_names instead, which does exactly that.

[overwatch-beta]

✅ Sentry found no issues in your recent changes ✅

[codecov-notifications]

Codecov Report

All modified and coverable lines are covered by tests ✅

✅ All tests successful. No failed tests found.

📢 Thoughts on this report? Let us know!

🚀 New features to boost your workflow:

• ❄ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 95.86%. Comparing base (f77e387) to head (877fe5b).
Report is 1 commits behind head on main.

✅ All tests successful. No failed tests found.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #1191   +/-   ##
=======================================
  Coverage   95.86%   95.86%           
=======================================
  Files         493      493           
  Lines       16870    16873    +3     
=======================================
+ Hits        16173    16176    +3     
  Misses        697      697           

Flag	Coverage Δ
unit	95.86% <100.00%> (+<0.01%)	⬆️

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

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

[sentry]

Suspect Issues

This pull request was deployed and Sentry observed the following issues:

• ‼️ AttributeError: 'NoneType' object has no attribute 'get_flag_names' /api/v2/{service}/{owner_username}/repos/{repo_... View Issue
• ‼️ GraphQLError: 'NoneType' object has no attribute 'get_flag_names' /graphql/{service} View Issue

_{Did you find this useful? React with a 👍 or 👎}