feat(test): Add double migration test (#827)

We saw some bugs when the migration code wasn't reliable and, if we ran
safe migration after forced one, there would be conflicts reported
(where they shouldn't have been).
This extra test ensures that the destination is in sync what it scans
from database and what is taken from spec.

Author: candiduslynx

plugins/destination/plugin_testing_migrate.go

@@ -15,6 +15,7 @@ import (
 	"github.com/cloudquery/plugin-sdk/v2/types"
 	"github.com/google/uuid"
 	"github.com/rs/zerolog"
+	"github.com/stretchr/testify/require"
 )
 
 func tableUUIDSuffix() string {
@@ -241,4 +242,18 @@ func (*PluginTestSuite) destinationPluginTestMigrate(
 			t.Fatal(err)
 		}
 	})
+
+	t.Run("double_migration", func(t *testing.T) {
+		tableName := "double_migration_" + tableUUIDSuffix()
+		table := schema.CQSchemaToArrow(testdata.TestTable(tableName))
+
+		p := newPlugin()
+		require.NoError(t, p.Init(ctx, logger, spec))
+		require.NoError(t, p.Migrate(ctx, schema.Schemas{table}))
+
+		nonForced := spec
+		nonForced.MigrateMode = specs.MigrateModeSafe
+		require.NoError(t, p.Init(ctx, logger, nonForced))
+		require.NoError(t, p.Migrate(ctx, schema.Schemas{table}))
+	})
 }