test: allow assert_op_expr in macro_argument_binding and use it directly

Add `assert_op_expr` to the rule's `allow_extra` list: the macro evaluates
each operand exactly once, so it carries no drop-or-duplicate hazard, unlike
the `debug_assert_op*` pair already in `deny_extra`. With the exemption in
place, pass the position expressions to `assert_op_expr!` directly instead of
binding them to locals first.

https://claude.ai/code/session_01Nj3xEp1eoDKRp1MUCxNQRQ

Author: claude

dylint.toml

@@ -26,6 +26,7 @@ prefix = [
 
 ["perfectionist::macro_argument_binding"]
 deny_extra = ["debug_assert_op", "debug_assert_op_expr"]
+allow_extra = ["assert_op_expr"]
 
 ["perfectionist::single_letter_closure_param"]
 extra_trivial_callback_methods = ["sort_reflection_by"]

tests/json.rs

@@ -4,7 +4,7 @@
 pub mod _utils;
 pub use _utils::*;
 
-use assert_cmp::assert_op;
+use assert_cmp::assert_op_expr;
 use command_extra::CommandExtra;
 use parallel_disk_usage::{
     bytes_format::BytesFormat,
@@ -252,12 +252,8 @@ fn json_input_no_sort() {
     // `--no-sort` preserves the ascending input order `a, b, c`, whereas the
     // default sorts by descending size, so their relative positions flip.
     let position = |text: &str, name: &str| text.find(name).expect("entry must be present");
-    let actual_a = position(&actual, "a");
-    let actual_c = position(&actual, "c");
-    assert_op!(actual_a > actual_c);
-    let sorted_a = position(&sorted, "a");
-    let sorted_c = position(&sorted, "c");
-    assert_op!(sorted_a < sorted_c);
+    assert_op_expr!(position(&actual, "a"), >, position(&actual, "c"));
+    assert_op_expr!(position(&sorted, "a"), <, position(&sorted, "c"));
 }
 
 #[test]