fix: support hexadecimal OpenFeature numeric variants

Author: marandaneto

.sampo/changesets/cranky-duchess-ilmatar.md

@@ -0,0 +1,5 @@
+---
+pypi/openfeature-provider-posthog: patch
+---
+
+Support hexadecimal numeric variants in the OpenFeature provider.

openfeature-provider/openfeature/contrib/provider/posthog/provider.py

@@ -8,6 +8,7 @@
 from __future__ import annotations
 
 import logging
+import re
 from typing import Any, Callable, Mapping, Optional, Sequence, TypeVar, Union
 
 from openfeature.evaluation_context import EvaluationContext
@@ -53,7 +54,9 @@ class PostHogProvider(AbstractProvider):
     Flag-type mapping (all via ``get_feature_flag_result``):
         * boolean -> ``enabled``
         * string  -> the multivariate ``variant`` key
-        * int/float -> the ``variant`` parsed to a number
+        * int/float -> the ``variant`` parsed to a number, including unsigned
+          hexadecimal integers such as ``0x10`` (returned as int or float,
+          respectively)
         * object  -> the flag's JSON ``payload``
 
     Args:
@@ -213,7 +216,10 @@ def _resolve_number(
                 f"Flag '{flag_key}' has no variant to parse as {ctor.__name__}."
             )
         try:
-            value = ctor(result.variant)
+            variant = result.variant
+            if re.fullmatch(r"0[xX][0-9a-fA-F]+", variant.strip()):
+                variant = str(int(variant, 16))
+            value = ctor(variant)
         except (TypeError, ValueError) as exc:
             raise TypeMismatchError(
                 f"Flag '{flag_key}' variant '{result.variant}' is not a valid "

openfeature-provider/tests/test_provider_unit.py

@@ -100,6 +100,35 @@ def test_number_variant_parse_failure(fake_client, resolver, variant):
         getattr(_provider(fake_client), resolver)("n", 0, EvaluationContext("u"))
 
 
+@pytest.mark.parametrize(
+    "resolver", ["resolve_integer_details", "resolve_float_details"]
+)
+@pytest.mark.parametrize(
+    ("variant", "expected"),
+    [("0x10", 16), ("0Xff", 255), (" 0xA0 ", 160), ("0x0", 0), ("042", 42)],
+)
+def test_hex_number_variant(fake_client, resolver, variant, expected):
+    fake_client.get_feature_flag_result.return_value = make_result(variant=variant)
+    details = getattr(_provider(fake_client), resolver)("n", 0, EvaluationContext("u"))
+    assert details.value == expected
+    assert type(details.value) is (
+        int if resolver == "resolve_integer_details" else float
+    )
+    assert details.variant == variant
+
+
+@pytest.mark.parametrize(
+    "resolver", ["resolve_integer_details", "resolve_float_details"]
+)
+@pytest.mark.parametrize(
+    "variant", ["0x", "0xgg", "0x1.5", "0x_10", "0x1_0", "-0x10", "+0x10"]
+)
+def test_invalid_hex_number_variant(fake_client, resolver, variant):
+    fake_client.get_feature_flag_result.return_value = make_result(variant=variant)
+    with pytest.raises(TypeMismatchError):
+        getattr(_provider(fake_client), resolver)("n", 0, EvaluationContext("u"))
+
+
 def test_object_payload(fake_client):
     fake_client.get_feature_flag_result.return_value = make_result(
         enabled=True, variant="v1", payload={"color": "blue"}