fix(tests): v2 wip

Author: devthejo

pkg/node/process.go

@@ -22,7 +22,7 @@ type ProcessManager struct {
 	gcInterval          time.Duration
 	idleTimeout         time.Duration
 	tempDir             string
-	logger              logger.LogrusLogger
+	logger              logger.Logger
 	healthCheckWait     time.Duration
 	healthCheckInterval time.Duration
 	requestTimeout      time.Duration

tests/fixtures/domain-sdk/functions/simpleconfigmaps/composition.fn.ts

@@ -1,68 +1,53 @@
-import { logger, FieldRef, v1 } from "@crossplane-js/sdk"
-import type { CrossplaneDesiredResources, CrossplaneObservedResources } from "@crossplane-js/sdk"
+import {
+  logger,
+  // FieldRef,
+  v1,
+} from '@crossplane-js/sdk'
+import type {
+  CrossplaneDesiredResources,
+  // CrossplaneObservedResources
+} from '@crossplane-js/sdk'
 
-import type { SimpleConfigMap } from "@/models/test.crossplane.io/v1beta1"
+import type { SimpleConfigMap } from '../../models/test.crossplane.io/v1beta1'
 
-export default function(composite: SimpleConfigMap, _resources: CrossplaneObservedResources): CrossplaneDesiredResources {
-  logger.info("Composition function started")
+export default function (
+  composite: SimpleConfigMap
+  // resources: CrossplaneObservedResources
+): CrossplaneDesiredResources {
+  logger.info('SimpleConfigMap composition function started')
+  logger.info(`composite:${JSON.stringify(composite)}`)
 
   const namespace = composite.getNamespace()
   const isReady = composite.isReady()
 
-  logger.info(`Ready: ${isReady}`)
+  logger.info(`Namespace: ${namespace}, Ready: ${isReady}`)
   logger.info(composite)
-  
-  const data = composite.spec.data;
-  logger.info({ data }, "Input data")
-  
-  const uppercaseData: Record<string, string> = {};
-  for (const key in data) {
-    uppercaseData[key.toUpperCase()] = data[key].toUpperCase();
-  }
-  
+
+  // Transform data to uppercase for testing (same as original test logic)
+  const transformedData: Record<string, string> = {}
+  Object.entries(composite.spec.data).forEach(([key, value]) => {
+    transformedData[key.toUpperCase()] = value.toUpperCase()
+  })
+
   const testConfigMap = new v1.ConfigMap({
     metadata: {
-      name: "generated-configmap",
-      namespace: namespace,
-      labels: {
-        example: "true"
-      },
-    },
-    data: {
-      ...uppercaseData,
-      // hello: new FieldRef<string>(composite, "$.status.conditions[?(@.type=='Ready')].status", ""),
+      name: 'generated-configmap',
+      namespace: namespace || 'test-xfuncjs',
     },
+    data: transformedData,
   })
 
-  const desired = {
+  const desired: CrossplaneDesiredResources = {
     resources: {
       configmap: {
-        resource: {
-          apiVersion: "kubernetes.crossplane.io/v1alpha1",
-          kind: "Object",
-          metadata: {
-            // name: `${composite.getName()}-configmap`,
-            name: `generated-configmap`,
-            annotations: {
-              "uptest.upbound.io/timeout": "60"
-            }
-          },
-          spec: {
-            forProvider: {
-              manifest: testConfigMap,
-            },
-            providerConfigRef: {
-              // name: "in-cluster",
-              name: "default",
-            },
-          },
-        },
+        resource: testConfigMap,
+        ready: true,
       },
     },
   }
-  
-  logger.info("Composition function completed")
-  logger.debug({ desired }, "Generated output")
-  
-  return desired;
+
+  logger.info('SimpleConfigMap composition function completed')
+  logger.debug({ desired }, 'Generated output')
+
+  return desired
 }

tests/fixtures/domain-sdk/functions/simpleconfigmaps/xrd.yaml

@@ -4,8 +4,11 @@ metadata:
   name: simpleconfigmaps.test.crossplane.io
 spec:
   scope: Namespaced
+  defaultCompositeDeletePolicy: Foreground
   group: test.crossplane.io
   names:
+    categories:
+      - crossplane
     kind: SimpleConfigMap
     plural: simpleconfigmaps
     singular: simpleconfigmap
@@ -22,5 +25,8 @@ spec:
               properties:
                 data:
                   type: object
+                  description: Key-value data to be transformed and stored in ConfigMap
                   additionalProperties:
                     type: string
+              required:
+                - data