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

allow spoon to show user defined png files in the report. #501

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

sarmadali20
Copy link

@sarmadali20 sarmadali20 commented Nov 28, 2017

allow spoon to show user defined png files in the report.

@@ -40,12 +40,14 @@ buildscript {
'espressoContrib': 'com.android.support.test.espresso:espresso-contrib:2.2.2',
'runner': 'com.android.support.test:runner:0.5',
'rules': 'com.android.support.test:rules:0.5',
'uiautomator': 'com.android.support.test.uiautomator:uiautomator-v18:2.1.1',
Copy link
Contributor

Choose a reason for hiding this comment

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

I would just add this dependency on the test-app, not to all modules

@dweebo
Copy link
Contributor

dweebo commented Nov 29, 2017

This is basically just a PR to take full screenshots (with dialogs) right? I can't see why else a user would want to add a png file. If you just want to attach an arbitrary file to a test you can already do that with SpoonRule.save

So IMO I'd rather not have another screenshot method which is confusing, I'd rather just fix #4

Sounds like there was some support from Jake to use the code from https://github.com/jraska/Falcon

@dweebo
Copy link
Contributor

dweebo commented Nov 29, 2017

See #502 as an alternative to this PR

@sarmadali20
Copy link
Author

@dweebo thanks for the ref, ill look at these issues/PRs.

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.

2 participants