Refactor of the dimm collector

[graepaul]

Summary

• Improve full memory dmidecode for linux.
• Adding a full decode of wmic for windows.
• Improved unit-tests

Here is what a dimm looks like

{
  "dimms": [
    {
      "size_bytes": 103079215104,
      "locator": "CPU0_A",
      "bank_locator": "_Node0_Channel0_Dimm0",
      "manufacturer": "Samsung",
      "part_number": "M321RYGA0PB0-CWMXJ",
      "serial_number": "1234-567889AB",
      "memory_type": "DDR5",
      "form_factor": "DIMM",
      "speed_mts": 5600,
      "configured_speed_mts": 5600,
      "rank": 2,
      "data_width_bits": 64,
      "total_width_bits": 80,
      "size": "96GB"
    },
...

Test plan

• pytest test/unit - Improved unit-tests which now runs against a large fixtures file validating functionality.
• pytest test/functional (if applicable) - N/A
• pre-commit run --all-files

Checklist

• Added/updated tests (or explained why not)
• Updated docs/README if behavior changed
• No secrets or credentials committed

[amd-ansonyim]

Lgtm, just one thing

[amd-ansonyim]

res is called but not used. Do we need to add that to the log_event? Right now I think end user will just see "Error checking dimms" with no further context

[graepaul]

Now it uses it see code snippet:

    def _log_cmd_error(self, res: CommandArtifact) -> None:
        """Log a failed memory query without aborting the rest of the collection.

        Args:
            res (CommandArtifact): result of the failed command.
        """
        self._log_event(
            category=EventCategory.OS,
            description="Error checking dimms",
            priority=EventPriority.ERROR,
            data=res.model_dump(include={"command", "stderr", "exit_code"}),
            console_log=True,
        )

[alexandraBara]

@graepaul updates look good, but utest needs to be updated

[graepaul]

I renamed the fixture name in the unit test and it passes now.