You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@storm.apache.org by GitBox <gi...@apache.org> on 2020/06/26 22:55:43 UTC

[GitHub] [storm] RuiLi8080 opened a new pull request #3298: [STORM-3663] move org.mockito.plugins.MockMaker to test level resource

RuiLi8080 opened a new pull request #3298:
URL: https://github.com/apache/storm/pull/3298


   ## What is the purpose of the change
   
   We are seeing Mockito related error building topology with latest storm version.
   ```
   Configuring TestNG with: TestNG652Configurator
   SLF4J: Class path contains multiple SLF4J bindings.
   SLF4J: Found binding in [jar:file:/Users/rli01/.m2/repository/org/slf4j/slf4j-log4j12/1.7.10/slf4j-log4j12-1.7.10.jar!/org/slf4j/impl/StaticLoggerBinder.class]
   SLF4J: Found binding in [jar:file:/Users/rli01/.m2/repository/org/apache/logging/log4j/log4j-slf4j-impl/2.11.2/log4j-slf4j-impl-2.11.2.jar!/org/slf4j/impl/StaticLoggerBinder.class]
   SLF4J: See http://www.slf4j.org/codes.html#multiple_bindings for an explanation.
   SLF4J: Actual binding is of type [org.slf4j.impl.Log4jLoggerFactory]
   Tests run: 37, Failures: 12, Errors: 0, Skipped: 0, Time elapsed: 1.135 sec <<< FAILURE! - in TestSuite
   testCycling(fetl.bullet.propane.PropaneSpoutTest)  Time elapsed: 0.147 sec  <<< FAILURE!
   java.lang.ExceptionInInitializerError: null
   	at org.mockito.internal.configuration.plugins.Plugins.getStackTraceCleanerProvider(Plugins.java:17)
   	at org.mockito.internal.exceptions.stacktrace.StackTraceFilter.<clinit>(StackTraceFilter.java:21)
   	at org.mockito.internal.exceptions.stacktrace.ConditionalStackTraceFilter.<init>(ConditionalStackTraceFilter.java:17)
   	at org.mockito.exceptions.base.MockitoException.filterStackTrace(MockitoException.java:41)
   	at org.mockito.exceptions.base.MockitoException.<init>(MockitoException.java:30)
   	at org.mockito.exceptions.misusing.MockitoConfigurationException.<init>(MockitoConfigurationException.java:18)
   	at org.mockito.internal.configuration.plugins.PluginLoader.loadImpl(PluginLoader.java:66)
   	at org.mockito.internal.configuration.plugins.PluginLoader.loadPlugin(PluginLoader.java:24)
   	at org.mockito.internal.configuration.plugins.PluginRegistry.<init>(PluginRegistry.java:12)
   	at org.mockito.internal.configuration.plugins.Plugins.<clinit>(Plugins.java:11)
   	at org.mockito.internal.util.MockUtil.<clinit>(MockUtil.java:24)
   	at org.mockito.internal.MockitoCore.<init>(MockitoCore.java:44)
   	at org.mockito.Mockito.<clinit>(Mockito.java:975)
   	at fetl.bullet.propane.PropaneSpoutTest.createKafkaConsumer(PropaneSpoutTest.java:71)
   	at fetl.bullet.propane.PropaneSpoutTest.createKafkaConsumer(PropaneSpoutTest.java:87)
   	at fetl.bullet.propane.PropaneSpoutTest.testCycling(PropaneSpoutTest.java:170)
   ```
   
   After making the changes in this PR, topology build succeeded with latest local storm dependencies.
   
   ## How was the change tested
    
   Build storm locally and rebuild topology depending on it.


----------------------------------------------------------------
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.

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



[GitHub] [storm] Ethanlm merged pull request #3298: [STORM-3663] move org.mockito.plugins.MockMaker to test level resource

Posted by GitBox <gi...@apache.org>.
Ethanlm merged pull request #3298:
URL: https://github.com/apache/storm/pull/3298


   


----------------------------------------------------------------
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.

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