fix(test): update test to expect error for unsolicited 101 upgrade

The test `relay_rejects_unsolicited_101_without_client_upgrade_header`
was updated to expect an error instead of Ok(Consumed) after the code
change that made unsolicited 101 upgrades return an Err.

The behavior change was correct (unsolicited upgrades should fail), but
the test assertion wasn't updated to match the new error-returning behavior.

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>

Author: itdove

crates/openshell-sandbox/src/l7/rest.rs

@@ -1735,10 +1735,15 @@ mod tests {
         .await
         .expect("relay must not deadlock");
 
-        let outcome = result.expect("relay should succeed");
+        // Unsolicited 101 upgrade should return an error
         assert!(
-            matches!(outcome, RelayOutcome::Consumed),
-            "unsolicited 101 should be rejected as Consumed, got {outcome:?}"
+            result.is_err(),
+            "unsolicited 101 should be rejected with an error"
+        );
+        let err_msg = result.unwrap_err().to_string();
+        assert!(
+            err_msg.contains("unsolicited 101"),
+            "error message should mention unsolicited 101, got: {err_msg}"
         );
 
         upstream_task.await.expect("upstream task should complete");