fix: respecting internals vs externals

BREAKING CHANGE: respects internals vs externals, to export things use MockBuilder

closes #44

lib/common/ng-mocks-universe.ts

@@ -12,7 +12,7 @@ export const ngMocksUniverse = {
   cacheMocks: new Map(),
   cacheProviders: new Map(),
   config: new Map(),
-  flags: new Set<string>(['cacheModule', 'cacheComponent', 'cacheDirective', 'cacheProvider']),
+  flags: new Set<string>(['cacheModule', 'cacheComponent', 'cacheDirective', 'cacheProvider', 'correctModuleExports']),
   global: new Map(),
   touches: new Set<AnyType<any> | InjectionToken<any>>(),
 };

lib/mock-helper/mock-helper.reset.ts

@@ -6,6 +6,12 @@ export default (): void => {
   ngMocksUniverse.cacheProviders = new Map();
   ngMocksUniverse.config = new Map();
   ngMocksUniverse.global = new Map();
-  ngMocksUniverse.flags = new Set(['cacheModule', 'cacheComponent', 'cacheDirective', 'cacheProvider']);
+  ngMocksUniverse.flags = new Set([
+    'cacheModule',
+    'cacheComponent',
+    'cacheDirective',
+    'cacheProvider',
+    'correctModuleExports',
+  ]);
   ngMocksUniverse.touches = new Set();
 };

lib/mock-module/mock-module.spec.fixtures.ts

@@ -54,6 +54,7 @@ export class ExampleConsumerComponent {}
 export class ChildModule {}
 
 @NgModule({
+  exports: [ChildModule],
   imports: [ChildModule],
 })
 export class ParentModule {}
@@ -69,11 +70,13 @@ export class SameImports1Module {}
 export class SameImports2Module {}
 
 @NgModule({
+  exports: [ChildModule],
   imports: [ChildModule],
 })
 export class LogicNestedModule {}
 
 @NgModule({
+  exports: [ChildModule, LogicNestedModule],
   imports: [ChildModule, LogicNestedModule],
 })
 export class LogicRootModule {}

tests/correct-module-exports-11/test.spec.ts

@@ -0,0 +1,38 @@
+import { Component, NgModule } from '@angular/core';
+import { TestBed } from '@angular/core/testing';
+import { MockModule, MockRender } from 'ng-mocks';
+
+@Component({
+  selector: 'internal',
+  template: 'internal',
+})
+class InternalComponent {}
+
+@Component({
+  selector: 'external',
+  template: 'external',
+})
+class ExternalComponent {}
+
+@NgModule({
+  declarations: [InternalComponent, ExternalComponent],
+  exports: [ExternalComponent],
+})
+class TargetModule {}
+
+describe('correct-module-exports-11', () => {
+  beforeEach(() =>
+    TestBed.configureTestingModule({
+      imports: [MockModule(TargetModule)],
+    }).compileComponents()
+  );
+
+  it('fails on not exported module', () => {
+    expect(() => MockRender(InternalComponent)).toThrowError(/'internal' is not a known element/);
+  });
+
+  it('renders an exported module', () => {
+    const fixture = MockRender(ExternalComponent);
+    expect(fixture.nativeElement.innerHTML).toEqual('<external></external>');
+  });
+});

tests/issue-222/common-module.spec.ts

@@ -11,6 +11,7 @@ class TargetComponent {}
 
 @NgModule({
   declarations: [TargetComponent],
+  exports: [TargetComponent],
   imports: [CommonModule],
 })
 class TargetModule {}