Skip to content

Commit 3cbebaa

Browse files
committed
Clarify TODO for refactoring generateProtocolTests usage
1 parent 894a514 commit 3cbebaa

File tree

1 file changed

+2
-0
lines changed
  • smithy-typescript-codegen/src/main/java/software/amazon/smithy/typescript/codegen/integration

1 file changed

+2
-0
lines changed

smithy-typescript-codegen/src/main/java/software/amazon/smithy/typescript/codegen/integration/ProtocolGenerator.java

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -312,6 +312,8 @@ public void setSymbolProvider(SymbolProvider symbolProvider) {
312312
// where the constructor asserts that both writer/writerDelegator are not set, instead of unsetting the other.
313313
// Or consider specialized GenerationContextWithWriter/GenerationContextWithWriterDelegator to use in
314314
// corresponding ProtocolGenerator methods that need writer v/s writerDelegator.
315+
// OR better refactor to add something like a `areProtocolTestsPresent()` so `generateProtocolTests` is called
316+
// and `writer` created only if needed.
315317
public TypeScriptDelegator getWriterDelegator() {
316318
return writerDelegator;
317319
}

0 commit comments

Comments
 (0)