Skip to content

Commit

Permalink
Deletes fs module.
Browse files Browse the repository at this point in the history
Refs #33.

This module's entire purpose was to proxy the underlying Node.js `fs` module in a way which could be spied upon. However, in native ESM we can't spy on even user modules which are imported. We already have `FileSystem` and `FileSystemFake` as a replacement and all tests were successfully migrated to it, so `fs` is no longer needed.
  • Loading branch information
dgp1130 committed Mar 4, 2023
1 parent 2f1da76 commit 4d0cb87
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 39 deletions.
7 changes: 0 additions & 7 deletions common/BUILD.bazel
Original file line number Diff line number Diff line change
Expand Up @@ -68,13 +68,6 @@ jasmine_node_test(
deps = [":formatters_test_lib"],
)

ts_project(
name = "fs",
srcs = ["fs.ts"],
visibility = ["//visibility:public"],
deps = ["//:node_modules/@types/node"],
)

ts_project(
name = "file_system",
srcs = ["file_system.ts"],
Expand Down
32 changes: 0 additions & 32 deletions common/fs.ts

This file was deleted.

0 comments on commit 4d0cb87

Please sign in to comment.