fix(angular): set proper dest property for accessor copy (#543)

* fix(angular): set proper dest property for accessor copy

* remove angular directives to verify same generation

* add missing dependency

* bring back radio accessor

* set prettierignore for Angular project

* prettier ignore generated files

* set gitattributes

* upgrade prettier

* fix test

* use vitest for unit test

* use inline snapshot matcher

.gitattributes

@@ -0,0 +1 @@
+* text=auto eol=lf

.github/workflows/actions/check-git-context/action.yml

@@ -0,0 +1,11 @@
+name: 'Check Git Context'
+description: 'checks for a dirty git context, failing if the context is dirty'
+runs:
+  using: composite
+  steps:
+    - name: Git status check
+      # here we check that there are no changed / new files.
+      # we use `git status`, grep out the build zip used throughout CI,
+      # and check if there are more than 0 lines in the output.
+      run: if [[ $(git status --short | grep -c -v stencil-core-build.zip) -ne 0 ]]; then STATUS=$(git status --verbose); printf "%s" "$STATUS"; git diff | cat; exit 1; fi
+      shell: bash

.github/workflows/ci.yml

@@ -38,6 +38,9 @@ jobs:
 
       - name: Test
         run: pnpm test
+
+      - name: Check Git Context
+        uses: ./.github/workflows/actions/check-git-context
   format:
     name: Format
     uses: ./.github/workflows/format.yml

example-project/component-library-angular/.prettierignore

@@ -0,0 +1,2 @@
+# autogenerated files from stencil
+src/directives/*.ts

example-project/component-library-angular/package.json

@@ -14,10 +14,11 @@
     "build.ng": "pnpm run build.es2015 && pnpm run build.es5",
     "build.es2015": "ngc -p tsconfig.json && rollup --config ./scripts/rollup.config.mjs",
     "build.es5": "ngc -p tsconfig.legacy.json && rollup --config ./scripts/rollup.config.legacy.mjs",
+    "clean": "rimraf dist build src/directives",
     "lint": "echo \"lint not configured\" && exit 0",
     "prerelease": "pnpm run validate && np prerelease --yolo --any-branch --tag next",
     "prettier": "pnpm run prettier.base --write",
-    "prettier.base": "prettier \"./({scripts,src,__tests__}/**/*.{ts,tsx,js,jsx})|*.{ts,tsx,js,jsx}\"",
+    "prettier.base": "prettier \"./({scripts,src,__tests__,!src/directives}/**/*.{ts,tsx,js,jsx})|*.{ts,tsx,js,jsx}\" --ignore-path=\"./.prettierignore\"",
     "prettier.dry-run": "pnpm run prettier.base --list-different",
     "test": "jest",
     "tsc": "tsc -p .",

example-project/component-library-angular/src/directives/boolean-value-accessor.ts

@@ -7,15 +7,15 @@ import { ValueAccessor } from './value-accessor';
   /* tslint:disable-next-line:directive-selector */
   selector: 'my-checkbox',
   host: {
-    '(myChange)': 'handleChangeEvent($event.target.checked)',
+    '(myChange)': 'handleChangeEvent($event.target.checked)'
   },
   providers: [
     {
       provide: NG_VALUE_ACCESSOR,
       useExisting: BooleanValueAccessor,
-      multi: true,
-    },
-  ],
+      multi: true
+    }
+  ]
 })
 export class BooleanValueAccessor extends ValueAccessor {
   constructor(el: ElementRef) {

example-project/component-library-angular/src/directives/number-value-accessor.ts

@@ -7,22 +7,22 @@ import { ValueAccessor } from './value-accessor';
   /* tslint:disable-next-line:directive-selector */
   selector: 'my-input[type=number]',
   host: {
-    '(myChange)': 'handleChangeEvent($event.target.value)',
+    '(myChange)': 'handleChangeEvent($event.target.value)'
   },
   providers: [
     {
       provide: NG_VALUE_ACCESSOR,
       useExisting: NumericValueAccessor,
-      multi: true,
-    },
-  ],
+      multi: true
+    }
+  ]
 })
 export class NumericValueAccessor extends ValueAccessor {
   constructor(el: ElementRef) {
     super(el);
   }
   override registerOnChange(fn: (_: number | null) => void) {
-    super.registerOnChange((value) => {
+    super.registerOnChange(value => {
       fn(value === '' ? null : parseFloat(value));
     });
   }