breautek commented on PR #642:
URL:
https://github.com/apache/cordova-plugin-file/pull/642#issuecomment-2445782453
I'm putting this in draft.
Good news is, this PR actually addresses that failing test that we've had
for some time:
> resolveLocalFileSystemURL for cdvfile
✓ file.spec.147 should be able to resolve cdvfile
applicationDirectory fs root
Bad news is, it introduces a lot more failing tests but I think most of them
is because the tests are expecting `file://` uris where it will now receive
`app://` uri on scheme-hosted apps. So I think it just a matter of making the
tests be aware of the iOS changes. I'll try to go through them tomorrow night.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscr...@cordova.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@cordova.apache.org
For additional commands, e-mail: issues-h...@cordova.apache.org
