Remove ViewChild from metadata

ViewChild will always be null, and so there is no point in mocking it. viewChild.required throws an error in mocked components without this change. Fixes GH-8634
help-me-mom · Apr 16, 2024 · cb45650 · cb45650
1 parent 78b1979
commit cb45650
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 65 deletions.
diff --git a/libs/ng-mocks/src/lib/common/decorate.queries.ts b/libs/ng-mocks/src/lib/common/decorate.queries.ts
@@ -27,7 +27,10 @@ const generateFinalQueries = (queries: {
 
   for (const key of Object.keys(queries)) {
     const query: Query & { ngMetadataName?: string } = queries[key];
-    final.push([key, query]);
+
+    if (!query.isViewQuery || isInternalKey(key)) {
+      final.push([key, query]);
+    }
 
     if (!query.isViewQuery && !isInternalKey(key)) {
       scanKeys.push(key);

diff --git a/libs/ng-mocks/src/lib/mock-component/mock-component.spec.ts b/libs/ng-mocks/src/lib/mock-component/mock-component.spec.ts
@@ -393,22 +393,6 @@ describe('MockComponent', () => {
           ngMetadataName: 'ContentChildren',
         }),
       ],
-      o3: [
-        jasmine.objectContaining({
-          selector: 'i3',
-          isViewQuery: true,
-          read: TemplateRef,
-          ngMetadataName: 'ViewChild',
-        }),
-      ],
-      o4: [
-        jasmine.objectContaining({
-          selector: 'i4',
-          isViewQuery: true,
-          read: TemplateRef,
-          ngMetadataName: 'ViewChildren',
-        }),
-      ],
       o5: [
         jasmine.objectContaining({
           selector: 'i5',
@@ -425,22 +409,6 @@ describe('MockComponent', () => {
           ngMetadataName: 'ContentChildren',
         }),
       ],
-      o7: [
-        jasmine.objectContaining({
-          selector: 'i7',
-          isViewQuery: true,
-          read: ElementRef,
-          ngMetadataName: 'ViewChild',
-        }),
-      ],
-      o8: [
-        jasmine.objectContaining({
-          selector: 'i8',
-          isViewQuery: true,
-          read: ElementRef,
-          ngMetadataName: 'ViewChildren',
-        }),
-      ],
 
       __ngMocksVcr_o1: [
         jasmine.objectContaining({

diff --git a/libs/ng-mocks/src/lib/mock-directive/mock-directive.spec.ts b/libs/ng-mocks/src/lib/mock-directive/mock-directive.spec.ts
@@ -300,22 +300,6 @@ describe('MockDirective', () => {
           ngMetadataName: 'ContentChildren',
         }),
       ],
-      o3: [
-        jasmine.objectContaining({
-          selector: 'i3',
-          isViewQuery: true,
-          read: TemplateRef,
-          ngMetadataName: 'ViewChild',
-        }),
-      ],
-      o4: [
-        jasmine.objectContaining({
-          selector: 'i4',
-          isViewQuery: true,
-          read: TemplateRef,
-          ngMetadataName: 'ViewChildren',
-        }),
-      ],
       o5: [
         jasmine.objectContaining({
           selector: 'i5',
@@ -332,22 +316,6 @@ describe('MockDirective', () => {
           ngMetadataName: 'ContentChildren',
         }),
       ],
-      o7: [
-        jasmine.objectContaining({
-          selector: 'i7',
-          isViewQuery: true,
-          read: ElementRef,
-          ngMetadataName: 'ViewChild',
-        }),
-      ],
-      o8: [
-        jasmine.objectContaining({
-          selector: 'i8',
-          isViewQuery: true,
-          read: ElementRef,
-          ngMetadataName: 'ViewChildren',
-        }),
-      ],
 
       __ngMocksVcr_o1: [
         jasmine.objectContaining({