Fix WindowFrame::new with order_by (apache#16537)

* Fix WindowFrame::new with order_by

Before the change, the frame constructed with
`WindowFrame::new(Some(true))` would not be fully functional, because
of the bound being of unexpected type.

* Change assert helper param order

Putting expected data second allows for natural inlining of expected
values.

(cherry picked from commit 20a723b)

Author: findepi