feat: remove more trailing whitespaces for component fixture snapshot (…

…#961)

Closes #287

.husky/_/husky.sh

@@ -0,0 +1,30 @@
+#!/bin/sh
+if [ -z "$husky_skip_init" ]; then
+  debug () {
+    [ "$HUSKY_DEBUG" = "1" ] && echo "husky (debug) - $1"
+  }
+
+  readonly hook_name="$(basename "$0")"
+  debug "starting $hook_name..."
+
+  if [ "$HUSKY" = "0" ]; then
+    debug "HUSKY env variable is set to 0, skipping hook"
+    exit 0
+  fi
+
+  if [ -f ~/.huskyrc ]; then
+    debug "sourcing ~/.huskyrc"
+    . ~/.huskyrc
+  fi
+
+  export readonly husky_skip_init=1
+  sh -e "$0" "$@"
+  exitCode="$?"
+
+  if [ $exitCode != 0 ]; then
+    echo "husky - $hook_name hook exited with code $exitCode (error)"
+    exit $exitCode
+  fi
+
+  exit 0
+fi

.husky/pre-commit

@@ -1,4 +1,6 @@
 #!/bin/sh
 . "$(dirname "$0")/_/husky.sh"
 
-yarn lint-staged
+yarn lint:fix
+yarn lint:prettier
+git add .

e2e/snapshot-serializers/__tests__/__snapshots__/foo.component.ts.snap

@@ -1,12 +1,36 @@
 // Jest Snapshot v1, https://goo.gl/fbAQLP
 
 exports[`snapshot should work 1`] = `
+<foo
+  __ngContext__={[Function LRootView]}
+  condition1={[Function Boolean]}
+  condition2="false"
+  value1={[Function String]}
+  value2={[Function String]}
+>
+  <p>
+    Line 1
+  </p><div>
+    <div>
+       val1 
+    </div>
+  </div>
+</foo>
+`;
+
+exports[`snapshot should work 2`] = `
 <div
   id="root0"
 >
   <p>
-     simple-with-styles works!
-
+    Line 1
   </p>
+  <div>
+    <div>
+       val1 
+    </div>
+    
+    
+  </div>
 </div>
 `;

e2e/snapshot-serializers/__tests__/foo.component.html

@@ -1,3 +1,9 @@
-<p>
-  simple-with-styles works!
-</p>
+<p>Line 1</p>
+<div>
+  <div *ngIf="condition1">
+    {{ value1 }}
+  </div>
+  <span *ngIf="condition2">
+    {{ value2 }}
+ </span>
+</div>

e2e/snapshot-serializers/__tests__/foo.component.ts

@@ -1,4 +1,4 @@
-import { Component } from '@angular/core';
+import { Component, Input } from '@angular/core';
 import { TestBed, waitForAsync } from '@angular/core/testing';
 
 @Component({
@@ -14,7 +14,13 @@ import { TestBed, waitForAsync } from '@angular/core/testing';
     `,
   ],
 })
-class FooComponent {}
+class FooComponent {
+  @Input() value1 = 'val1';
+  @Input() value2 = 'val2';
+
+  condition1 = true;
+  condition2 = false;
+}
 
 test(
   'snapshot should work',
@@ -26,6 +32,7 @@ test(
     const fixture = TestBed.createComponent(FooComponent);
     fixture.detectChanges();
 
+    expect(fixture).toMatchSnapshot();
     expect(fixture.debugElement.nativeElement).toMatchSnapshot();
   }),
 );

package.json

@@ -86,20 +86,13 @@
     "husky": "^6.0.0",
     "jest": "^27.0.4",
     "jest-snapshot-serializer-raw": "^1.2.0",
-    "lint-staged": "^11.0.0",
     "pinst": "^2.1.6",
     "prettier": "^2.3.1",
     "rxjs": "^6.6.7",
     "tslib": "^2.2.0",
     "typescript": "~4.1.5",
     "zone.js": "^0.11.4"
   },
-  "lint-staged": {
-    "*.{ts,js}": [
-      "eslint --fix",
-      "git add"
-    ]
-  },
   "engines": {
     "node": "^12.13.0 || ^14.15.0 || >=15.0.0"
   }

src/serializers/ng-snapshot.ts

@@ -97,7 +97,7 @@ const print = (
     // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment
     componentName = fixture.componentRef._elDef.element?.name;
     // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment
-    nodes = (fixture.componentRef._view.nodes || [])
+    nodes = (fixture.componentRef._view.nodes ?? [])
       // eslint-disable-next-line no-prototype-builtins
       .filter((node: VEDebugNode) => node?.hasOwnProperty('renderElement'))
       .map((node: VEDebugNode) => Array.from(node.renderElement.childNodes).map(print).join(''))
@@ -118,7 +118,7 @@ const print = (
     '\n</' +
     componentName +
     '>'
-  );
+  ).replace(/\n^\s*\n/gm, '\n');
 };
 
 // eslint-disable-next-line @typescript-eslint/no-explicit-any, @typescript-eslint/explicit-module-boundary-types