Use pytest's FixtureDef type

[tamird]

No description provided.

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 95.61%. Comparing base (9e254b4) to head (74a0383).

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #863      +/-   ##
==========================================
+ Coverage   95.59%   95.61%   +0.01%     
==========================================
  Files           2        2              
  Lines         477      479       +2     
  Branches       95       96       +1     
==========================================
+ Hits          456      458       +2     
  Misses         12       12              
  Partials        9        9              

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

[seifertm]

Thanks for the contribution.

I don't think this PR needs a changelog entry since the code changes should only affect developers, but not the users. Or am I missing something?

[webknjaz]

I don't think this PR needs a changelog entry since the code changes should only affect developers, but not the users. Or am I missing something?

In pytest itself, I added changelog categories for contributors/downstream folk intended for news/summaries for contributors/core devs. Perhaps, we need to do the same here. But in general, yes, it shouldn't need a change note for the end-users audience.

[seifertm]

I don't think this PR needs a changelog entry since the code changes should only affect developers, but not the users. Or am I missing something?

In pytest itself, I added changelog categories for contributors/downstream folk intended for news/summaries for contributors/core devs. Perhaps, we need to do the same here. But in general, yes, it shouldn't need a change note for the end-users audience.

Interesting idea. I'll have a look how this is done in pytest.

In the meantime, let's get the PR on the road.