Dmi resource sane_product to avoid in duplication (bugfix)

[Hook25]

Description

In many resource expression there is a duplicated array that is periodically updated to include new items as laptop like or dektop like. This adds a new key to the resource and uses that instead so that updates are way easier to maintain.

Resolved issues

N/A

Documentation

Added an explanation docstring to say why this is done

Tests

N/A

Please while reviewing this be very carefult that I actually succesfully moved desktops to non-portal and laptops to portable

[codecov]

Codecov Report

❌ Patch coverage is 70.00000% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 51.94%. Comparing base (881c0f3) to head (edacac1).
⚠️ Report is 107 commits behind head on main.

Files with missing lines	Patch %	Lines
providers/resource/bin/dmi_resource.py	70.00%	3 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #2046      +/-   ##
==========================================
+ Coverage   51.55%   51.94%   +0.39%     
==========================================
  Files         385      388       +3     
  Lines       41439    41766     +327     
  Branches     7697     7754      +57     
==========================================
+ Hits        21363    21697     +334     
+ Misses      19315    19300      -15     
- Partials      761      769       +8     

Flag	Coverage Δ
provider-base	28.02% <ø> (+1.47%)	⬆️
provider-resource	39.38% <70.00%> (+1.18%)	⬆️

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.

🚀 New features to boost your workflow:

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

[fernando79513]

Thanks for the change.
I checked in my laptop and seems to be detected correctly.
The change is simple enough that I don't think further on testing on real devices is required.
Nevertheless, I think you should add some unit tests for this parsing using the previous fields:
'Notebook','Laptop','Portable','All In One','All-In-One','AIO','Convertible', 'Tablet', 'Detachable' -> "portable"
'Desktop','Low Profile Desktop','Tower','Mini Tower','Space-saving', 'Mini PC' -> not-portable

[fernando79513]

LGTM +1!