Merge pull request #210 from bjohansebas/fix-migration

Fix rollback migration

Author: UlisesGascon

.github/workflows/ci.yml

@@ -51,5 +51,8 @@ jobs:
     - name: Run tests
       run: npm run test:ci
 
+    - name: Run rollback
+      run: npm run db:rollback
+
     - name: Build Docker image
       run: docker build -t visionboard .

src/database/migrations/1737629097303_add_resources_for_compliance_checks.js

@@ -4,11 +4,11 @@ exports.up = async (knex) => {
     table.integer('compliance_check_id')
       .unsigned()
       .notNullable()
-      .references('id').inTable('compliance_checks')
+      .references('id').inTable('compliance_checks').onDelete('CASCADE').onUpdate('CASCADE')
     table.integer('compliance_check_resource_id')
       .unsigned()
       .notNullable()
-      .references('id').inTable('compliance_checks_resources')
+      .references('id').inTable('compliance_checks_resources').onDelete('CASCADE').onUpdate('CASCADE')
 
     table.timestamp('created_at').defaultTo(knex.fn.now()).notNullable()
   })

src/database/migrations/1737629113552_populate_compliance_check_resources.js

@@ -1410,5 +1410,5 @@ exports.up = async (knex) => {
 
 exports.down = async (knex) => {
   await knex('resources_for_compliance_checks').truncate()
-  await knex('compliance_checks_resources').truncate()
+  await knex.raw('TRUNCATE TABLE compliance_checks_resources CASCADE')
 }