fix: sequelize latest (#232)

* fix: sequelize latest

* fix: remove sequelize-6-only property

.github/workflows/ci.yml

@@ -7,22 +7,19 @@ jobs:
     strategy:
       matrix:
         node-version: [12.x, 10.x]
-        sequelize-version: [null, next]
+        sequelize-version: [null, latest]
     steps:
     - uses: actions/checkout@v1
     - name: Use Node.js ${{ matrix.node-version }}
       uses: actions/setup-node@v1
       with:
         node-version: ${{ matrix.node-version }}
+    - run: npm install
     - run: npm install sequelize@${{ matrix.sequelize-version }}
       if: matrix.sequelize-version != null
-    - name: run ci
-      run: |
-        npm install
-        npm run lint
-        npm run test -- --coverage
-      env:
-        CI: true
+    - run: npm run lint
+      if: matrix.sequelize-version == null
+    - run: npm run test -- --coverage
     - name: Coverage
       uses: codecov/codecov-action@v1
       if: matrix.node-version == '12.x' && matrix.sequelize-version == null

test/storage/sequelize.test.ts

@@ -10,7 +10,12 @@ import jetpack = require('fs-jetpack');
 
 // TODO [>=3.0.0]: Investigate whether we are mis-using `model.describe()` here, and get rid of `any`.
 // See https://github.com/sequelize/umzug/pull/226 and https://github.com/sequelize/sequelize/issues/12296 for details
-const describeModel = (model: any) => model.describe();
+const describeModel = (model: any) =>
+	model.describe().then(d => {
+		// FIXME [sequelize@>=6] remove this hack when only sequelize>=6 is supported
+		Object.keys(d).forEach(k => delete d[k].unique);
+		return d;
+	});
 
 describe('sequelize', () => {
 	jetpack.cwd(__dirname).dir('tmp', { empty: true });