fix: fix prisma#24068 (#5515)

[ORM-1099](https://linear.app/prisma-company/issue/ORM-1099/24068-sqlserver-migrate-issues)

We've been discarding the namespace in a few places. I kept the
namespace wherever it's available even if it's currently not used (mysql
and sqlite).

Fixes prisma/prisma#24068

Author: jacek-prisma

schema-engine/connectors/sql-schema-connector/src/flavour/mssql/destructive_change_checker.rs

@@ -76,7 +76,7 @@ impl DestructiveChangeCheckerFlavour for MssqlDestructiveChangeCheckerFlavour {
                 plan.push_warning(
                     SqlMigrationWarningCheck::RiskyCast {
                         table: columns.previous.table().name().to_owned(),
-                        namespace: None,
+                        namespace: columns.previous.table().namespace().map(str::to_owned),
                         column: columns.previous.name().to_owned(),
                         previous_type,
                         next_type,
@@ -88,7 +88,7 @@ impl DestructiveChangeCheckerFlavour for MssqlDestructiveChangeCheckerFlavour {
                 plan.push_warning(
                     SqlMigrationWarningCheck::NotCastable {
                         table: columns.previous.table().name().to_owned(),
-                        namespace: None,
+                        namespace: columns.previous.table().namespace().map(str::to_owned),
                         column: columns.previous.name().to_owned(),
                         previous_type: format!("{:?}", columns.previous.column_type_family()),
                         next_type: format!("{:?}", columns.next.column_type_family()),
@@ -134,7 +134,7 @@ impl DestructiveChangeCheckerFlavour for MssqlDestructiveChangeCheckerFlavour {
                 SqlMigrationWarningCheck::DropAndRecreateColumn {
                     column: columns.previous.name().to_owned(),
                     table: columns.previous.table().name().to_owned(),
-                    namespace: None,
+                    namespace: columns.previous.table().namespace().map(str::to_owned),
                 },
                 step_index,
             )

schema-engine/connectors/sql-schema-connector/src/flavour/mysql/destructive_change_checker.rs

@@ -74,7 +74,7 @@ impl DestructiveChangeCheckerFlavour for MysqlDestructiveChangeCheckerFlavour {
                 plan.push_warning(
                     SqlMigrationWarningCheck::RiskyCast {
                         table: columns.previous.table().name().to_owned(),
-                        namespace: None,
+                        namespace: columns.previous.table().namespace().map(str::to_owned),
                         column: columns.previous.name().to_owned(),
                         previous_type,
                         next_type,
@@ -86,7 +86,7 @@ impl DestructiveChangeCheckerFlavour for MysqlDestructiveChangeCheckerFlavour {
                 plan.push_warning(
                     SqlMigrationWarningCheck::NotCastable {
                         table: columns.previous.table().name().to_owned(),
-                        namespace: None,
+                        namespace: columns.previous.table().namespace().map(str::to_owned),
                         column: columns.previous.name().to_owned(),
                         previous_type,
                         next_type,
@@ -133,7 +133,7 @@ impl DestructiveChangeCheckerFlavour for MysqlDestructiveChangeCheckerFlavour {
                 SqlMigrationWarningCheck::DropAndRecreateColumn {
                     column: columns.previous.name().to_owned(),
                     table: columns.previous.table().name().to_owned(),
-                    namespace: None,
+                    namespace: columns.previous.table().namespace().map(str::to_owned),
                 },
                 step_index,
             )

schema-engine/connectors/sql-schema-connector/src/flavour/sqlite/destructive_change_checker.rs

@@ -61,7 +61,7 @@ impl DestructiveChangeCheckerFlavour for SqliteDestructiveChangeCheckerFlavour {
                 plan.push_warning(
                     SqlMigrationWarningCheck::RiskyCast {
                         table: columns.previous.table().name().to_owned(),
-                        namespace: None,
+                        namespace: columns.previous.table().namespace().map(str::to_owned),
                         column: columns.previous.name().to_owned(),
                         previous_type: format!("{:?}", columns.previous.column_type_family()),
                         next_type: format!("{:?}", columns.next.column_type_family()),

schema-engine/sql-migration-tests/tests/migrations/mssql/multi_schema.rs

@@ -1075,6 +1075,42 @@ fn multi_schema_tests(_api: TestApi) {
             }),
             skip: None,
         },
+        TestData {
+            name: "issue prisma/prisma#24068",
+            description: "Modify the primary key of a table in a namespace",
+            schema: Schema {
+                common: base_schema.into(),
+                first: indoc! {r#"
+                    model Report {
+                      id String @id
+
+                      @@schema("two")
+                    }"#}
+                .into(),
+                second: Some(indoc! {r#"
+                    model Report {
+                      id String @id @unique @db.NVarChar(450)
+
+                      @@schema("two")
+                    }"#}
+                .into())
+            },
+            namespaces,
+            schema_push: SchemaPush::PushAnd(WithSchema::First, &SchemaPush::PushCustomAnd(CustomPushStep {
+                warnings: &["A unique constraint covering the columns `[id]` on the table `Report` will be added. If there are existing duplicate values, this will fail."],
+                errors: &[],
+                with_schema: WithSchema::Second,
+                executed_steps: ExecutedSteps::NonZero,
+            }, &SchemaPush::Done)),
+            assertion: Box::new(|assert| {
+                assert
+                    .assert_has_table_with_ns("two", "Report")
+                    .assert_table_with_ns("two", "Report", |table| {
+                        table.assert_column("id", |column| column.assert_is_required().assert_type_is_string())
+                    });
+            }),
+            skip: None,
+        },
     ];
 
     // traverse_ is always the answer