The one failure is due to an unexpected pass. This should be fixed in a separate PR.

This PR also deals with the issue discussed in #1159. This might not be the best solution.

Hi @theseion !
Thanks for picking this one up again.

I wonder if it would not be easier (and keep the codebase more succint) to write binaryWriteStreamOn:do: in terms of the existing writeFileStreamOn: aString do: aBlock binary: aBoolean method? Is it because passing through the filename instead of the filereference does not work for this? The advantage is also we have some tests for the existing methods.

You're right, the new method is unnecessary. I've removed it.

@theseion the unexpected pass is indeed because svenvc/zinc#86 is now fixed in Pharo10. I now removed it in ce61d53

I would say we need a test for the new method.
But I will take care of it before marking a new release and merge this one in for now so we can merge the Seaside PRs

Thanks, much appreciated!