Renamed UnhandledStreamPipe.ql to a better fitting name and ID

Napalys · asgerf · Napalys · commit d1869941c298 · 2025-06-03T13:57:10.000+02:00
As a side effect of merge `security-and-quality` does not contain anymore related new query.

Co-Authored-By: Asger F &lt;316427+asgerf@users.noreply.github.com&gt;
diff --git a/javascript/ql/integration-tests/query-suite/javascript-code-quality.qls.expected b/javascript/ql/integration-tests/query-suite/javascript-code-quality.qls.expected
@@ -2,5 +2,5 @@ ql/javascript/ql/src/Declarations/IneffectiveParameterType.ql
 ql/javascript/ql/src/Expressions/ExprHasNoEffect.ql
 ql/javascript/ql/src/Expressions/MissingAwait.ql
 ql/javascript/ql/src/LanguageFeatures/SpuriousArguments.ql
-ql/javascript/ql/src/Quality/UnhandledStreamPipe.ql
+ql/javascript/ql/src/Quality/UnhandledErrorInStreamPipeline.ql
 ql/javascript/ql/src/RegExp/RegExpAlwaysMatches.ql
diff --git a/javascript/ql/integration-tests/query-suite/javascript-security-and-quality.qls.expected b/javascript/ql/integration-tests/query-suite/javascript-security-and-quality.qls.expected
@@ -80,7 +80,6 @@ ql/javascript/ql/src/NodeJS/InvalidExport.ql
 ql/javascript/ql/src/NodeJS/MissingExports.ql
 ql/javascript/ql/src/Performance/PolynomialReDoS.ql
 ql/javascript/ql/src/Performance/ReDoS.ql
-ql/javascript/ql/src/Quality/UnhandledStreamPipe.ql
 ql/javascript/ql/src/React/DirectStateMutation.ql
 ql/javascript/ql/src/React/InconsistentStateUpdate.ql
 ql/javascript/ql/src/React/UnsupportedStateUpdateInLifecycleMethod.ql
diff --git a/javascript/ql/src/Quality/UnhandledErrorInStreamPipeline.qhelp b/javascript/ql/src/Quality/UnhandledErrorInStreamPipeline.qhelp
diff --git a/javascript/ql/src/Quality/UnhandledErrorInStreamPipeline.ql b/javascript/ql/src/Quality/UnhandledErrorInStreamPipeline.ql
@@ -1,6 +1,6 @@
 /**
- * @id js/nodejs-stream-pipe-without-error-handling
- * @name Node.js stream pipe without error handling
+ * @id js/unhandled-error-in-stream-pipeline
+ * @name Unhandled error in stream pipeline
  * @description Calling `pipe()` on a stream without error handling will drop errors coming from the input stream
  * @kind problem
  * @problem.severity warning
diff --git a/javascript/ql/test/query-tests/Quality/UnhandledStreamPipe/test.qlref b/javascript/ql/test/query-tests/Quality/UnhandledStreamPipe/test.qlref
@@ -1,2 +1,2 @@
-query: Quality/UnhandledStreamPipe.ql
+query: Quality/UnhandledErrorInStreamPipeline.ql
 postprocess: utils/test/InlineExpectationsTestQuery.ql

Original file line number	Diff line number	Diff line change
`@@ -1,2 +1,2 @@`
`1`		`-query: Quality/UnhandledStreamPipe.ql`
	`1`	`+query: Quality/UnhandledErrorInStreamPipeline.ql`
`2`	`2`	`postprocess: utils/test/InlineExpectationsTestQuery.ql`