Check if application properties and properties.env are not specified

pkg/corerp/api/v20231001preview/application_conversion.go

@@ -26,6 +26,15 @@ import (
 // ConvertTo converts from the versioned Application resource to version-agnostic datamodel.
 func (src *ApplicationResource) ConvertTo() (v1.DataModelInterface, error) {
 	// Note: SystemData conversion isn't required since this property comes ARM and datastore.
+
+	if src.Properties == nil {
+		return nil, &v1.ErrModelConversion{PropertyName: "$.properties", ValidValue: "not nil"}
+	}
+
+	if src.Properties.Environment == nil {
+		return nil, &v1.ErrModelConversion{PropertyName: "$.properties.environment", ValidValue: "not nil"}
+	}
+
 	converted := &datamodel.Application{
 		BaseResource: v1.BaseResource{
 			TrackedResource: v1.TrackedResource{

pkg/corerp/api/v20231001preview/application_conversion_test.go

@@ -48,6 +48,16 @@ func TestApplicationConvertVersionedToDataModel(t *testing.T) {
 			err:      nil,
 			emptyExt: true,
 		},
+		{
+			filename: "applicationresource-no-properties.json",
+			err:      &v1.ErrModelConversion{PropertyName: "$.properties", ValidValue: "not nil"},
+			emptyExt: true,
+		},
+		{
+			filename: "applicationresource-no-env.json",
+			err:      &v1.ErrModelConversion{PropertyName: "$.properties.environment", ValidValue: "not nil"},
+			emptyExt: true,
+		},
 	}
 
 	for _, tt := range conversionTests {

pkg/corerp/api/v20231001preview/testdata/applicationresource-no-env.json

@@ -0,0 +1,24 @@
+{
+  "id": "/subscriptions/00000000-0000-0000-0000-000000000000/resourceGroups/radius-test-rg/providers/Applications.Core/applications/app0",
+  "name": "app0",
+  "type": "Applications.Core/applications",
+  "properties": {
+    "extensions": [
+      {
+        "kind": "kubernetesMetadata",
+        "annotations": {
+          "prometheus.io/scrape": "true",
+          "prometheus.io/port": "80"
+        },
+        "labels": {
+          "foo/bar/team": "credit",
+          "foo/bar/contact": "radiususer"
+        }
+      },
+      {
+        "kind": "kubernetesNamespace",
+        "namespace": "app0-ns"
+      }
+    ]
+  }
+}

pkg/corerp/api/v20231001preview/testdata/applicationresource-no-properties.json

@@ -0,0 +1,3 @@
+{  
+  "name": "myapp"
+}

pkg/corerp/datamodel/converter/application_converter.go

@@ -46,6 +46,9 @@ func ApplicationDataModelFromVersioned(content []byte, version string) (*datamod
 			return nil, err
 		}
 		dm, err := am.ConvertTo()
+		if err != nil {
+			return nil, err
+		}
 		return dm.(*datamodel.Application), err
 
 	default: