fixed issue #6983

[aman99dex]

Rewrote TestRecordDroppedAttributes to properly test limit-based drops separately from duplicate key drops
Updated TestApplyAttrLimitsDeduplication expectations to not count dedup drops
All tests pass.

[linux-foundation-easycla]

The committers listed above are authorized under a signed CLA.

• ✅ login: aman99dex / name: Aman Gupta (378d889)

[MrAlias]

@aman99dex please sign the CLA so this contribution can be considered.

[aman99dex]

Hey I think I fixed the cla part by signing the agreement

[flc1125]

Hey I think I fixed the cla part by signing the agreement

If the signing is successful, #7737 (comment) will be automatically updated to the correct state.

Please ensure that the information you sign with is the same as the information of all contributors in your commit logs.

[aman99dex]

is there anything else i need to do after this?

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 86.2%. Comparing base (655c08a) to head (7f9faf3).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files

@@          Coverage Diff          @@
##            main   #7737   +/-   ##
=====================================
  Coverage   86.2%   86.2%           
=====================================
  Files        302     302           
  Lines      21991   21996    +5     
=====================================
+ Hits       18968   18974    +6     
  Misses      2642    2642           
+ Partials     381     380    -1     

Files with missing lines	Coverage Δ
sdk/log/record.go	100.0% <100.0%> (ø)

... and 2 files with indirect coverage changes

🚀 New features to boost your workflow:

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

[flc1125]

Duplicate with #7662

[pellared]

@aman99dex, can you please review #7662 which was created and reviewed prior this PR?