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

removed obselete gson-2.6.2 dependency. #507

Merged

Conversation

drapostolos
Copy link
Contributor

Change list

Removed dependency from build.gradle dependencies list:
compile 'com.google.code.gson:gson:2.6.2'
as there already is a later version (2.7) included in the dependencies list.

    compile 'com.google.code.gson:gson:2.7'            <--------
    compile 'org.apache.httpcomponents:httpclient:4.5.2'
    compile 'com.google.guava:guava:19.0'
    compile 'cglib:cglib:3.2.4'
    compile 'commons-validator:commons-validator:1.5.1'
    compile 'org.apache.commons:commons-lang3:3.4'
    compile 'commons-io:commons-io:2.5'
    compile 'com.google.code.gson:gson:2.6.2'         <--------
    compile 'org.springframework:spring-context:4.3.2.RELEASE'
    compile 'org.aspectj:aspectjweaver:1.8.9'

Types of changes

What types of changes are you proposing/introducing to Java client?
Put an x in the boxes that apply

  • [x ] Bugfix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Details

@jsf-clabot
Copy link

jsf-clabot commented Nov 5, 2016

CLA assistant check
Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.

@SrinivasanTarget
Copy link
Member

@drapostolos Can you sign CLA? LGTM 👍

Copy link
Member

@SrinivasanTarget SrinivasanTarget left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@TikhomirovSergey TikhomirovSergey merged commit 9f214cd into appium:master Nov 8, 2016
@TikhomirovSergey
Copy link
Contributor

@drapostolos Ok. I have merged it. Please sign CLA next time. Ok?

TikhomirovSergey pushed a commit that referenced this pull request Nov 18, 2016
* Fixed broken tests

Tests were failing because of an indentation problem in
IOSGesturesTest.java and because a static import in
XCUIAutomationTest.java was coming after the rest of the imports instead
of before

* Added 'touch_id' endpoint

(See https://support.apple.com/en-ca/HT201371 for description of the Touch ID feature)
This endpoint simulates the TouchID feature.

* removed obselete gson-2.6.2 dependency. (#507)

* Fix documentation

* Fix documentation

* Don't commit 'touchID()' 

This will be added post release

* Added touchId test

This test only executes the driver.touchId method, it doesn't do any assertions. Just verifies that it doesn't throw any exceptions.

* Added assertion to keep codacy happy

* Fixed broken tests

Tests were failing because of an indentation problem in
IOSGesturesTest.java and because a static import in
XCUIAutomationTest.java was coming after the rest of the imports instead
of before

* Added 'touch_id' endpoint

(See https://support.apple.com/en-ca/HT201371 for description of the Touch ID feature)
This endpoint simulates the TouchID feature.

* Fix documentation

* Fix documentation

* Don't commit 'touchID()' 

This will be added post release

* Added touchId test

This test only executes the driver.touchId method, it doesn't do any assertions. Just verifies that it doesn't throw any exceptions.

* Added assertion to keep codacy happy

* Moved touchID into new file

-Moved touchID out of IOSDeviceActionShorcuts and into PerformsTouchID
-Made IOSDriver implement PerformsTouchID
-Fixed typo that I noticed in CommandExecutionHelper
-Fixed linting errors

* Fixed conflicts

* Update IOSMobileCommandHelper.java

* Throws exception instead of bogus assertion
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

Successfully merging this pull request may close these issues.

5 participants