test: restructure app tests (#42)

Author: timdeschryver

package.json

@@ -50,6 +50,7 @@
     "@angular/cli": "~8.0.0",
     "@angular/compiler-cli": "^8.0.0",
     "@angular/language-service": "^8.0.0",
+    "@testing-library/jest-dom": "^4.1.0",
     "@types/jest": "~24.0.11",
     "@types/node": "~12.0.3",
     "codelyzer": "^5.0.1",

projects/testing-library/tests/counter/counter.spec.ts

@@ -5,35 +5,49 @@ import { render } from '../src/public_api';
 
 @Component({
   selector: 'fixture',
-  template: ``,
+  template: `
+    <input type="text" data-testid="input" />
+    <button>button</button>
+  `,
 })
 class FixtureComponent {}
 
+test('creates queries and events', async () => {
+  const component = await render(FixtureComponent);
+
+  component.input(component.getByTestId('input'), { target: { value: 'a super awesome input' } });
+  component.getByDisplayValue('a super awesome input');
+  component.click(component.getByText('button'));
+});
+
 @NgModule({
   declarations: [FixtureComponent],
 })
 export class FixtureModule {}
-
-test('should not throw if component is declared in an import', async () => {
-  await render(FixtureComponent, {
-    imports: [FixtureModule],
-    excludeComponentDeclaration: true,
+describe('excludeComponentDeclaration', () => {
+  test('should not throw if component is declared in an import', async () => {
+    await render(FixtureComponent, {
+      imports: [FixtureModule],
+      excludeComponentDeclaration: true,
+    });
   });
 });
 
-test('should add NoopAnimationsModule by default', async () => {
-  await render(FixtureComponent);
-  const noopAnimationsModule = TestBed.get<NoopAnimationsModule>(NoopAnimationsModule);
-  expect(noopAnimationsModule).toBeDefined();
-});
-
-test('should not add NoopAnimationsModule if BrowserAnimationsModule is an import', async () => {
-  await render(FixtureComponent, {
-    imports: [BrowserAnimationsModule],
+describe('animationModule', () => {
+  test('should add NoopAnimationsModule by default', async () => {
+    await render(FixtureComponent);
+    const noopAnimationsModule = TestBed.get<NoopAnimationsModule>(NoopAnimationsModule);
+    expect(noopAnimationsModule).toBeDefined();
   });
 
-  const browserAnimationsModule = TestBed.get<BrowserAnimationsModule>(BrowserAnimationsModule);
-  expect(browserAnimationsModule).toBeDefined();
+  test('should not add NoopAnimationsModule if BrowserAnimationsModule is an import', async () => {
+    await render(FixtureComponent, {
+      imports: [BrowserAnimationsModule],
+    });
 
-  expect(() => TestBed.get<NoopAnimationsModule>(NoopAnimationsModule)).toThrow();
+    const browserAnimationsModule = TestBed.get<BrowserAnimationsModule>(BrowserAnimationsModule);
+    expect(browserAnimationsModule).toBeDefined();
+
+    expect(() => TestBed.get<NoopAnimationsModule>(NoopAnimationsModule)).toThrow();
+  });
 });

projects/testing-library/tests/form/form.component.html

@@ -20,7 +20,6 @@ class WrapperComponent implements OnInit {
 }
 
 test('allows for a custom wrapper', async () => {
-  jest.spyOn(console, 'log').mockImplementation(() => {});
   const { getByText } = await render('<fixture></fixture>', {
     declarations: [FixtureComponent],
     wrapper: WrapperComponent,