Fine tune unit tests and enable self.resources.newresource = { ... }

examples/get-started-app/composition.yaml

@@ -19,7 +19,7 @@ spec:
           def compose(self):
             labels = {'example.crossplane.io/app': self.metadata.name}
 
-            d = self.resources.deployment('apps/v1', 'Deployment')
+            d = self.resources.deployment('apps/v1', 'Deployment', self.metadata.namespace)
             d.metadata.labels = labels
             d.spec.replicas = 2
             d.spec.selector.matchLabels = labels
@@ -29,7 +29,7 @@ spec:
             d.spec.template.spec.containers[0].ports[0].containerPort = 80
             d.ready = d.conditions.Available.status
 
-            s = self.resources.service('v1', 'Service')
+            s = self.resources.service('v1', 'Service', self.metadata.namespace)
             s.metadata.labels = labels
             s.spec.selector = labels
             s.spec.ports[0].protocol = 'TCP'

function/composite.py

@@ -90,7 +90,7 @@ def __iter__(self):
 
 class Resources:
     def __init__(self, composite):
-        self._composite = composite
+        self.__dict__['_composite'] = composite
 
     def __getattr__(self, key):
         return self[key]
@@ -111,6 +111,13 @@ def __iter__(self):
         for name, resource in self._composite.response.desired.resources:
             yield name, self[name]
 
+    def __setattr__(self, key, resource):
+        self[key] = resource
+
+    def __setitem__(self, key, resource):
+        print('SETITEM', key, resource)
+        self._composite.response.desired.resources[key].resource = resource
+
     def __delattr__(self, key):
         del self[key]
 

tests/fn_cases/buckets.yaml

@@ -0,0 +1,37 @@
+request:
+  input:
+    composite: |
+      class Composite(BaseComposite):
+        def compose(self):
+
+          self.resources.bucket1.apiVersion = 's3.aws.upbound.io/v1beta2'
+          self.resources.bucket1.kind = 'Bucket'
+          self.resources.bucket1.spec.forProvider.region = 'us-east-1'
+
+          self.resources.bucket2 = {
+            'apiVersion': 's3.aws.upbound.io/v1beta2',
+            'kind': 'Bucket',
+            'spec': {
+              'forProvider': {
+                'region': 'us-east-1',
+              },
+            },
+          }
+
+response:
+  desired:
+    resources:
+      bucket1:
+        resource:
+          apiVersion: s3.aws.upbound.io/v1beta2
+          kind: Bucket
+          spec:
+            forProvider:
+              region: us-east-1
+      bucket2:
+        resource:
+          apiVersion: s3.aws.upbound.io/v1beta2
+          kind: Bucket
+          spec:
+            forProvider:
+              region: us-east-1

tests/fn_cases/get-started-app.yaml

@@ -3,6 +3,7 @@ request:
     composite:
       resource:
         metadata:
+          namespace: my-namespace
           name: my-app
         spec:
           image: nginx
@@ -26,7 +27,7 @@ request:
           def compose(self):
             labels = {'example.crossplane.io/app': self.metadata.name}
 
-            d = self.resources.deployment('apps/v1', 'Deployment')
+            d = self.resources.deployment('apps/v1', 'Deployment', self.metadata.namespace)
             d.metadata.labels = labels
             d.spec.replicas = 2
             d.spec.selector.matchLabels = labels
@@ -36,7 +37,7 @@ request:
             d.spec.template.spec.containers[0].ports[0].containerPort = 80
             d.ready = d.conditions.Available.status
 
-            s = self.resources.service('v1', 'Service')
+            s = self.resources.service('v1', 'Service', self.metadata.namespace)
             s.metadata.labels = labels
             s.spec.selector = labels
             s.spec.ports[0].protocol = 'TCP'
@@ -62,6 +63,7 @@ response:
           metadata:
             labels:
               example.crossplane.io/app: my-app
+            namespace: my-namespace
           spec:
             replicas: 2
             selector:
@@ -85,6 +87,7 @@ response:
           metadata:
             labels:
               example.crossplane.io/app: my-app
+            namespace: my-namespace
           spec:
             selector:
               example.crossplane.io/app: my-app