Fix missing element key for map values brick

[twschiller]

What does this PR do?

• Fixes missing element key in brick outline for map values brick
• Also fixes the variable being missing in the variable popover

Demo

Future Work

• We'll probably want to refactor getVariableKeyForSubPipeline to use the brick definition instead of defining in a separate method. Classic https://en.wikipedia.org/wiki/Expression_problem

Checklist

• Add tests
• New files added to src/tsconfig.strictNullChecks.json (if possible): N/A
• Designate a primary reviewer: @BLoe

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (bcebc58) 71.10% compared to head (956b6f6) 71.10%.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #7118      +/-   ##
==========================================
- Coverage   71.10%   71.10%   -0.01%     
==========================================
  Files        1209     1209              
  Lines       37503    37505       +2     
  Branches     7068     7068              
==========================================
  Hits        26668    26668              
- Misses      10835    10837       +2     

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

[github-actions]

No loom links were found in the first post. Please add one there if you'd like to it to appear on Slack.

Do not edit this comment manually.