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

[iOS] Add Device.force_touch? #478

Merged
merged 1 commit into from May 11, 2016

Conversation

hboon
Copy link
Contributor

@hboon hboon commented Oct 18, 2015

No description provided.

@markrickert
Copy link
Collaborator

Nice! I'm wondering if tests for this would be of any benefit?

@hboon
Copy link
Contributor Author

hboon commented Oct 19, 2015

Do you mean test as in a spec for this or whether this is useful? For the former, I'm not sure how to write one, other than verifying that it's false (because it's in the simulator)?

@markrickert
Copy link
Collaborator

Oh, i definitely think it's useful... but tests can run on the device as well. The test should probably check to see what device it is and if it's a 6s or a 6s Plus, it should return true.

@hboon
Copy link
Contributor Author

hboon commented Oct 19, 2015

Ah, OK.

I usually use this: https://gist.github.com/hboon/e22943f155cc628c782d. I don't think -moHardwareString can be translated to RubyMotion though. Any idea how to check for device type otherwise?

@hboon
Copy link
Contributor Author

hboon commented Jan 16, 2016

I want to get this in patch in because I'm already using it :) Anyone have any idea how to write a test for this? Ping @clayallsopp @colinta for help.

(The Travis failure looks unrelated).

@colinta
Copy link
Contributor

colinta commented Jan 16, 2016

Cool! I'm until the sim adds support, we can't test it. I'll take a look and merge soon.

On Jan 15, 2016, at 11:06 PM, Hwee-Boon Yar notifications@github.com wrote:

I want to get this in patch in because I'm already using it :) Anyone have any idea how to write a test for this? Ping @clayallsopp @colinta for help.

(The Travis failure looks unrelated).


Reply to this email directly or view it on GitHub.

@hboon
Copy link
Contributor Author

hboon commented May 11, 2016

@colinta did you have a chance to look at this? Want to get rid of my private fork :)

@colinta
Copy link
Contributor

colinta commented May 11, 2016

Ah yes, thank's for reminding me! @markrickert I'm gonna punt on requiring specs, because those won't be picked up in automated testing. If he's been using this code successfully this whole time than it's obviously working, that's good enough for me.

@colinta colinta merged commit 3703151 into rubymotion-community:master May 11, 2016
@hboon
Copy link
Contributor Author

hboon commented May 11, 2016

@colinta thanks!

@markrickert
Copy link
Collaborator

lol, no worries... I'm pretty sure it's not possible to automate the testing for this.

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.

None yet

3 participants