gerlowskija commented on code in PR #4721:
URL: https://github.com/apache/solr/pull/4721#discussion_r3749073267


##########
solr/core/src/test/org/apache/solr/handler/component/QueryElevationComponentTest.java:
##########
@@ -172,11 +163,14 @@ public void testFq() throws Exception {
       // exclude docs 1 and 3 even though those docs are elevated
       assertQ(
           "",
-          req(
-              CommonParams.Q, "ZZZZ",
-              CommonParams.QT, "/elevate",
-              CommonParams.FL, "id, score, [elevated]",
-              CommonParams.FQ, "str_s:b"),
+          reqWithPath(
+              "/elevate",
+              CommonParams.Q,

Review Comment:
   I keep trying to combine the param name+value into a single line in these 
invocations, but `tidy` really doesn't like it.
   
   Not sure if it's a line length thing or something else; kindof a bummer : (



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to