#selenium

/

      • apailthorp-sl
        `WebDriverException: Message: Parameter 'duration' was not a positive integer`
      • lucast
        ha
      • that's strange since it's supposed to be multiplied...
      • kpschmid_
        simonstewart: barancev: The custom RemoteWebElement class "wraps" calls to findElement to find an element that could exist in a nested frame(s)
      • barancev
        kpschmid_: you walk deep through all nested frames and their subframes and loko for elements with the given locator?
      • *look
      • lucast
        ah that's a bug...
      • kpschmid_
        barancev: correct
      • barancev
        kpschmid_: well, give me some time, on Monday I hope to create a wrapper around public API that implements this feature
      • kpschmid_
        barancev: Did you look at my gist?
      • niQo has quit
      • barancev
        kpschmid_: yes, I just wanted to clarify the intention
      • kpschmid_
        barancev: This approach of overriding JsonToWebElementConverter.newRemoteWebElement() has worked very well for us for several years
      • barancev
        because you also told about "gui maps", it's a different feature
      • simonstewart looks at ProtocolConverterTest
      • simonstewart weeps a little
      • kpschmid_
        barancev: Ignore the guimap stuff, not looking for a "new" feature, just have the override working again
      • barancev
        kpschmid_: simonstewart is working on the fix :)
      • kpschmid_: but I'm going to suggest you an alternatve solution based on public API
      • kpschmid_
        barancev: I'm a little confused as to how overriding JsonToWebElementConverter.newRemoteWebElement() is not "public"?
      • barancev
        it's protected, at least :)))
      • kpschmid_
        It *was* changed to private in 3.4.0 and then changed back to protected in 3.5.x
      • barancev
        hope simonstewart fill fix it
      • simonstewart
        Working on it
      • barancev
        but I'd suggest to deprecate it and remove soon :)
      • kpschmid_
        simonstewart: barancev: understand
      • barancev
        it was a *bad* idea to implement this injection hook
      • kpschmid_
        barancev: Unfortunately, the person that implemented our framework leveraged it dating back to 2.x and looks like some extensive refactoring to "un-do" this ditty
      • Selenium-Github
        [13selenium] 15natelowry opened pull request #4624: fix typo in docs for driverExecutableFileName (06master...06patch-1) 02https://git.io/v5RU0
      • barancev
        kpschmid_: I mean us, selenium devs, bad idea to provide this hook, to make it available to end users
      • lucast
        apailthorp-sl: it appears there's lots of multiplying going on in the python bindings... which is why you're probably seeing it sit there for a long time
      • errant_rider has quit
      • Selenium-Github
        [13selenium] 15shs96c pushed 2 new commits to 06master: 02https://git.io/v5RTv
      • 13selenium/06master 14d80f86d 15Simon Stewart: Revivify the ProtocolConverterTest
      • 13selenium/06master 14c8f6bf9 15Simon Stewart: Move element converter so it's only used in RemoteWebDriver...
      • apailthorp-sl
        @lucast strangeness abounds, getting hangs in places I don't think much is going on in. Not feeling warm and fuzzy about geckodriver.
      • simonstewart
        barancev: I bet it made sense at the time
      • barancev
        sure, all decisions have resons
      • lucast
        the pause issue would be python's fault (at least until i diagnose it further)
      • apailthorp-sl
        going to let this thing sit while I go get a burrito. Maybe it will throw a useful error if I wait long enough and spin in my chair three times.
      • adamp joined the channel
      • simonstewart
        Pushed the fix
      • adamp
        Hi, does anyone know if it's possible to reconnect to a previously existing remote webdriver session?
      • I've been looking in the group and a lot of googling but I can't get a straight answer
      • kpschmid_
        simonstewart: Thank you
      • js-sl
        adamp, I don't remember if it was you or someone else, but yes it's technically possible (I don't remember how) but it's not advisable to do AFAIK.
      • adamp
        ok, I've been able to get it working with the python bindings by setting driver.session_id however in the java world the set session functions are protected - so yeah it doesn't look like they're meant to be exposed
      • simonstewart
        adamp: if you know the session id it's possible
      • adamp
        yeah I have the session id, that's easily available. I'm just not able to use it to force a remote webdriver to latch onto the existing session. Are there any java api examples out there does anyone know?
      • I can see there's a setSessionId function: setSessionId(String) - Method in class org.openqa.selenium.remote.RemoteWebDriver
      • but it's protected not public
      • lucast
        apailthorp-sl: i'm not getting any hanging issues that you were
      • what version of geckodriver do you have?
      • apailthorp-sl
        @lucast 0.18
      • 64 bit on Ubuntu
      • cristi joined the channel
      • lucast
        same here
      • apailthorp-sl
        not consistent, hung near the end of test suite before lunch, now hung at start. If I break and restart betting I will get another example, but different.
      • maeriens-sl has quit
      • simonstewart
        I wonder if it's safe to turn the AlertsTest back on
      • lucast
        apailthorp-sl: can you try this?
      • Selenium-Github
        [13selenium] 15shs96c pushed 1 new commit to 06master: 02https://git.io/v5Rtr
      • 13selenium/06master 144cacc9d 15Simon Stewart: Add back test ensuring preservation of pointer location...
      • simonstewart
        apailthorp-sl and lucast: you'll need to add the pause to the keyboard events too. Apparently there are two lists to track
      • errant_rider joined the channel
      • apailthorp-sl
        @lucast sure right after this status meeting...
      • kereliuk
        simonstewart: I'll check up on the status of that issue
      • simonstewart
        Thank you
      • apailthorp-sl
        Keyboard events? Not following.
      • lucast
        simonstewart: not if there are no keyboard events :)
      • Rapture joined the channel
      • simonstewart
        I beliebve it'll send both sets of events
      • Rapture has quit
      • lucast
        it'll still send some pauses for keyboard, but since im not using it they dont matter
      • Rapture joined the channel
      • p0deje joined the channel
      • now if i tried to send some keys afterward.. that'd be an issue
      • Selenium-Github
        [13selenium] 15juangj pushed 1 new commit to 06master: 02https://git.io/v5RmT
      • 13selenium/06master 14b5999f6 15Jakub Vrána: Always use JSON.parse for bot.json.parse (#4603)...
      • NOTICE: [13docs] 15andreastt closed pull request #139: To Fix issue#74 (06gh-pages...06gh-pages) 02https://github.com/SeleniumHQ/docs/pull/139
      • errant_rider has quit
      • js-sl has quit
      • jlipps_ joined the channel
      • p0deje has quit
      • jlipps has quit
      • p0deje joined the channel
      • tourdedave has quit
      • js-sl joined the channel
      • indistylo has quit
      • apailthorp-sl
        @lucast that all seemed to go well, output is https://gist.github.com/980b9fb0d6aeae121370ace...
      • js-sl
        Not sure how many folks would be keen on learning more stuff, but this seems like an awesome deal! https://deals.gdgt.com/sales/pwyw-learn-to-code...
      • lucast
        Not sure why yours isn't working then
      • kpschmidt joined the channel
      • apailthorp-sl
        Thanks in any case, appreciate the consideration
      • kpschmidt has quit
      • kpschmidt joined the channel
      • kpschmid_ has quit
      • kpschmidt has quit
      • cristi has quit
      • Selenium-Github
        [13selenium] 15jimevans pushed 4 new commits to 06master: 02https://git.io/v5RsB
      • 13selenium/06master 14f3208bf 15Jim Evans: Changing .NET ChromeOptions to use goog:chromeOptions as property name...
      • 13selenium/06master 145638427 15Jim Evans: Simplifying Proxy properties marked as obsolete in .NET
      • 13selenium/06master 1474fb803 15Jim Evans: Moving WebDriverWait into main WebDriver.dll assembly...
      • selbot2
      • adamp has quit
      • andrejpodgors-sl joined the channel
      • p0deje has quit
      • p0deje joined the channel
      • Staunch has quit
      • neoncontrails joined the channel
      • energizer joined the channel
      • tourdedave joined the channel
      • tourdedave has quit
      • js-sl has quit
      • apailthorp-sl has quit
      • p0deje has quit
      • Selenium-Github
        [13selenium] 15shs96c pushed 1 new commit to 06master: 02https://git.io/v5RlJ
      • 13selenium/06master 1407b7b54 15Simon Stewart: Replace cglib with bytebuddy...
      • simonstewart has quit
      • js-sl joined the channel
      • js-sl
        webpage says that an element is at (13,683) but when I try clicking it, the page outputs that the click was at (37,695)
      • Anyone have tips on why or how to resolve this? (aside from manually adjusting the numbers?)
      • In my last test I clicked two different times and it seems like the y access was the same amount incorrect. 683 vs 695
      • andrejpodgors-sl has quit
      • Though, I suppose that may not matter as on click events for the buttons are outputting text. :grumbles off to work on it some more:
      • hazmeister-sl joined the channel
      • hazmeister-sl
        js-sl any animation involved?
      • js-sl
        Absolutely. :P
      • Now that I'm not chasing click location issues, it's likely apple's screwery around video playback.
      • I just solved this on iOS 10 but now trying to get it solved again on 7-9
      • hazmeister-sl
        geeze iOS7?
      • what are your usage numbers back there?
      • hazmeister-sl shakes fist at iPhone 4
      • js-sl
        I don't have specific numbers. lol. I probably shouldn't even both with it tbh. Apple says that <9.0 is 3% share
      • hazmeister-sl
        we mainly stick with current and previous iOS versions
      • js-sl
        Right now I can't even get this resolved on 9 though :P
      • hazmeister-sl
        so we support 9 & 10 now
      • you using Appium?
      • js-sl
        II work for an online video saas platform, so our customers could have stupid requirements.
      • Yeah, through Test Object.
      • hazmeister-sl
        I there was a bug with locations and viewports I raised a while back
      • js-sl
        I'm testing mobile web in Safari
      • hazmeister-sl
        you might need to use javascript to find the position
      • js-sl
        Maybe, but I've got this outputting text appropriately:
      • apailthorp-sl joined the channel
      • So, that's got me thinking it's more than a misalignment there
      • hazmeister-sl