Apply review suggestions

Author: eemeli

fluent.runtime/fluent/runtime/resolver.py

@@ -398,7 +398,7 @@ def __call__(self, env: ResolverEnvironment) -> Any:
 
 class NamedArgument(FTL.NamedArgument, BaseResolver):
     name: Identifier
-    value: Union[NumberLiteral, StringLiteral]
+    value: Union[NumberLiteral, StringLiteral, VariableReference]
 
 
 InlineExpression = Union[

fluent.runtime/tests/format/test_parameterized_terms.py

@@ -73,6 +73,13 @@ def test_variable_named_arg(self, bundle):
         assert val == "a thing"
         assert errs == []
 
+    def test_missing_variable_named_arg(self, bundle):
+        val, errs = bundle.format_pattern(
+            bundle.get_message("thing-variable-arg").value, {}
+        )
+        assert val == "the thing"
+        assert errs == [FluentReferenceError('Unknown external: art')]
+
     def test_no_implicit_access_to_external_args(self, bundle):
         # The '-thing' term should not get passed article="indefinite"
         val, errs = bundle.format_pattern(