Update tests for lambda discard parameters

[sharwell]

Only changes tests.

Closes #3977

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 97.45%. Comparing base (ff14d11) to head (55477d9).
⚠️ Report is 19 commits behind head on master.

Additional details and impacted files

@@           Coverage Diff           @@
##           master    #4056   +/-   ##
=======================================
  Coverage   97.45%   97.45%           
=======================================
  Files         989      991    +2     
  Lines      115570   115626   +56     
  Branches     3423     3423           
=======================================
+ Hits       112632   112688   +56     
  Misses       1943     1943           
  Partials      995      995           

🚀 New features to boost your workflow:

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

[bjornhellander]

Irrelevant test, since "handler" is the only variable here?

[sharwell]

It's a verification that _ is still ignored when it's treated as a true discard by the compiler. I was torn on whether to keep this test or not considering SA1313 is the one for parameters.

[bjornhellander]

My assumption was that this diagnostic would only analyze "handler", not the discards, but that might be wrong

[sharwell]

That's exactly the correct assumption. The question is, considering we can't cover all possible scenarios in all possible cases, where do we draw the line on tests to include for "unrelated items"? This one seems pretty close to the not-worth-it line, but ... 🤷