galen icon indicating copy to clipboard operation
galen copied to clipboard

Remove dependency to selenium-edge-driver

Open MichaelKunze opened this issue 8 years ago • 4 comments

For some reasons galen has a dependency to org.seleniumhq.selenium:selenium-edge-driver and no other selenium driver. Why is that? Is that really needed? Can that be removed?

Galen also uses some transitive dependencies from com.codeborne:phantomjsdriver because that one does include all of selenium. Once again. Is that really needed or does galen only use the api?

I'd like to minimize the selenium dependency hell for one. So this might be a start.

Michael

MichaelKunze avatar Aug 24 '17 10:08 MichaelKunze

the other drivers are included also, see chrome driver. I exclude the selenium reps from phantomjsdriver.

Regarding the driver reps: We could make a smaller lib jar which doesn't have the drivers included. @ishubin What do you think?

hypery2k avatar Aug 24 '17 11:08 hypery2k

Our grid runs on selenium 3.3. i was told we cannot use selenium 3.5 because of incompatibilities, therefore i can't use the newest galen, which fixes #514 for me. any chance to let the using project provide the drivers needed? galen iteself could depend on selenium-api only (or something)

MichaelKunze avatar Sep 07 '17 07:09 MichaelKunze

@ishubin what do you think about a core Version?

hypery2k avatar Sep 16 '17 11:09 hypery2k

Is there an explicit reason multiple selenium jars are explicitly listed as dependencies? If there would be only one dependency (on selenium-java, which depends on all of the other explicitly depended on .jars) management in projects depending on galen-framework would be easier (e.g. use a newer version of selenium, and exclude the version specified by galen-framework, for instance).

fhoeben avatar Nov 13 '17 13:11 fhoeben