Allow RegfValue.type to fall back to int

[JSCU-CNI]

This PR fixes edge cases where RegfValue.type cannot be translated to the RegistryValueType enum.

[Horofic]

Would you mind adding a test case for this behaviour in test_regutil?

Not sure, but should this not return REG_NONE in this case? Took a cursory peek here https://learn.microsoft.com/en-us/windows/win32/sysinfo/registry-value-types.

[Schamper]

Using _missing_ is probably nicer: https://docs.python.org/3/library/enum.html#enum.Enum._missing_

[JSCU-CNI]

This fix is not for catching REG_NONE (which is defined in the enum) but for arbitrary values not defined in winnt.h. Will update this to use _missing_.

[JSCU-CNI]

Unfortunately it seems _missing_() is not allowed to return anything not inside the enum: https://stackoverflow.com/a/59951984.

[Schamper]

You could go the cstruct route and insert new members as needed: https://github.com/fox-it/dissect.cstruct/blob/main/dissect/cstruct/types/enum.py#L183

[JSCU-CNI]

You could go the cstruct route and insert new members as needed

Added in 308fda1. Perhaps the Enum should be moved to the dissect.regf project?

[Schamper]

Added in 308fda1. Perhaps the Enum should be moved to the dissect.regf project?

Maybe, I'm fine with it in here for now since this enum serves more as an abstraction layer than a "parsing" enum.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 77.91%. Comparing base (67d34fe) to head (829e591).
Report is 1 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #956   +/-   ##
=======================================
  Coverage   77.90%   77.91%           
=======================================
  Files         324      324           
  Lines       27857    27865    +8     
=======================================
+ Hits        21702    21710    +8     
  Misses       6155     6155           

Flag	Coverage Δ
unittests	77.91% <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.