DOP-3420 (#726)

* DOP-3203: Add upsert functionality, asset upserts * fixup * DOP-3203: Stub functions, Stub tests, finalized merge query + logic * DOP-3203: Reorder logic * Improve the typing * fixup * stub toctreeorder * fixup * fixup * fixup * fixup * fixup * Use project for node options, not name * fixup * DOP-3203: Invoke the merging logic * DOP-3203: Coalesce feedback * DOP-3203: Rename associations to associated_products * DOP-3203: Null safety w/ default empty objects in insertions * DOP-3203: Unset _id in new shared metadata documents * DOP-3203: Address feedback further * init test on connect. verify why errors not consoled * export testing fn * update jest config for jest mongodb * add tests for associated products * update tests for associated products. add comments for connector * add test cmd * change tests to snapshots * add tests+snapshots for ToC * minor. return awaited value * run linter for json files * revert index file * configure workflows for persistence tests. update linter scopes * update cd to working-directory * update install command * test with cd vs working-directory * change to cd * update lint config for persistence module * echo current path * add eslint-plugin to persistence package * update package lock * test with dep * retest with eslint config file * add typescript parser * add dev dependencies, update step * update comment * add path filtering and remove bypass tests * remove comment * remove eslintconfig and set eslintrc as root Co-authored-by: Cassidy Schaufele <[email protected]>
mongodb · Jan 5, 2023 · 76bc13f · 76bc13f
1 parent 6ad9150
commit 76bc13f
Show file tree

Hide file tree

Showing 7 changed files with 620 additions and 198 deletions.
diff --git a/.github/workflows/test-persistence.yml b/.github/workflows/test-persistence.yml
@@ -0,0 +1,42 @@
+name: CI-persistence
+
+on:
+  push:
+    branches:
+    - master
+  pull_request:
+    branches:
+    - master
+
+jobs:
+  # Checks for changes to the modules/persistence directory
+  changes:
+    runs-on: ubuntu-latest
+    outputs:
+      persistence: ${{ steps.filter.outputs.persistence }}
+    steps:
+      - uses: dorny/paths-filter@v2
+        id: filter
+        with:
+          filters: |
+            persistence:
+              - 'modules/persistence/**'
+
+  test:
+    needs: changes
+    if: ${{ needs.changes.outputs.persistence == 'true' }}
+    defaults:
+      run:
+        working-directory: 'modules/persistence'
+    runs-on: ubuntu-latest
+    steps:
+    - uses: actions/checkout@v1
+    - uses: actions/setup-node@v1
+      with:
+        node-version: '14.x'
+    - name: Install dependencies
+      run: npm install --dev
+    - name: Lint
+      run: npm run lint
+    - name: Test
+      run: npm test
diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
@@ -7,8 +7,26 @@ on:
   pull_request:
     branches:
     - master
+
 jobs:
+  # Checks for changes outside of /modules directory
+  changes:
+    runs-on: ubuntu-latest
+    outputs:
+      autobuilder: ${{ steps.filter.outputs.autobuilder }}
+    steps:
+    - uses: dorny/paths-filter@v2
+      id: filter
+      with:
+        # set autobuilder filter true
+        # if any changes outside of modules directory
+        filters: |
+          autobuilder:
+            - '!(modules/**)'
+
   test:
+    needs: changes
+    if: ${{ needs.changes.outputs.autobuilder }}
     runs-on: ubuntu-latest
     steps:
     - uses: actions/checkout@v1

diff --git a/modules/persistence/.eslintrc.js b/modules/persistence/.eslintrc.js
@@ -1,4 +1,5 @@
 module.exports = {
+  root: true,
   env: {
     browser: true,
     es6: true,
@@ -14,6 +15,7 @@ module.exports = {
     sourceType: 'module',
   },
   rules: {
+    '@typescript-eslint/no-explicit-any': 'off',
     'no-console': 'off',
   },
 };