b348276
Merge pull request #60 from zed-industries/fix-failures
Click to expand commit body
Fix more test flakiness
Antonio Scandurra
created
37f0ba9
Increase timeout for fs-dependent conditions
Antonio Scandurra
created
4884a05
Change buffer atomically when reloading from disk
Antonio Scandurra
created
19f51bc
Rely more on conditions for state derived after worktree changes
Antonio Scandurra
created
2c43066
Merge pull request #59 from zed-industries/more-resilient-fs-events
Click to expand commit body
Use `flush_fs_events` more after performing synchronous fs mutations
Antonio Scandurra
created
eea9cb4
Use `flush_fs_events` more after performing synchronous fs mutations
Click to expand commit body
I am not sure I have caught all the examples of this, but in general
I think we always want to perform a `flush_fs_events` as opposed to
`next_scan_complete` when doing synchronous I/O. Indeed, the file
system may inform us about the events caused by the just-performed
I/O over multiple batches, and `next_scan_complete` may return
before seeing all of them.
Note that this also removes a few assertions which were ensuring
that, on start, a worktree's file handle wouldn't know its deleted
status, even if the file didn't exist for sure on disk. However,
now that `file` is an async API, it's possible that by the time the
`FileHandle` is resolved, `Worktree` has already completed scanning.
We test a similar behavior further along in the test where those
assertions were removed, so it felt okay to proceed without them.
Antonio Scandurra
created
0187b6d
Merge pull request #54 from zed-industries/file-changed-on-disk
Click to expand commit body
Handle buffers' files changing on disk from outside of Zed
Max Brunsfeld
created
520cbfb
Read file's mtime in background when getting a FileHandle
Click to expand commit body
Co-Authored-By: Antonio Scandurra <me@as-cii.com>
Max Brunsfeld
and
Antonio Scandurra
created
d76d532
Merge branch 'master' into file-changed-on-disk
Max Brunsfeld
created
d0b7377
Merge pull request #57 from zed-industries/dropped-entities
Click to expand commit body
Fix panic when an entity's ref count is decremented to 0 and then is incremented again
Max Brunsfeld
created
493643c
Remove dropped entities even if there are no effects to flush
Max Brunsfeld
created
9dac491
Don't remove entities whose ref count has become positive again
Nathan Sobo
created
e81a28e
Remove dropped entities after every effect
Nathan Sobo
created
c757b3f
Allow a longer timeout in buffer conflict test
Max Brunsfeld
created
4910bc5
Merge branch 'master' into file-changed-on-disk
Max Brunsfeld
created
36163f7
Merge pull request #56 from zed-industries/async-ctx
Click to expand commit body
Simplify spawning and asynchronous code
Max Brunsfeld
created
d6c8952
Remove commented-out tests for spawn and spawn_stream
Max Brunsfeld
created
a3be559
Pass a handle to the current view model when spawning
Click to expand commit body
Most of the time, we'll want a way to get a reference back to the current view or model, so this facilitates that common case.
Nathan Sobo
created
fa6bd1f
Introduce AsyncAppContext and simplify spawning
Click to expand commit body
Now when you call spawn in various context, you pass an FnOnce that is called with an AsyncAppContext and returns a static future. This allows you to write async code similar to how our tests work, borrowing the guts of the AsyncAppContext when needed to interact, but using normal async await semantics instead of callbacks.
Co-Authored-By: Max Brunsfeld <maxbrunsfeld@gmail.com>
Nathan Sobo
and
Max Brunsfeld
created
d0f69c7
Merge pull request #55 from zed-industries/test-macros
Click to expand commit body
Introduce a `gpui::test` macro for less ceremony when writing tests