Skip to content

Conversation

maikhanhbui
Copy link
Contributor

@maikhanhbui maikhanhbui commented Mar 15, 2023

This PR allows partial support for filtering against window functions, which was added in Django 4.2.

Fixes the following Django 4.2 tests:

expressions_window.tests.WindowFunctionTests.test_exclude,
expressions_window.tests.WindowFunctionTests.test_filter,
expressions_window.tests.WindowFunctionTests.test_filter_alias,
expressions_window.tests.WindowFunctionTests.test_filter_column_ref_rhs,
expressions_window.tests.WindowFunctionTests.test_filter_conditional_annotation,
expressions_window.tests.WindowFunctionTests.test_filter_conditional_expression,
expressions_window.tests.WindowFunctionTests.test_filter_count,
expressions_window.tests.WindowFunctionTests.test_filter_select_related,
expressions_window.tests.WindowFunctionTests.test_filter_values,
expressions_window.tests.WindowFunctionTests.test_heterogeneous_filter,

The following Django 4.2 WindowFunction test failure still remains open:

expressions_window.tests.WindowFunctionTests.test_limited_filter

@maikhanhbui maikhanhbui merged commit 5a630ba into django42 Mar 23, 2023
@maikhanhbui maikhanhbui deleted the fix-windows-tests branch March 23, 2023 23:46
mShan0 added a commit that referenced this pull request May 31, 2023
* Allow Django 4.2 (#227)

* Allow Django 4.2

* allow Django 4.2

* Fix errors with raising FullResultSet exception and with alter_column_type_sql() and collate_sql() functions (#229)

* fix error with raising fullresultset

* add django4.2 condition

* fix alter_column_type_sql and collate_sql to take 2 additional arguments

* delete argument 'old_rel_collation'

* fix arguments names

* fix last_executed_query() to properly replace placeholders with params (#234)

* disable allows_group_by_select_index

* unskip old tests

* unskip some tests

* skip more tests

* Use latest Django 4.2 beta for tox (#238)

* use 4.2 rc1 branch (#240)

* allow partial support for filtering against window functions (#239)

* add subsecond support to Now() (#242)

* assign value to display_size (#244)

* add latest django 4.2 branch to ci

* raise an error when batch_size is zero. (#259)

* replicate get or create test for mssql (#265)

* Add skipped tests to Django 4.2 (#268)

* skip django 4.2 failing tests

* skip schema test

* skip aggregate annotation pruning test

---------

Co-authored-by: mShan0 <mark.shan19@gmail.com>

* syntax fix

* ci fix

* bump version to 1.3

---------

Co-authored-by: Khanh Bui <85855766+khanhmaibui@users.noreply.github.com>
Co-authored-by: Khanh Bui <khanhmb815@gmail.com>
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.

2 participants