Fix missing validation bug in names rules

counterexamples/common/names/names-invalid-language.yaml

@@ -19,10 +19,13 @@ properties:
     rules:
       - language: etymology:wikidata
         value: Not allowed to have a colon
+        variant: common
       - language: etymology
         value: Too long
+        variant: alternate
       - language: pronunciation
         value: Too long
+        variant: official
   ext_expected_errors:
     - "additionalProperties 'zh_pinyin', 'en1', 'ja_rm' not allowed"
     - "[I#/properties/names/rules/0/language] [S#/$defs/propertyDefinitions/language/pattern] does not match pattern"

counterexamples/common/names/names-invalid-variant.yaml

@@ -0,0 +1,22 @@
+---
+id: names-invalid-variant
+type: Feature
+geometry:
+  type: LineString
+  coordinates: [[0, 0], [1, 1]]
+properties:
+  theme: transportation
+  type: segment
+  version: 1
+  subtype: road
+  class: primary
+  names:
+    primary: Bar
+    rules:
+      - value: I am missing the variant
+      - language: en-us
+        value: I have a fake/unsupported variant
+        variant: fake-variant
+  ext_expected_errors:
+    - "'/properties/names/rules/0': missing property 'variant'"
+    - "'/properties/names/rules/1/variant': value must be one of 'common', 'official', 'alternate', 'short'"

examples/common/names/names-lexically-valid-language-tag.yaml

@@ -21,5 +21,7 @@ properties:
     rules:
       - language: be-Latn
         value: Use this instead of be-tarask.
+        variant: common
       - language: ja-Latn
         value: Use this instead of ja_rm.
+        variant: alternate

schema/defs.yaml

@@ -185,7 +185,7 @@ description: Common schema definitions shared by all themes
             5. *("-" extension)
     nameRule:
       type: object
-      required: [value]
+      required: [variant, value]
       allOf:
         - { "$ref": "#/$defs/propertyContainers/geometricRangeScopeContainer" }
         - { "$ref": "#/$defs/propertyContainers/sideScopeContainer" }