fix: replace NaN values in deduped TA table

[joseph-sentry]

2 things:

• moving dedup from the test_analytics resolver to get_results
• changing dedup to fill_nan values with 0

[seer-by-sentry]

🚨 Sentry detected 10 potential issues in your recent changes 🚨

Lower risk findings

• 1% chance WARNING at Line 86, graphql_api/tests/test_test_analytics.py:
• 1% chance WARNING at Line 86, graphql_api/tests/test_test_analytics.py:
• 1% chance WARNING at Line 86, graphql_api/tests/test_test_analytics.py:
• 1% chance WARNING at Line 86, graphql_api/tests/test_test_analytics.py:
• 1% chance WARNING at Line 86, graphql_api/tests/test_test_analytics.py:
• 1% chance WARNING at Line 86, graphql_api/tests/test_test_analytics.py:
• 1% chance WARNING at Line 70, upload/views/test_results.py:
• 1% chance WARNING at Line 70, upload/views/test_results.py:
• 1% chance WARNING at Line 70, upload/views/test_results.py:
• 1% chance WARNING at Line 70, upload/views/test_results.py:

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

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 96.31%. Comparing base (a9bc72c) to head (9d146d2).
Report is 2 commits behind head on main.

✅ All tests successful. No failed tests found.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #1211   +/-   ##
=======================================
  Coverage   96.31%   96.31%           
=======================================
  Files         492      492           
  Lines       16810    16814    +4     
=======================================
+ Hits        16190    16194    +4     
  Misses        620      620           

Flag	Coverage Δ
unit	96.31% <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.

[ajay-sentry]

very helpful comment ;)

[ajay-sentry]

lgtm

[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!