Skip to content
This repository has been archived by the owner on May 7, 2020. It is now read-only.

[Test failures] Wemo test failure #4927

Closed
sjsf opened this issue Jan 15, 2018 · 0 comments
Closed

[Test failures] Wemo test failure #4927

sjsf opened this issue Jan 15, 2018 · 0 comments
Labels
Binding-Wemo Test

Comments

@sjsf
Copy link
Contributor

sjsf commented Jan 15, 2018

As seen here:
https://travis-ci.org/eclipse/smarthome/builds/329017654?utm_source=github_status&utm_medium=notification

assert that thing updates channels at start(org.eclipse.smarthome.binding.wemo.handler.test.WemoLightHandlerOSGiTest)  Time elapsed: 11.127 sec  <<< ERROR!
java.util.ConcurrentModificationException: null
	at java.util.HashMap$HashIterator.nextNode(HashMap.java:1442)
	at java.util.HashMap$KeyIterator.next(HashMap.java:1466)
	at com.google.common.collect.AbstractMapBasedMultimap$WrappedCollection$WrappedIterator.next(AbstractMapBasedMultimap.java:486)
	at java.util.AbstractCollection.addAll(AbstractCollection.java:343)
	at java.util.HashSet.<init>(HashSet.java:120)
	at sun.reflect.GeneratedConstructorAccessor31.newInstance(Unknown Source)
	at sun.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:45)
	at java.lang.reflect.Constructor.newInstance(Constructor.java:423)
	at org.codehaus.groovy.reflection.CachedConstructor.invoke(CachedConstructor.java:77)
	at org.codehaus.groovy.runtime.callsite.ConstructorSite$ConstructorSiteNoUnwrapNoCoerce.callConstructor(ConstructorSite.java:102)
	at org.codehaus.groovy.runtime.callsite.AbstractCallSite.callConstructor(AbstractCallSite.java:190)
	at org.eclipse.smarthome.binding.wemo.test.GenericWemoOSGiTest.getThingHandlers(GenericWemoOSGiTest.groovy:254)
	at sun.reflect.GeneratedMethodAccessor35.invoke(Unknown Source)
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.lang.reflect.Method.invoke(Method.java:498)
	at org.codehaus.groovy.runtime.callsite.PogoMetaMethodSite$PogoCachedMethodSiteNoUnwrapNoCoerce.invoke(PogoMetaMethodSite.java:207)
	at org.codehaus.groovy.runtime.callsite.PogoMetaMethodSite.callCurrent(PogoMetaMethodSite.java:56)
	at org.codehaus.groovy.runtime.callsite.AbstractCallSite.callCurrent(AbstractCallSite.java:141)
	at org.eclipse.smarthome.binding.wemo.test.GenericWemoOSGiTest.getThingHandler(GenericWemoOSGiTest.groovy:241)
	at sun.reflect.GeneratedMethodAccessor29.invoke(Unknown Source)
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.lang.reflect.Method.invoke(Method.java:498)
	at org.codehaus.groovy.reflection.CachedMethod.invoke(CachedMethod.java:90)
	at groovy.lang.MetaMethod.doMethodInvoke(MetaMethod.java:324)
	at org.codehaus.groovy.runtime.metaclass.ClosureMetaClass.invokeMethod(ClosureMetaClass.java:368)
	at groovy.lang.MetaClassImpl.invokeMethod(MetaClassImpl.java:1016)
	at org.codehaus.groovy.runtime.callsite.PogoMetaClassSite.callCurrent(PogoMetaClassSite.java:66)
	at org.codehaus.groovy.runtime.callsite.AbstractCallSite.callCurrent(AbstractCallSite.java:141)
	at org.eclipse.smarthome.binding.wemo.test.GenericWemoLightOSGiTest$_removeThing_closure2.doCall(GenericWemoLightOSGiTest.groovy:121)
	at org.eclipse.smarthome.binding.wemo.test.GenericWemoLightOSGiTest$_removeThing_closure2.doCall(GenericWemoLightOSGiTest.groovy)
	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.lang.reflect.Method.invoke(Method.java:498)
	at org.codehaus.groovy.reflection.CachedMethod.invoke(CachedMethod.java:90)
	at groovy.lang.MetaMethod.doMethodInvoke(MetaMethod.java:324)
	at org.codehaus.groovy.runtime.metaclass.ClosureMetaClass.invokeMethod(ClosureMetaClass.java:278)
	at groovy.lang.MetaClassImpl.invokeMethod(MetaClassImpl.java:1016)
	at org.codehaus.groovy.runtime.callsite.PogoMetaClassSite.call(PogoMetaClassSite.java:39)
	at org.codehaus.groovy.runtime.callsite.CallSiteArray.defaultCall(CallSiteArray.java:45)
	at org.codehaus.groovy.runtime.callsite.PogoMetaClassSite.call(PogoMetaClassSite.java:54)
	at org.codehaus.groovy.runtime.callsite.AbstractCallSite.call(AbstractCallSite.java:112)
	at org.eclipse.smarthome.test.OSGiTest.waitForAssert(OSGiTest.groovy:243)
	at sun.reflect.GeneratedMethodAccessor14.invoke(Unknown Source)
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.lang.reflect.Method.invoke(Method.java:498)
	at org.codehaus.groovy.runtime.callsite.PogoMetaMethodSite$PogoCachedMethodSiteNoUnwrap.invoke(PogoMetaMethodSite.java:187)
	at org.codehaus.groovy.runtime.callsite.PogoMetaMethodSite.callCurrent(PogoMetaMethodSite.java:56)
	at org.codehaus.groovy.runtime.callsite.CallSiteArray.defaultCallCurrent(CallSiteArray.java:49)
	at org.codehaus.groovy.runtime.callsite.PogoMetaMethodSite.callCurrent(PogoMetaMethodSite.java:61)
	at org.codehaus.groovy.runtime.callsite.AbstractCallSite.callCurrent(AbstractCallSite.java:153)
	at org.eclipse.smarthome.test.OSGiTest.waitForAssert(OSGiTest.groovy:223)
	at org.eclipse.smarthome.test.OSGiTest.waitForAssert(OSGiTest.groovy)
	at sun.reflect.GeneratedMethodAccessor18.invoke(Unknown Source)
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.lang.reflect.Method.invoke(Method.java:498)
	at org.codehaus.groovy.runtime.callsite.PogoMetaMethodSite$PogoCachedMethodSiteNoUnwrapNoCoerce.invoke(PogoMetaMethodSite.java:207)
	at org.codehaus.groovy.runtime.callsite.PogoMetaMethodSite.callCurrent(PogoMetaMethodSite.java:56)
	at org.codehaus.groovy.runtime.callsite.AbstractCallSite.callCurrent(AbstractCallSite.java:141)
	at org.eclipse.smarthome.binding.wemo.test.GenericWemoLightOSGiTest.removeThing(GenericWemoLightOSGiTest.groovy:120)
	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.lang.reflect.Method.invoke(Method.java:498)
	at org.codehaus.groovy.runtime.callsite.PogoMetaMethodSite$PogoCachedMethodSiteNoUnwrapNoCoerce.invoke(PogoMetaMethodSite.java:207)
	at org.codehaus.groovy.runtime.callsite.PogoMetaMethodSite.callCurrent(PogoMetaMethodSite.java:56)
	at org.codehaus.groovy.runtime.callsite.AbstractCallSite.callCurrent(AbstractCallSite.java:137)
	at org.eclipse.smarthome.binding.wemo.handler.test.WemoLightHandlerOSGiTest.tearDown(WemoLightHandlerOSGiTest.groovy:60)

