[java] Fix v*Network.java conditions (#14585)

mk868 · pujagani · web-flow · commit 274f43f75ed5 · 2024-10-14T11:23:37.000+05:30
Co-authored-by: Puja Jagani &lt;puja.jagani93@gmail.com&gt;
diff --git a/java/src/org/openqa/selenium/devtools/v127/v127Network.java b/java/src/org/openqa/selenium/devtools/v127/v127Network.java
@@ -127,7 +127,7 @@ public Either<HttpRequest, HttpResponse> createSeMessages(RequestPaused pausedRe
       } catch (DevToolsException e) {
         // Redirects don't seem to have bodies
         int code = pausedReq.getResponseStatusCode().orElse(HTTP_OK);
-        if (code < 300 && code > 399) {
+        if (code >= 300 && code <= 399) {
           LOG.warning("Unable to get body for request id " + pausedReq.getRequestId());
         }
 
diff --git a/java/src/org/openqa/selenium/devtools/v128/v128Network.java b/java/src/org/openqa/selenium/devtools/v128/v128Network.java
@@ -127,7 +127,7 @@ public Either<HttpRequest, HttpResponse> createSeMessages(RequestPaused pausedRe
       } catch (DevToolsException e) {
         // Redirects don't seem to have bodies
         int code = pausedReq.getResponseStatusCode().orElse(HTTP_OK);
-        if (code < 300 && code > 399) {
+        if (code >= 300 && code <= 399) {
           LOG.warning("Unable to get body for request id " + pausedReq.getRequestId());
         }
 
diff --git a/java/src/org/openqa/selenium/devtools/v129/v129Network.java b/java/src/org/openqa/selenium/devtools/v129/v129Network.java
@@ -127,7 +127,7 @@ public Either<HttpRequest, HttpResponse> createSeMessages(RequestPaused pausedRe
       } catch (DevToolsException e) {
         // Redirects don't seem to have bodies
         int code = pausedReq.getResponseStatusCode().orElse(HTTP_OK);
-        if (code < 300 && code > 399) {
+        if (code >= 300 && code <= 399) {
           LOG.warning("Unable to get body for request id " + pausedReq.getRequestId());
         }
 
diff --git a/java/src/org/openqa/selenium/devtools/v85/V85Network.java b/java/src/org/openqa/selenium/devtools/v85/V85Network.java
@@ -137,7 +137,7 @@ public Either<HttpRequest, HttpResponse> createSeMessages(RequestPaused pausedRe
       } catch (DevToolsException e) {
         // Redirects don't seem to have bodies
         int code = pausedReq.getResponseStatusCode().orElse(HTTP_OK);
-        if (code < 300 && code > 399) {
+        if (code >= 300 && code <= 399) {
           LOG.warning("Unable to get body for request id " + pausedReq.getRequestId());
         }
 

Original file line number	Diff line number	Diff line change
`@@ -127,7 +127,7 @@ public Either<HttpRequest, HttpResponse> createSeMessages(RequestPaused pausedRe`
`127`	`127`	`} catch (DevToolsException e) {`
`128`	`128`	`// Redirects don't seem to have bodies`
`129`	`129`	`int code = pausedReq.getResponseStatusCode().orElse(HTTP_OK);`
`130`		`- if (code < 300 && code > 399) {`
	`130`	`+ if (code >= 300 && code <= 399) {`
`131`	`131`	`LOG.warning("Unable to get body for request id " + pausedReq.getRequestId());`
`132`	`132`	`}`
`133`	`133`
Original file line number	Diff line number	Diff line change
`@@ -137,7 +137,7 @@ public Either<HttpRequest, HttpResponse> createSeMessages(RequestPaused pausedRe`
`137`	`137`	`} catch (DevToolsException e) {`
`138`	`138`	`// Redirects don't seem to have bodies`
`139`	`139`	`int code = pausedReq.getResponseStatusCode().orElse(HTTP_OK);`
`140`		`- if (code < 300 && code > 399) {`
	`140`	`+ if (code >= 300 && code <= 399) {`
`141`	`141`	`LOG.warning("Unable to get body for request id " + pausedReq.getRequestId());`
`142`	`142`	`}`
`143`	`143`