Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix test to work with newer byte-buddy #6029

Merged
merged 1 commit into from
Jul 10, 2018
Merged

Fix test to work with newer byte-buddy #6029

merged 1 commit into from
Jul 10, 2018

Conversation

joshbruning
Copy link
Contributor

@joshbruning joshbruning commented Jun 15, 2018

Fix test to work with newer byte-buddy

Unblocks byte-buddy update from 1.8.3 to 1.8.12+. 1.8.12 (at
least) requires the generated class has access to its super-interfaces,
and will otherwise fail with messages like:
java.lang.IllegalAccessError: class org.openqa.selenium.remote.RemoteWebDriver$ByteBuddy$CuMcRkYl
cannot access its superinterface org.openqa.selenium.remote.BaseAugmenterTest$MagicNumberHolder

Fix test to work with newer byte-buddy

Unblocks byte-buddy update from 1.8.3 to 1.8.12+. 1.8.12 (at 
least) requires the generated class has access to its super-interfaces,
and will otherwise fail with messages like:
java.lang.IllegalAccessError: class org.openqa.selenium.remote.RemoteWebDriver$ByteBuddy$CuMcRkYl
cannot access its superinterface org.openqa.selenium.remote.BaseAugmenterTest$MagicNumberHolder
@p0deje
Copy link
Member

p0deje commented Jun 18, 2018

@joshbruning Have you signed CLA?

@joshbruning
Copy link
Contributor Author

Sorry, yeah, I have. I even have the commit bit - perhaps I should have used it.

@p0deje
Copy link
Member

p0deje commented Jun 19, 2018

@joshbruning Yeah, I guess you could have just committed and pushed ;)

@shs96c
Copy link
Member

shs96c commented Jun 25, 2018

@joshbruning, push it!

@shs96c shs96c merged commit 5248338 into master Jul 10, 2018
grigaman pushed a commit to grigaman/selenium that referenced this pull request Jul 12, 2018
Fix test to work with newer byte-buddy

Unblocks byte-buddy update from 1.8.3 to 1.8.12+. 1.8.12 (at 
least) requires the generated class has access to its super-interfaces,
and will otherwise fail with messages like:
java.lang.IllegalAccessError: class org.openqa.selenium.remote.RemoteWebDriver$ByteBuddy$CuMcRkYl
cannot access its superinterface org.openqa.selenium.remote.BaseAugmenterTest$MagicNumberHolder
@joshbruning joshbruning deleted the joshbruning-patch-1 branch July 22, 2018 16:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants