sauce_bindings icon indicating copy to clipboard operation
sauce_bindings copied to clipboard

WIP for mobile support

Open titusfortner opened this issue 4 years ago • 2 comments

This is mostly working, but not yet ready for prime time;

Working code with specs:

  • EmuSim Browser
  • EmuSim App
  • Test Object Browser

It's going to need more unit tests for the options and ideally a new release of Appium bindings.

Feedback welcome.

Also, note that this works around the issue of most of our mobile endpoints not yet supporting w3c. So this "works" but does not fulfill the goal of the bindings to generate future-compliant code. That said, this is why bindings are a good idea in that we can toggle over to generate the "correct" syntax as soon as Sauce supports it.

titusfortner avatar Apr 13 '20 01:04 titusfortner

Codacy Here is an overview of what got changed by this pull request:


Issues
======
- Added 9
           

Complexity increasing per file
==============================
- java/src/main/java/com/saucelabs/saucebindings/SauceMobileSession.java  3
- java/src/main/java/com/saucelabs/saucebindings/DeviceOrientation.java  1
- java/src/test/java/com/saucelabs/saucebindings/acceptance/SauceDesktopTest.java  1
- java/src/main/java/com/saucelabs/saucebindings/SauceAndroidOptions.java  1
- java/src/test/java/com/saucelabs/saucebindings/SauceAndroidOptionsTest.java  1
- java/src/main/java/com/saucelabs/saucebindings/SauceSession.java  1
- java/src/main/java/com/saucelabs/saucebindings/SauceMobileOptions.java  4
- java/src/test/java/com/saucelabs/saucebindings/acceptance/SauceRealDeviceBrowserTest.java  1
- java/src/main/java/com/saucelabs/saucebindings/SauceIOSOptions.java  1
- java/src/test/java/com/saucelabs/saucebindings/acceptance/SauceEmuSimBrowserTest.java  1
- java/src/test/java/com/saucelabs/saucebindings/SauceIOSOptionsTest.java  1
         

Clones added
============
- java/src/test/java/com/saucelabs/saucebindings/SauceOptionsTest.java  1
         

See the complete overview on Codacy

nadvolod avatar Apr 16 '20 18:04 nadvolod

And... we can't authenticate via options, so we'll have to pull out that code... :(

titusfortner avatar Apr 27 '20 14:04 titusfortner