Full log: log.txt
triller-telekom added a commit to triller-telekom/smarthome that referenced this issue May 17, 2018
@triller-telekom
Wemo Binding: Converted groovy tests to Java
04d86d4 
Within the conversion I also did the following:
- Made WemoHttpCall non-static to mock them inside tests
- All Handler inherit from AbstractWemoHandler which has a setter for WemoHttpCall
- Removed necessity for servlets in tests by checking the content sent to the server
- Removed dependecies to the ItemRegistry

Fixes eclipse-archived#4927
Fixes eclipse-archived#4071

Signed-off-by: Stefan Triller <stefan.triller@telekom.de>
@triller-telekom triller-telekom mentioned this issue May 17, 2018
Merged
htreu pushed a commit that referenced this issue May 18, 2018
@triller-telekom @htreu
Wemo Binding: Converted Groovy tests to Java (#5586)
475c5d1 
* Wemo Binding: Converted groovy tests to Java

Within the conversion I also did the following:
- Made WemoHttpCall non-static to mock them inside tests
- All Handler inherit from AbstractWemoHandler which has a setter for WemoHttpCall
- Removed necessity for servlets in tests by checking the content sent to the server
- Removed dependecies to the ItemRegistry

Fixes #4927
Fixes #4071

* Add field modifiers which did not exist in groovy

Signed-off-by: Stefan Triller <stefan.triller@telekom.de>
ermartens pushed a commit to ermartens/smarthome that referenced this issue Jun 15, 2018
@triller-telekom
Wemo Binding: Converted Groovy tests to Java (eclipse-archived#5586)
a4519e4 
* Wemo Binding: Converted groovy tests to Java

Within the conversion I also did the following:
- Made WemoHttpCall non-static to mock them inside tests
- All Handler inherit from AbstractWemoHandler which has a setter for WemoHttpCall
- Removed necessity for servlets in tests by checking the content sent to the server
- Removed dependecies to the ItemRegistry

Fixes eclipse-archived#4927
Fixes eclipse-archived#4071

* Add field modifiers which did not exist in groovy

Signed-off-by: Stefan Triller <stefan.triller@telekom.de>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
Binding-Wemo Test
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@sjsf