Enable "File > open" tests for Replay tests about file.dart HOT 4 CLOSED

I looked into the failures.

Failure type 1:

type 'List' is not a subtype of type 'Uint8List' of 'input'

ReplayRandomAccessFile currently uses const Passthrough<Uint8List>() for the #readSync converter. The input is a List<int> that isn't a Uint8List. I don't yet understand why.

Failure type 2:

No matching invocation found: readIntoSync(Instance(length:1024) of '_GrowableList', 0, null)

This happens because on the recording side, we write the event to the manifest after invoking the method, but the method mutates the supplied List argument, so we write the mutated argument, not the original one. The replay side then fails to find the corresponding entry since the List argument no longer matches.

from file.dart.

ReplayRandomAccessFile currently uses const Passthrough<Uint8List>() for the #readSync converter. The input is a List<int> that isn't a Uint8List. I don't yet understand why.

The data is serialized as a JSON array of integers, so I think an explicit conversion to Uint8List should be expected.

This happens because on the recording side, we write the event to the manifest after invoking the method,

Maybe RecordingProxyMixin.noSuchMethod could eagerly encode the argument lists to JSON and pass them directly to LiveMethodEvent.

from file.dart.

Ugh, there is yet another problem in that there's currently no way for record/replay to replicate mutations to method arguments. The recording site probably would need to encode the arguments before invocation and after, and then the replay side would need to figure out how to update arguments in-place. Or maybe there could be some less general mechanism since all of this would be needed only for RandomFileAccess.readInto/readIntoSync.

from file.dart.

This is no longer relevant since record/replay was removed by #162.

from file.dart.