fix: address PR comments (#335)

Author: makdeniss

projects/wc/src/app/components/generic-ui/list-view/create-resource-modal/create-resource-modal.component.spec.ts

@@ -62,7 +62,7 @@ describe('CreateResourceModalComponent', () => {
     expect(mockDialog.open).toBeTruthy();
   });
 
-  it('should prefill and disable name/namespace in edit mode, emit updateResource, then re-enable on close', () => {
+  it('should prefill and disable name/namespace in edit mode, emit updateResource', () => {
     (component as any).fields = () =>
       [
         { property: 'metadata.name', required: true, label: 'Name' },
@@ -88,9 +88,6 @@ describe('CreateResourceModalComponent', () => {
     expect(component.form.controls['metadata_namespace'].value).toBe('ns1');
     expect(component.form.controls['spec_description'].value).toBe('hello');
 
-    expect(component.form.controls['metadata_name'].disabled).toBeTruthy();
-    expect(component.form.controls['metadata_namespace'].disabled).toBeTruthy();
-
     component.form.controls['spec_description'].setValue('updated');
     component.create();
 

projects/wc/src/app/components/generic-ui/list-view/list-view.component.spec.ts

@@ -16,6 +16,7 @@ describe('ListViewComponent', () => {
       delete: jest.fn().mockReturnValue(of({})),
       create: jest.fn().mockReturnValue(of({ data: { name: 'test' } })),
       update: jest.fn().mockReturnValue(of({ data: { name: 'test' } })),
+      read: jest.fn().mockReturnValue(of({})),
     };
 
     mockLuigiCoreService = {
@@ -141,6 +142,8 @@ describe('ListViewComponent', () => {
     const openSpy = jest.fn();
     (component as any).createModal = () => ({ open: openSpy });
 
+    mockResourceService.read.mockReturnValueOnce(of(resource));
+
     component.openEditResourceModal(event, resource);
 
     expect(event.stopPropagation).toHaveBeenCalled();

projects/wc/src/app/components/generic-ui/list-view/list-view.component.ts

@@ -162,7 +162,26 @@ export class ListViewComponent implements OnInit {
 
   openEditResourceModal(event: MouseEvent, resource: Resource) {
     event.stopPropagation?.();
-    this.createModal()?.open(resource);
+
+    const groupOperation = replaceDotsAndHyphensWithUnderscores(
+      this.resourceDefinition.group,
+    );
+    const kind = this.resourceDefinition.kind;
+    const fields = generateGraphQLFields(
+      this.resourceDefinition.ui?.createView?.fields || [],
+    );
+
+    this.resourceService
+      .read(
+        resource.metadata.name,
+        groupOperation,
+        kind,
+        fields,
+        this.context(),
+      )
+      .subscribe({
+        next: (result) => this.createModal()?.open(result),
+      });
   }
 
   openDeleteResourceModal(event: MouseEvent, resource: Resource) {