@jscutlery/playwright-ct-angular 0.1.1 → 0.1.3

package/index.cjs

@@ -62,45 +62,49 @@ playwright__namespace['_addRunnerPlugin'](() => {
 });
 const mountTest = playwright__namespace.test.extend(mount.fixtures);
 const test = mountTest.extend({
-  /**
-   * Override mount to return output spies and enjoy type inference.
-   * This allows this:
-   *
-   * const { spies } = mount(..., {outputs: {citySelect: jest.fn()}})
-   *
-   * instead of this:
-   *
-   * const citySelectSpy: jest.MockedFunction<(city: string) => void> = jest.fn();
-   * mount(..., {outputs: {citySelect: citySelectSpy}});
-   *
-   * @hack we are willingly not using `async mount(...)` syntax here
-   * because it would be transpiled to `_async(function mount() { ... }`
-   * by Nx's default rollout config.
-   * Playwright dynamically parses the given function so converting
-   * async / await to a wrapped function would simply break the library causing
-   * a `TypeError: mount is not a function` error.
+  /**
+   * Override mount to return output spies and enjoy type inference.
+   * This allows this:
+   *
+   * const { spies } = mount(..., {outputs: {citySelect: jest.fn()}})
+   *
+   * instead of this:
+   *
+   * const citySelectSpy: jest.MockedFunction<(city: string) => void> = jest.fn();
+   * mount(..., {outputs: {citySelect: citySelectSpy}});
+   *
+   * @hack we are willingly not using `async mount(...)` syntax here
+   * because it would be transpiled to `_async(function mount() { ... }`
+   * by Nx's default rollout config.
+   * Playwright dynamically parses the given function so converting
+   * async / await to a wrapped function would simply break the library causing
+   * a `TypeError: mount is not a function` error.
    */
   mount({
     mount,
     page
   }, use) {
     return use(_async(function (component, options = {}) {
-      var _options$spyOutputs$r, _options$spyOutputs;
       /* 1. Create a spy for each output in `spyOutputs`... */
-      const spies = (_options$spyOutputs$r = (_options$spyOutputs = options.spyOutputs) == null ? void 0 : _options$spyOutputs.reduce((acc, key) => {
-        return Object.assign({}, acc, {
+      const spies = options.spyOutputs?.reduce((acc, key) => {
+        return {
+          ...acc,
           [key]: jestMock.fn()
-        });
-      }, {})) != null ? _options$spyOutputs$r : {};
+        };
+      }, {}) ?? {};
       /* 2. ...then use it as an output... */
-      return _await(mount(component, Object.assign({}, options, {
-        outputs: Object.assign({}, options.outputs, spies)
-      })), function () {
-        /* Grab a locator to the TestBed's root element.
-         * We are loading a fresh new page for each test
-         * so the id should always be "root0".
-         * Otherwise, we'd have to hack our way and somehow grab all
-         * the `#root*`, sort them and grab the last one.
+      return _await(mount(component, {
+        ...options,
+        outputs: {
+          ...options.outputs,
+          ...spies
+        }
+      }), function () {
+        /* Grab a locator to the TestBed's root element.
+         * We are loading a fresh new page for each test
+         * so the id should always be "root0".
+         * Otherwise, we'd have to hack our way and somehow grab all
+         * the `#root*`, sort them and grab the last one.
          * As a matter of fact, we can't grab a reference to the fixture. */
         const locator = page.locator('#root0');
         /* Add spies to the returned locator. */

package/index.js

@@ -15,45 +15,49 @@ playwright['_addRunnerPlugin'](() => {
 });
 const mountTest = playwright.test.extend(fixtures);
 const test = mountTest.extend({
-  /**
-   * Override mount to return output spies and enjoy type inference.
-   * This allows this:
-   *
-   * const { spies } = mount(..., {outputs: {citySelect: jest.fn()}})
-   *
-   * instead of this:
-   *
-   * const citySelectSpy: jest.MockedFunction<(city: string) => void> = jest.fn();
-   * mount(..., {outputs: {citySelect: citySelectSpy}});
-   *
-   * @hack we are willingly not using `async mount(...)` syntax here
-   * because it would be transpiled to `_async(function mount() { ... }`
-   * by Nx's default rollout config.
-   * Playwright dynamically parses the given function so converting
-   * async / await to a wrapped function would simply break the library causing
-   * a `TypeError: mount is not a function` error.
+  /**
+   * Override mount to return output spies and enjoy type inference.
+   * This allows this:
+   *
+   * const { spies } = mount(..., {outputs: {citySelect: jest.fn()}})
+   *
+   * instead of this:
+   *
+   * const citySelectSpy: jest.MockedFunction<(city: string) => void> = jest.fn();
+   * mount(..., {outputs: {citySelect: citySelectSpy}});
+   *
+   * @hack we are willingly not using `async mount(...)` syntax here
+   * because it would be transpiled to `_async(function mount() { ... }`
+   * by Nx's default rollout config.
+   * Playwright dynamically parses the given function so converting
+   * async / await to a wrapped function would simply break the library causing
+   * a `TypeError: mount is not a function` error.
    */
   mount({
     mount,
     page
   }, use) {
     return use(async (component, options = {}) => {
-      var _options$spyOutputs$r, _options$spyOutputs;
       /* 1. Create a spy for each output in `spyOutputs`... */
-      const spies = (_options$spyOutputs$r = (_options$spyOutputs = options.spyOutputs) == null ? void 0 : _options$spyOutputs.reduce((acc, key) => {
-        return Object.assign({}, acc, {
+      const spies = options.spyOutputs?.reduce((acc, key) => {
+        return {
+          ...acc,
           [key]: fn()
-        });
-      }, {})) != null ? _options$spyOutputs$r : {};
+        };
+      }, {}) ?? {};
       /* 2. ...then use it as an output... */
-      await mount(component, Object.assign({}, options, {
-        outputs: Object.assign({}, options.outputs, spies)
-      }));
-      /* Grab a locator to the TestBed's root element.
-       * We are loading a fresh new page for each test
-       * so the id should always be "root0".
-       * Otherwise, we'd have to hack our way and somehow grab all
-       * the `#root*`, sort them and grab the last one.
+      await mount(component, {
+        ...options,
+        outputs: {
+          ...options.outputs,
+          ...spies
+        }
+      });
+      /* Grab a locator to the TestBed's root element.
+       * We are loading a fresh new page for each test
+       * so the id should always be "root0".
+       * Otherwise, we'd have to hack our way and somehow grab all
+       * the `#root*`, sort them and grab the last one.
        * As a matter of fact, we can't grab a reference to the fixture. */
       const locator = page.locator('#root0');
       /* Add spies to the returned locator. */

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jscutlery/playwright-ct-angular",
-  "version": "0.1.1",
+  "version": "0.1.3",
   "dependencies": {
     "jest-mock": "^28.1.3",
     "rxjs": "^7.5.7",
@@ -11,15 +11,17 @@
     "@angular/platform-browser": "*",
     "@angular/platform-browser-dynamic": "*",
     "@playwright/test": "^1.28.0",
-    "tslib": "*"
+    "tslib": "*",
+    "@angular/compiler": "16.0.1",
+    "zone.js": "0.13.0"
   },
   "module": "./index.js",
   "main": "./index.cjs",
   "type": "module",
-  "types": "./index.d.ts",
+  "types": "./src/index.d.ts",
   "exports": {
     ".": {
-      "types": "./index.d.ts",
+      "types": "./src/index.d.ts",
       "import": "./index.js",
       "require": "./index.cjs"
     }

package/{index.d.ts → src/index.d.ts}

@@ -1,2 +1,2 @@
-export { devices, expect, test } from './lib/playwright-ct-angular';
-export type { ComponentFixtures, PlaywrightTestConfig } from './lib/playwright-ct-angular';
+export { devices, expect, test } from './lib/playwright-ct-angular';
+export type { ComponentFixtures, PlaywrightTestConfig } from './lib/playwright-ct-angular';