From 36baa676baf8a7a5654205715135095dbf655007 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=C5=81ukasz=20Cio=C5=82ecki?= Date: Sat, 16 Nov 2024 09:40:50 +0100 Subject: [PATCH] Improvements to AI comments :) --- docs/Changelog.md | 21 ++++++++++++------- .../nussknacker/test/RandomImplicits.scala | 6 ++++-- 2 files changed, 18 insertions(+), 9 deletions(-) diff --git a/docs/Changelog.md b/docs/Changelog.md index 3d8a04c68da..cd3132a2435 100644 --- a/docs/Changelog.md +++ b/docs/Changelog.md @@ -98,13 +98,20 @@ * [#7021](https://github.com/TouK/nussknacker/pull/7021) Definitions service can return definition without UI config * [#7010](https://github.com/TouK/nussknacker/pull/7010) Dynamic access allowed via indexer operator (`[]`) on expressions typed as `Unknown` * [#7063](https://github.com/TouK/nussknacker/pull/7063) Introduce conversion extension methods in SpeL: - * is(className)/to(className)/toOrNull(className) - * isBoolean/toBoolean/toBooleanOrNull - * isLong/toLong/toLongOrNull - * isDouble/toDouble/toDoubleOrNull - * isBigDecimal/toBigDecimal/toBigDecimalOrNull - * isList/toList/toListOrNull - * isMap/toMap/toMapOrNull - the list of key-value pairs or unknown map can be converted to a map. + * canBe(className)/to(className)/toOrNull(className) + * canBeBoolean/toBoolean/toBooleanOrNull + * canBeLong/toLong/toLongOrNull + * canBeDouble/toDouble/toDoubleOrNull + * canBeBigDecimal/toBigDecimal/toBigDecimalOrNull + * canBeList/toList/toListOrNull + * canBeMap/toMap/toMapOrNull - the list of key-value pairs or unknown map can be converted to a map. +* [#7106](https://github.com/TouK/nussknacker/pull/7106) Fix an issue where pressing the “Esc” key did not remove focus from input fields in dialogs, which prevented the dialog window from closing +* [#7002](https://github.com/TouK/nussknacker/pull/7002) Resolve an issue with union nodes output expression when nodes were copied and pasted +* [#6994](https://github.com/TouK/nussknacker/pull/6994) Fix styling issues for form checkboxes in Firefox +* [#6721](https://github.com/TouK/nussknacker/pull/6721) Provide a popover to display additional information about count +* [#7099](https://github.com/TouK/nussknacker/pull/7099) Provide an option to embedded video to the markdown +* [#7102](https://github.com/TouK/nussknacker/pull/7102) Introduce a new UI to defining aggregations within nodes +* [#7147](https://github.com/TouK/nussknacker/pull/7147) Fix redundant "ParameterName(...)" wrapper string in exported PDFs in nodes details ## 1.17 diff --git a/utils/test-utils/src/main/scala/pl/touk/nussknacker/test/RandomImplicits.scala b/utils/test-utils/src/main/scala/pl/touk/nussknacker/test/RandomImplicits.scala index 5b9f5c2ae0d..e0e8c01b80d 100644 --- a/utils/test-utils/src/main/scala/pl/touk/nussknacker/test/RandomImplicits.scala +++ b/utils/test-utils/src/main/scala/pl/touk/nussknacker/test/RandomImplicits.scala @@ -15,8 +15,10 @@ object RandomImplicits { def nextString(): String = randomString(MinStringLength + rand.nextInt(MaxStringLength - MinStringLength)) - def randomString(length: Int): String = - (0 until length).map(_ => AllowedStringLetters(rand.nextInt(AllowedStringLetters.length))).mkString + def randomString(length: Int): String = { + require(length >= 0, "Length must be non-negative") ∑ + (0 until length).map(_ => AllowedStringLetters(rand.nextInt(AllowedStringLetters.length))).mkString + } }