Merge pull request #34 from aerogear/github-actions

GitHub actions

Author: wtrocki

.circleci/config.yml

@@ -0,0 +1,36 @@
+name: Build, Lint & Test
+
+on:
+  push:
+    branches: [master]
+  pull_request:
+    types: [opened, synchronize, reopened, ready_for_review]
+
+jobs:
+  build:
+    runs-on: ubuntu-latest
+    timeout-minutes: 60
+    steps:
+    - name: Checkout master
+      uses: actions/checkout@v2
+    - name: Use Node.js
+      uses: actions/setup-node@v1
+      with:
+        node-version: 12.x
+    - name: Cache Dependencies
+      id: cache-dependencies
+      uses: actions/cache@master
+      with:
+        path: |
+          node_modules
+          */*/node_modules
+        key: ${{ runner.os }}-${{ hashFiles('**/package.json') }}
+    - name: Install dependencies
+      if: steps.cache-dependencies.outputs.cache-hit != 'true'
+      run: yarn
+    - name: Build
+      run: yarn build
+    - name: Lint
+      run: yarn lint
+    - name: Unit Test
+      run: yarn test

.github/workflows/build.yml

@@ -0,0 +1,34 @@
+name: Release workflow
+on:
+  push:
+    tags:
+      - '*'
+jobs:
+  build:
+    runs-on: ubuntu-latest
+    timeout-minutes: 60
+    env:
+      CI: true
+    steps:
+    - name: Checkout master
+      uses: actions/checkout@v2
+    - name: Use Node.js
+      uses: actions/setup-node@v1
+      with:
+        node-version: 12.x
+    - name: Cache Dependencies
+      id: cache-dependencies
+      uses: actions/cache@master
+      with:
+        path: |
+          node_modules
+          */*/node_modules
+        key: ${{ runner.os }}-${{ hashFiles('**/package.json') }}
+    - name: Build
+      run: yarn build
+    - name: Lint
+      run: yarn lint
+    - name: Unit Tests
+      run: yarn test
+    - name: Publish
+      run: echo  "//registry.npmjs.org/:_authToken=${{secrets.NPM_AUTH_TOKEN}}" > ~/.npmrc && TAG=${GITHUB_REF#"refs/tags/"} npm run release:publish

.github/workflows/release.yml

@@ -4,11 +4,18 @@ import { IonButton } from '@ionic/react';
 
 import createContext from './_createContext';
 import mount from './_mount';
+import { merge } from 'lodash';
 
-const parent = {
-  maxCount: 3,
-  value: [],
-};
+const onChange = jest.fn();
+const context = (schema?: {}) =>
+  createContext(
+    merge({ x: { type: Array, maxCount: 3 }, 'x.$': String }, schema),
+    { onChange, model: { x: [] } },
+  );
+
+beforeEach(() => {
+  onChange.mockClear();
+});
 
 test('<ListAddField> - works', () => {
   const element = <ListAddField name="x.$" parent={parent} />;
@@ -58,20 +65,9 @@ test('<ListAddField> - prevents onClick when limit reached', () => {
 });
 
 test('<ListAddField> - correctly reacts on click', () => {
-  const onChange = jest.fn();
-
-  const element = (
-    <ListAddField
-      name="x.1"
-      parent={Object.assign({}, parent, { onChange })}
-      value="y"
-    />
-  );
-  const wrapper = mount(
-    element,
-    createContext({ x: { type: Array }, 'x.$': { type: String } }),
-  );
+  const element = <ListAddField name="x.1" value="y" />;
+  const wrapper = mount(element, context());
 
   expect(wrapper.find(IonButton).simulate('click')).toBeTruthy();
-  expect(onChange).toHaveBeenLastCalledWith(['y']);
+  expect(onChange).toHaveBeenLastCalledWith('x', ['y']);
 });

__tests__/ListAddField.tsx

@@ -4,12 +4,18 @@ import { IonButton } from '@ionic/react';
 
 import createContext from './_createContext';
 import mount from './_mount';
+import { merge } from 'lodash';
 
-const parent = {
-  maxCount: 3,
-  minCount: 0,
-  value: ['x', 'y', 'z'],
-};
+const onChange = jest.fn();
+const context = (schema?: {}) =>
+  createContext(
+    merge({ x: { type: Array, maxCount: 3 }, 'x.$': String }, schema),
+    { onChange, model: { x: ['x', 'y', 'z'] } },
+  );
+
+beforeEach(() => {
+  onChange.mockClear();
+});
 
 test('<ListDelField> - works', () => {
   const element = <ListDelField name="x.1" parent={parent} />;
@@ -59,16 +65,9 @@ test('<ListDelField> - prevents onClick when limit reached', () => {
 });
 
 test('<ListDelField> - correctly reacts on click', () => {
-  const onChange = jest.fn();
-
-  const element = (
-    <ListDelField name="x.1" parent={Object.assign({}, parent, { onChange })} />
-  );
-  const wrapper = mount(
-    element,
-    createContext({ x: { type: Array }, 'x.$': { type: String } }),
-  );
+  const element = <ListDelField name="x.1" />;
+ const wrapper = mount(element, context());
 
-  expect(wrapper.find(IonButton).simulate('click')).toBeTruthy();
-  expect(onChange).toHaveBeenLastCalledWith(['x', 'y']);
+ expect(wrapper.find(IonButton).simulate('click')).toBeTruthy();
+ expect(onChange).toHaveBeenLastCalledWith('x', ['x', 'z']);
 });

__tests__/ListDelField.tsx

@@ -114,13 +114,13 @@ test('<ListField> - renders children with correct name (children)', () => {
       .find(Child)
       .at(0)
       .prop('name'),
-  ).toBe('x.0');
+  ).toBe('0');
   expect(
     wrapper
       .find(Child)
       .at(1)
       .prop('name'),
-  ).toBe('x.1');
+  ).toBe('1');
 });
 
 test('<ListField> - renders children with correct name (value)', () => {
@@ -135,11 +135,11 @@ test('<ListField> - renders children with correct name (value)', () => {
       .find(ListItemField)
       .at(0)
       .prop('name'),
-  ).toBe('x.0');
+  ).toBe('0');
   expect(
     wrapper
       .find(ListItemField)
       .at(1)
       .prop('name'),
-  ).toBe('x.1');
+  ).toBe('1');
 });

__tests__/ListField.tsx

@@ -22,13 +22,13 @@ test('<NestField> - renders an <AutoField> for each field', () => {
       .find(AutoField)
       .at(0)
       .prop('name'),
-  ).toBe('x.a');
+  ).toBe('a');
   expect(
     wrapper
       .find(AutoField)
       .at(1)
       .prop('name'),
-  ).toBe('x.b');
+  ).toBe('b');
 });
 
 test('<NestField> - renders custom content if given', () => {

__tests__/NestField.tsx

@@ -13,6 +13,8 @@
     "coverage": "jest --coverage --runInBand",
     "lint": "eslint --ext js,ts,tsx src",
     "reset": "rimraf dist node_modules",
+    "release:validate": "./scripts/validateRelease.sh TAG=1.2.3",
+    "release:publish": "./scripts/publishRelease.sh",
     "test": "jest --runInBand"
   },
   "devDependencies": {
@@ -24,16 +26,16 @@
     "@ionic/react-test-utils": "^0.0.3",
     "@testing-library/react": "^10.0.2",
     "@types/classnames": "2.2.10",
-    "@types/enzyme": "3.10.5",
+    "@types/enzyme": "3.10.8",
     "@types/invariant": "^2.2.31",
     "@types/jest": "^25.1.5",
     "@types/lodash": "4.14.149",
     "@typescript-eslint/eslint-plugin": "2.25.0",
     "@typescript-eslint/parser": "2.25.0",
     "babel-eslint": "^10.1.0",
     "classnames": "^2.2.6",
-    "enzyme": "^3.11.0",
-    "enzyme-adapter-react-16": "^1.15.2",
+    "enzyme": "3.11.0",
+    "enzyme-adapter-react-16": "1.15.6",
     "eslint": "6.8.0",
     "eslint-config-prettier": "6.10.1",
     "eslint-config-vazco": "5.2.0",
@@ -60,8 +62,8 @@
     "ts-node": "8.8.1",
     "tslib": "1.11.1",
     "typescript": "3.8.3",
-    "uniforms": "v3.0.0-alpha.5",
-    "uniforms-bridge-simple-schema-2": "v3.0.0-alpha.5"
+    "uniforms": "3.0.0",
+    "uniforms-bridge-simple-schema-2": "3.0.0"
   },
   "engines": {
     "npm": ">=5.0.0"

package.json

@@ -0,0 +1,18 @@
+#!/bin/bash
+echo "Preparing release"
+
+set -e
+
+rm -Rf node_modules
+yarn
+yarn clean
+yarn test
+yarn lint
+
+# don't run in CI
+if [ ! "$CI" = true ]; then
+  npm publish
+fi
+
+echo "Repository is ready for release."
+

scripts/prepareRelease.sh

@@ -0,0 +1,29 @@
+#!/bin/bash
+
+# explicit declaration that this script needs a $TAG variable passed in e.g TAG=1.2.3 ./script.sh
+TAG=$TAG
+
+RELEASE_SYNTAX='^[0-9]+\.[0-9]+\.[0-9]+$'
+PRERELEASE_SYNTAX='^[0-9]+\.[0-9]+\.[0-9]+(-.+)+$'
+
+if [ ! "$CI" = true ]; then
+  echo "Warning: this script should not be run outside of the CI"
+  echo "If you really need to run this script, you can use"
+  echo "CI=true ./scripts/publishRelease.sh"
+  exit 1
+fi
+
+if [[ "$(echo $TAG | grep -E $RELEASE_SYNTAX)" == "$TAG" ]]; then
+  TAG=$TAG npm run release:validate
+  echo "publishing a new release: $TAG"
+  npm publish
+elif [[ "$(echo $TAG | grep -E $PRERELEASE_SYNTAX)" == "$TAG" ]]; then
+  npm publish --tag next 
+  echo "publishing a new pre release: $TAG" 
+  "npm publish --tag next"
+else
+  echo "Error: the tag $TAG is not valid. exiting..."
+  exit 1
+fi
+
+