Rolled back to using just pullfolder not flat mapping and that seems to have fixed it..

@plastiv can you address @dailypush comment?

@dailypush Thank you for taking time to review these changes. Can you please confirm that issue is with these MR and you are able to run released composer successfully and screenshots are present in html report?

I don't understand how to reproduce NPE, which looks to be the same as #124 and likely exists regardless of this MR.

@plastiv To a certain extent I would agree about #124 .. but maybe then merge the two for a proper working solution. As it breaks with just the suggested changes... So basically right now I have working code by just changing the folder directory to the sdcard and not using the this latest flat mapping.

.pullFolder( // TODO: Add support for internal storage and external storage strategies. folderOnDevice = "/sdcard/app_spoon-screenshots/${test.className}/${test.testName}", folderOnHostMachine = screenshotsFolderOnHostMachine, logErrors = verboseOutput )

The original code from this branch I am getting that Null exception that seems to be residing from around here (which I believe you have another branch and pull request to address?)

.map { screenshotsFolderOnHostMachine -> PulledFiles( files = emptyList(), // TODO: Pull test files. screenshots = screenshotsFolderOnHostMachine.let { when (it.exists()) { true -> it.listFiles().toList() else -> emptyList() } } )

There doesn't seem to be a unit test that would flush this out for reproducing and is a little harder to show examples.

(Not sure why we need a flatmap inside the other flatmap with a containing map?, I am not that well versed in kotlin but just seems odd )

Hey guys, just wondering if there's any progress with this, will be handy to have it.

Hum...guys? anyone is gonna work on Composer anymore? or is just abandoned?