I think once you make the changes we talked about yesterday, clearCurrentlyViewedFile() & setCurrentlyViewedFile() won't need to be exposed outside EditorManager except for unit tests.  In cases like that, we use an "_" prefix and usually have a separate list in the exports section with a comment saying they're being exported for unit testing only.