Skip to content
This repository has been archived by the owner on Dec 15, 2022. It is now read-only.

Tweak tests so that they work when Pane.close becomes async #448

Merged
merged 1 commit into from
Jun 1, 2017

Conversation

maxbrunsfeld
Copy link
Contributor

The TextBuffer.save method is going to become async in atom/atom#14435; instead of doing a synchronous write, it will initiate an asynchronous write and return a promise when the write completes. As a result, a few methods that depend on saving the buffer (like Pane.close, which is private) are now async as well. This shouldn't affect the user-facing functionality of many packages; but several packages may have test failures.

This PR updates the two tests in tabs that broke due to this change, so that they work regardless of whether save is synchronous or asynchronous.

@maxbrunsfeld maxbrunsfeld merged commit 17341e0 into master Jun 1, 2017
@maxbrunsfeld maxbrunsfeld deleted the mb-allow-pane-close-to-be-async branch June 1, 2017 17:49
This was referenced Jun 14, 2017
This was referenced Jun 14, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant