Skip to content
This repository was archived by the owner on Oct 7, 2025. It is now read-only.

Conversation

@sarmadali20
Copy link

@sarmadali20 sarmadali20 commented Nov 28, 2017

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

'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 subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants