Skip to content

CHORE: Unskip some schema tests#293

Open
mShan0 wants to merge 11 commits into
devfrom
unskip-schema-tests
Open

CHORE: Unskip some schema tests#293
mShan0 wants to merge 11 commits into
devfrom
unskip-schema-tests

Conversation

@mShan0
Copy link
Copy Markdown
Contributor

@mShan0 mShan0 commented Aug 16, 2023

These tests are now passing:

    'schema.tests.SchemaTests.test_alter_implicit_id_to_explicit',
    'schema.tests.SchemaTests.test_alter_int_pk_to_bigautofield_pk',
    'schema.tests.SchemaTests.test_alter_pk_with_self_referential_field',
    'schema.tests.SchemaTests.test_unique_together_with_fk',
    'schema.tests.SchemaTests.test_unique_together_with_fk_with_existing_index',

@mShan0 mShan0 changed the title unskips some schema tests unskip some schema tests Aug 16, 2023
Copilot AI review requested due to automatic review settings May 21, 2026 12:53
@bewithgaurav bewithgaurav changed the title unskip some schema tests CHORE: Unskip some schema tests May 21, 2026
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the Django test runner configuration for mssql-django by unskipping (removing from EXCLUDED_TESTS) several upstream Django schema tests that are now expected to pass on SQL Server.

Changes:

  • Removed several schema.tests.SchemaTests.* entries from testapp/settings.py’s EXCLUDED_TESTS list so they run again.
  • Made a whitespace-only change in test.sh (introduced a space-only line).

Reviewed changes

Copilot reviewed 1 out of 2 changed files in this pull request and generated 2 comments.

File Description
testapp/settings.py Unskips selected Django schema tests by removing them from EXCLUDED_TESTS.
test.sh Whitespace-only change near the set -e header.

Comment thread testapp/settings.py
Comment thread test.sh
@bewithgaurav
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

This was referenced May 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants