De-dot 'kibana.version' is /package JSON responses

This changes the response JSON from

```
  "conditions": {
    "kibana.version": "^7.15.0"
  },
```

to

```
  "conditions": {
    "kibana": {
      "version": "^7.15.0"
    }
  },
```

Closes #730

CHANGELOG.md

@@ -8,6 +8,8 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ### Breaking changes
 
+* Change format of responses to `/package` to make `{"constraint": {"kibana.version": "7.16.0"}}` be `{"constraint": {"kibana": {"version": "7.16.0"}}}`. [#733](https://github.com/elastic/package-registry/pull/733)
+
 ### Bugfixes
 
 ### Added

testdata/generated/package.json

@@ -15,7 +15,9 @@
     "azure"
   ],
   "conditions": {
-    "kibana.version": "~7.x.x"
+    "kibana": {
+      "version": "~7.x.x"
+    }
   },
   "screenshots": [
     {

testdata/generated/package/elasticsearch_privileges/1.0.0/index.json

@@ -14,7 +14,9 @@
     "custom"
   ],
   "conditions": {
-    "kibana.version": "\u003e=7.16.0"
+    "kibana": {
+      "version": "\u003e=7.16.0"
+    }
   },
   "assets": [
     "/package/elasticsearch_privileges/1.0.0/manifest.yml",

testdata/generated/package/example/0.0.2/index.json

@@ -14,7 +14,9 @@
     "web"
   ],
   "conditions": {
-    "kibana.version": "\u003e=6.0.0"
+    "kibana": {
+      "version": "\u003e=6.0.0"
+    }
   },
   "assets": [
     "/package/example/0.0.2/manifest.yml",

testdata/generated/package/example/1.0.0/index.json

@@ -15,7 +15,9 @@
     "azure"
   ],
   "conditions": {
-    "kibana.version": "~7.x.x"
+    "kibana": {
+      "version": "~7.x.x"
+    }
   },
   "screenshots": [
     {

testdata/generated/package/foo/1.0.0/index.json

@@ -14,7 +14,9 @@
     "custom"
   ],
   "conditions": {
-    "kibana.version": "\u003e=7.0.0"
+    "kibana": {
+      "version": "\u003e=7.0.0"
+    }
   },
   "assets": [
     "/package/foo/1.0.0/manifest.yml",

testdata/generated/package/hidden/1.0.0/index.json

@@ -14,7 +14,9 @@
     "custom"
   ],
   "conditions": {
-    "kibana.version": "\u003e=7.0.0"
+    "kibana": {
+      "version": "\u003e=7.0.0"
+    }
   },
   "assets": [
     "/package/hidden/1.0.0/manifest.yml",

testdata/generated/package/ilm_policy/1.0.0/index.json

@@ -14,7 +14,9 @@
     "custom"
   ],
   "conditions": {
-    "kibana.version": "\u003e=7.0.0"
+    "kibana": {
+      "version": "\u003e=7.0.0"
+    }
   },
   "assets": [
     "/package/ilmpolicy/1.0.0/manifest.yml",

testdata/generated/package/input_groups/0.0.1/index.json

@@ -24,7 +24,9 @@
     "cloud"
   ],
   "conditions": {
-    "kibana.version": "~7.x.x"
+    "kibana": {
+      "version": "~7.x.x"
+    }
   },
   "screenshots": [
     {

testdata/generated/package/input_level_templates/1.0.0/index.json

@@ -14,7 +14,9 @@
     "custom"
   ],
   "conditions": {
-    "kibana.version": "\u003e=7.11.0"
+    "kibana": {
+      "version": "\u003e=7.11.0"
+    }
   },
   "assets": [
     "/package/input_level_templates/1.0.0/manifest.yml",

testdata/generated/package/longdocs/1.0.4/index.json

@@ -22,7 +22,9 @@
     "web"
   ],
   "conditions": {
-    "kibana.version": "\u003e6.7.0"
+    "kibana": {
+      "version": "\u003e6.7.0"
+    }
   },
   "assets": [
     "/package/longdocs/1.0.4/manifest.yml",

testdata/generated/package/multiversion/1.0.3/index.json

@@ -22,7 +22,9 @@
     "web"
   ],
   "conditions": {
-    "kibana.version": "\u003e6.7.0"
+    "kibana": {
+      "version": "\u003e6.7.0"
+    }
   },
   "assets": [
     "/package/multiversion/1.0.3/changelog.yml",

testdata/generated/package/multiversion/1.0.4/index.json

@@ -22,7 +22,9 @@
     "web"
   ],
   "conditions": {
-    "kibana.version": "\u003e6.7.0"
+    "kibana": {
+      "version": "\u003e6.7.0"
+    }
   },
   "assets": [
     "/package/multiversion/1.0.4/changelog.yml",

testdata/generated/package/multiversion/1.1.0/index.json

@@ -22,7 +22,9 @@
     "web"
   ],
   "conditions": {
-    "kibana.version": "\u003e6.7.0"
+    "kibana": {
+      "version": "\u003e6.7.0"
+    }
   },
   "assets": [
     "/package/multiversion/1.1.0/changelog.yml",

testdata/generated/package/reference/1.0.0/index.json

@@ -23,7 +23,9 @@
     "web"
   ],
   "conditions": {
-    "kibana.version": "\u003e6.7.0  \u003c7.6.0"
+    "kibana": {
+      "version": "\u003e6.7.0  \u003c7.6.0"
+    }
   },
   "assets": [
     "/package/reference/1.0.0/changelog.yml",

testdata/generated/package/traces/1.0.0/index.json

@@ -14,7 +14,9 @@
     "monitoring"
   ],
   "conditions": {
-    "kibana.version": "~7.x.x"
+    "kibana": {
+      "version": "~7.x.x"
+    }
   },
   "assets": [
     "/package/fakeapm/1.0.0/manifest.yml",

util/package.go

@@ -107,8 +107,13 @@ type PolicyTemplate struct {
 }
 
 type Conditions struct {
-	KibanaVersion    string `config:"kibana.version,omitempty" json:"kibana.version,omitempty" yaml:"kibana.version,omitempty"`
-	kibanaConstraint *semver.Constraints
+	Kibana *KibanaConditions `config:"kibana,omitempty" json:"kibana,omitempty" yaml:"kibana,omitempty"`
+}
+
+// KibanaConditions defines conditions for Kibana (e.g. required version).
+type KibanaConditions struct {
+	Version    string `config:"version" json:"version" yaml:"version"`
+	constraint *semver.Constraints
 }
 
 type Version struct {
@@ -239,10 +244,10 @@ func NewPackage(basePath string) (*Package, error) {
 		}
 	}
 
-	if p.Conditions != nil && p.Conditions.KibanaVersion != "" {
-		p.Conditions.kibanaConstraint, err = semver.NewConstraint(p.Conditions.KibanaVersion)
+	if p.Conditions != nil && p.Conditions.Kibana != nil {
+		p.Conditions.Kibana.constraint, err = semver.NewConstraint(p.Conditions.Kibana.Version)
 		if err != nil {
-			return nil, errors.Wrapf(err, "invalid Kibana versions range: %s", p.Conditions.KibanaVersion)
+			return nil, errors.Wrapf(err, "invalid Kibana versions range: %s", p.Conditions.Kibana.Version)
 		}
 	}
 
@@ -299,13 +304,12 @@ func (p *Package) HasPolicyTemplateWithCategory(category string) bool {
 }
 
 func (p *Package) HasKibanaVersion(version *semver.Version) bool {
-
 	// If the version is not specified, it is for all versions
-	if p.Conditions == nil || version == nil || p.Conditions.kibanaConstraint == nil {
+	if p.Conditions == nil || p.Conditions.Kibana == nil || p.Conditions.Kibana.constraint == nil || version == nil {
 		return true
 	}
 
-	return p.Conditions.kibanaConstraint.Check(version)
+	return p.Conditions.Kibana.constraint.Check(version)
 }
 
 func (p *Package) IsNewerOrEqual(pp Package) bool {

util/package_test.go

@@ -38,9 +38,9 @@ var packageTests = []struct {
 		Package{
 			BasePackage: BasePackage{
 				Title: &title,
-			},
-			Conditions: &Conditions{
-				KibanaVersion: "bar",
+				Conditions: &Conditions{
+					Kibana: &KibanaConditions{Version: "bar"},
+				},
 			},
 		},
 		false,
@@ -51,9 +51,9 @@ var packageTests = []struct {
 			BasePackage: BasePackage{
 				Title:       &title,
 				Description: "my description",
-			},
-			Conditions: &Conditions{
-				KibanaVersion: ">=1.2.3 <=4.5.6",
+				Conditions: &Conditions{
+					Kibana: &KibanaConditions{Version: ">=1.2.3 <=4.5.6"},
+				},
 			},
 			Categories: []string{"custom", "foo"},
 		},
@@ -189,8 +189,12 @@ func TestHasKibanaVersion(t *testing.T) {
 			assert.NoError(t, err)
 
 			p := Package{
-				Conditions: &Conditions{
-					kibanaConstraint: constraint,
+				BasePackage: BasePackage{
+					Conditions: &Conditions{
+						Kibana: &KibanaConditions{
+							constraint: constraint,
+						},
+					},
 				},
 			}