iamsanjay commented on PR #2373:
URL: https://github.com/apache/solr/pull/2373#issuecomment-2094115387

   This test failed again:-
   ``` 
   44056 INFO 
(TEST-JWTAuthPluginIntegrationTest.mockOAuth2Server-seed#[F98DD07D54DABA29]) 
[n: c: s: r: x: t:] o.a.s.SolrTestCaseJ4 ###Ending mockOAuth2Server    6149 > 
org.junit.ComparisonFailure: Should have received 401 code expected:<[401]> but 
was:<[200]>        6150 > at 
__randomizedtesting.SeedInfo.seed([F98DD07D54DABA29:7E8D312D316646C5]:0)    
6151 > at 
   org.junit.Assert.assertEquals(Assert.java:117)    6152 > at 
   
org.apache.solr.security.jwt.JWTAuthPluginIntegrationTest.mockOAuth2Server(JWTAuthPluginIntegrationTest.java:145)
    
   6153 > at 
java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)  
  6154 > at 
   
java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
    6155 > at 
   
java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
    6156 > at 
   java.base/java.lang.reflect.Method.invoke(Method.java:566)    6157 > at 
   
com.carrotsearch.randomizedtesting.RandomizedRunner.invoke(RandomizedRunner.java:1758)
    6158 > at 
   
com.carrotsearch.randomizedtesting.RandomizedRunner$8.evaluate(RandomizedRunner.java:946)
    6159 > at 
   
com.carrotsearch.randomizedtesting.RandomizedRunner$9.evaluate(RandomizedRunner.java:982)
    6160 > at 
   
com.carrotsearch.randomizedtesting.RandomizedRunner$10.evaluate(RandomizedRunner.java:996)
    6161 > at 
   
com.carrotsearch.randomizedtesting.rules.SystemPropertiesRestoreRule$1.evaluate(SystemPropertiesRestoreRule.java:80)
    
   6162 > at org.junit.rules.RunRules.evaluate(RunRules.java:20)   
   ```
    
   
   Also there are bunch of other classMethod failures, because of resource 
leak. Once assert failed it did not shutdown the cluster. Two ways to solved 
it, either the whole test block should be inside the try block and in finally 
we should shutdown the cluster. Or, use the existing cluster reference from 
SolrCloudTestCase and that will take care of shutting down the cluster.


-- 
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: issues-unsubscr...@solr.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@solr.apache.org
For additional commands, e-mail: issues-h...@solr.apache.org

Reply via email to