Exclude __EFMigrationsLock from SQLite scaffolding

[TomasMoralesBarr]

Summary

Fixes #38148

When scaffolding a SQLite database that has had EF migrations applied, the __EFMigrationsLock table was incorrectly included in the scaffolded model. This table is EF-internal and should be excluded just like __EFMigrationsHistory.

Root Cause

SqliteDatabaseModelFactory.GetTables explicitly excluded __EFMigrationsHistory via HistoryRepository.DefaultTableName, but did not exclude __EFMigrationsLock, which is created by SqliteHistoryRepository to implement distributed locking during migrations on SQLite.

Changes

• Added DefaultLockTableName public const to SqliteHistoryRepository to avoid hardcoding the table name in multiple places.
• Added __EFMigrationsLock to the exclusion list in SqliteDatabaseModelFactory.GetTables.
• Added a test in SqliteDatabaseModelFactoryTest to verify that EF-internal tables are not scaffolded.

• I've read the guidelines for contributing and seen the walkthrough
• I've posted a comment on an issue with a detailed description of how I am planning to contribute and got approval from a member of the team
• The code builds and tests pass locally (also verified by our automated build checks)
• Commit messages follow this format:

        Summary of the changes
        - Detail 1
        - Detail 2

        Fixes #bugnumber

• Tests for the changes have been added (for bug fixes / features)
• Code follows the same patterns and style as existing code in this repo

[Copilot]

Pull request overview

This PR fixes SQLite reverse-engineering (scaffolding) to exclude EF-internal migrations infrastructure tables, specifically __EFMigrationsLock, matching the existing behavior for __EFMigrationsHistory (Fixes #38148).

Changes:

• Exclude __EFMigrationsLock from SqliteDatabaseModelFactory.GetTables results.
• Introduce a shared constant for the default lock table name in SqliteHistoryRepository.
• Add a functional test verifying EF-internal tables aren’t scaffolded.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 4 comments.

File	Description
test/EFCore.Sqlite.FunctionalTests/Scaffolding/SqliteDatabaseModelFactoryTest.cs	Adds a regression test ensuring __EFMigrationsLock isn’t scaffolded.
src/EFCore.Sqlite.Core/Scaffolding/Internal/SqliteDatabaseModelFactory.cs	Updates the sqlite_master query exclusion list to filter out the lock table.
src/EFCore.Sqlite.Core/Migrations/Internal/SqliteHistoryRepository.cs	Adds DefaultLockTableName constant and reuses it for LockTableName.

[TomasMoralesBarr]

@dotnet-policy-service agree

[roji]

Thanks, LGTM.