Skip to content

Commit 11eb8cb

Browse files
committed
[INTERNAL] readers: Remove Transformer
We currently have no use case for this reader
1 parent afd2ac9 commit 11eb8cb

File tree

4 files changed

+0
-226
lines changed

4 files changed

+0
-226
lines changed

lib/AbstractReader.js

Lines changed: 0 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -99,22 +99,6 @@ class AbstractReader {
9999
});
100100
}
101101

102-
/**
103-
* Create a [Transformer-Reader]{@link @ui5/fs/readers/Transformer} from the current reader.
104-
*
105-
* @private
106-
* @param {@ui5/fs/readers/Transformer~callback} callback
107-
* Callback to check and eventually transform any resource passed through the reader
108-
* @returns {Promise<@ui5/fs/readers/Transformer>} Promise resolving with transformer instance
109-
*/
110-
async transform(callback) {
111-
const {default: Transformer} = await import("./readers/Transformer.js");
112-
return new Transformer({
113-
reader: this,
114-
callback
115-
});
116-
}
117-
118102
/**
119103
* Create a [Link-Reader]{@link @ui5/fs/readers/Link} where all requests are prefixed with
120104
* <code>/resources/<namespace></code>.

lib/readers/Transformer.js

Lines changed: 0 additions & 99 deletions
This file was deleted.

test/lib/readers/Transformer.js

Lines changed: 0 additions & 87 deletions
This file was deleted.

test/lib/resources.js

Lines changed: 0 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -87,30 +87,6 @@ test("Filter resources", async (t) => {
8787
t.is(resources[0].getPath(), "/app/test.js", "Found correct resource");
8888
});
8989

90-
test("Transform resources", async (t) => {
91-
const source = createAdapter({
92-
fsBasePath: "./test/fixtures/application.a/webapp",
93-
virBasePath: "/app/"
94-
});
95-
const transformedSource = await source.transform(async (resourcePath, getResource) => {
96-
if (resourcePath === "/app/test.js") {
97-
const resource = await getResource();
98-
resource.setPath("/app/transformed-test.js");
99-
}
100-
});
101-
102-
const resources = await transformedSource.byGlob("**/*.js");
103-
t.is(resources.length, 1, "Found one resource via transformer");
104-
t.is(resources[0].getPath(), "/app/transformed-test.js", "Found correct resource");
105-
106-
const sourceResources = await source.byGlob("**/*.js");
107-
t.is(sourceResources.length, 1, "Found one resource via source");
108-
t.is(sourceResources[0].getPath(), "/app/test.js", "Found resource with original path");
109-
110-
t.is(await resources[0].getString(), await sourceResources[0].getString(),
111-
"Resources have identical content");
112-
});
113-
11490
test("Flatten resources", async (t) => {
11591
const source = createAdapter({
11692
fsBasePath: "./test/fixtures/application.a/webapp",

0 commit comments

Comments
 (0)