Merge pull request #228 from platformsh/cxtra-146-the-validate-command-does-not-work-with-composable-image

Add support for composable images when validating configs

Author: akalipetis

validator/schema/platformsh.application.json

@@ -888,11 +888,44 @@
         "type": "string"
       },
       "title": "Resolve additional IPs to domain names."
+    },
+    "stack": {
+      "oneOf": [
+        {
+          "type": "string"
+        },
+        {
+          "type": "object"
+        },
+        {
+          "type": "array",
+          "items": {
+            "oneOf": [
+              {
+                "type": "string"
+              },
+              {
+                "type": "object"
+              }
+            ]
+          }
+        }
+      ]
     }
   },
-  "required": [
-    "name",
-    "type"
+  "oneOf": [
+    {
+      "required": [
+        "name",
+        "type"
+      ]
+    },
+    {
+      "required": [
+        "name",
+        "stack"
+      ]
+    }
   ],
   "additionalProperties": false
 }

validator/schema/upsun.json

@@ -875,10 +875,42 @@
               "type": "string"
             },
             "title": "Resolve additional IPs to domain names."
+          },
+          "stack": {
+            "oneOf": [
+              {
+                "type": "string"
+              },
+              {
+                "type": "object"
+              },
+              {
+                "type": "array",
+                "items": {
+                  "oneOf": [
+                    {
+                      "type": "string"
+                    },
+                    {
+                      "type": "object"
+                    }
+                  ]
+                }
+              }
+            ]
           }
         },
-        "required": [
-          "type"
+        "oneOf": [
+          {
+            "required": [
+              "type"
+            ]
+          },
+          {
+            "required": [
+              "stack"
+            ]
+          }
         ],
         "additionalProperties": false
       }

validator/validator_test.go

@@ -127,6 +127,73 @@ services:
 			},
 			wantErr: false,
 		},
+		{
+			name: "stack",
+			args: args{
+				path: fstest.MapFS{
+					".upsun/config.yaml": &fstest.MapFile{
+						Data: []byte(`
+applications:
+  app1:
+    stack:
+      - "[email protected]":
+        extensions:
+          - apcu
+          - sodium
+          - xsl
+          - pdo_sqlite
+      - "nodejs@20"
+      - "[email protected]"
+`,
+						),
+					},
+					".upsun/services.yaml": &fstest.MapFile{
+						Data: []byte(`
+services:
+  redis:
+    type: redis:6.2
+    size: AUTO
+`,
+						),
+					},
+				},
+			},
+			wantErr: false,
+		},
+		{
+			name: "stack-and-type",
+			args: args{
+				path: fstest.MapFS{
+					".upsun/config.yaml": &fstest.MapFile{
+						Data: []byte(`
+applications:
+  app1:
+    type: "python:3.11"
+    stack:
+      - "[email protected]":
+        extensions:
+          - apcu
+          - sodium
+          - xsl
+          - pdo_sqlite
+      - "nodejs@20"
+      - "[email protected]"
+`,
+						),
+					},
+					".upsun/services.yaml": &fstest.MapFile{
+						Data: []byte(`
+services:
+  redis:
+    type: redis:6.2
+    size: AUTO
+`,
+						),
+					},
+				},
+			},
+			wantErr: true,
+		},
 	}
 	for _, tt := range tests {
 		t.Run(tt.name, func(t *testing.T) {