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

Bug: NoMethodError: undefined method `write_session_id' for #<Appium::Core::Driver:0x00007fa69119ce88> #686

Closed
1 of 3 tasks
luizlohn opened this issue Oct 5, 2017 · 6 comments
Milestone

Comments

@luizlohn
Copy link

luizlohn commented Oct 5, 2017

This is a

  • Bug report
  • Question
  • Freature Request

Summary

Environment

  • Appium version (or git revision): 1.7.1
  • ruby_lib version: 2.4.2
  • Mobile platform/version/device under test: android and ios

Actual behaviour and steps to reproduce

When I start my tests in the new version of appium_lib this error broke my test:
NoMethodError: undefined method `write_session_id' for #Appium::Core::Driver:0x00007fa69119ce88

Expected behaviour

Link to Appium/Ruby logs

Create a GIST which is a paste of your full Appium logs, and link them here.

Any additional comments

@KazuCocoa
Copy link
Member

Thanks!

https://github.com/appium/ruby_lib/search?utf8=✓&q=write_session_id&type=
the method defined in lib/appium_lib/driver.rb but called in lib/appium_lib/core/driver.rb. I'll move the method from core to not core I can. If no, I'll move the definition to core.

@KazuCocoa KazuCocoa added this to the v9.7.2 milestone Oct 6, 2017
@luizlohn
Copy link
Author

luizlohn commented Oct 6, 2017

Thank you @KazuCocoa if you want some help, I can help you!
:)

@luizlohn
Copy link
Author

luizlohn commented Oct 6, 2017

@KazuCocoa you will release ruby_lib with this correction today?

@KazuCocoa
Copy link
Member

I've confirmed the reproducibility and fixed in #689
branch: https://github.com/KazuCocoa/ruby_lib/tree/fix_export_session

I'd love to release if current PRs are all green and be able to merge, but I can't promise it because CI queue is stacking.
If you'd like to use the fix version, you can use from https://github.com/KazuCocoa/ruby_lib/tree/fix_export_session so far.
@luizlohn

@luizlohn
Copy link
Author

luizlohn commented Oct 6, 2017

Perfect @KazuCocoa i will use! THank you

@KazuCocoa
Copy link
Member

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants