Selenide swallows exception in some cases #401

Closed
asolntsev opened this Issue Sep 19, 2016 · 1 comment

Projects

None yet

1 participant

@asolntsev
Member

For example, if exception is thrown from constructor of class Commands, Selenide will swallow it, and report just ElementShouldBe(visible) exception (or something similar).

java.lang.NoClassDefFoundError: org/apache/http/ssl/TrustStrategy
    at com.codeborne.selenide.commands.DownloadFile.<init>(DownloadFile.java:21)
    at com.codeborne.selenide.commands.Commands.addFileCommands(Commands.java:106)
    at com.codeborne.selenide.commands.Commands.resetDefaults(Commands.java:32)
    at com.codeborne.selenide.commands.Commands.<init>(Commands.java:17)
    at com.codeborne.selenide.commands.Commands.<clinit>(Commands.java:12)
       ...
Caused by: java.lang.ClassNotFoundException: org.apache.http.ssl.TrustStrategy
    at java.net.URLClassLoader.findClass(URLClassLoader.java:381)
    at java.lang.ClassLoader.loadClass(ClassLoader.java:424)
    at sun.misc.Launcher$AppClassLoader.loadClass(Launcher.java:331)
    at java.lang.ClassLoader.loadClass(ClassLoader.java:357)
    ... 33 more
@asolntsev asolntsev added a commit that referenced this issue Sep 19, 2016
@asolntsev asolntsev #401 simplify code
it's easier to just put "synchronized" to method than using "synchronize(this)" blocks
966f1b2
@asolntsev asolntsev added this to the 3.10 milestone Sep 19, 2016
@asolntsev
Member

Fixed in Selenide 3.10

@asolntsev asolntsev closed this Sep 19, 2016
@BorisOsipov BorisOsipov added a commit to BorisOsipov/selenide that referenced this issue Nov 23, 2016
@asolntsev @BorisOsipov asolntsev + BorisOsipov #401 simplify code
it's easier to just put "synchronized" to method than using "synchronize(this)" blocks
7c57110
@BorisOsipov BorisOsipov added a commit to BorisOsipov/selenide that referenced this issue Nov 23, 2016
@asolntsev @BorisOsipov asolntsev + BorisOsipov #401 do not retry on class loading errors 0403947
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment