#1873 refactor code and add tests for CSV with big content on columns#1889

robfrank · 2025-01-03T12:10:15Z

What does this PR do?

This PR provides a little refactor to the CSV importer to use maxProperties and maxPropertySize in the correct way.
A new test with a dedicated CSV file is provided

Motivation

Be sure that #1873 is completed

Checklist

I have run the build using mvn clean package command
My unit tests cover both failure and success scenarios

codacy-production · 2025-01-03T12:36:38Z

Coverage summary from Codacy

See diff coverage on Codacy

Coverage variation	Diff coverage
✅ +0.09%	✅ 0.00%

Coverage variation details

	Coverable lines	Covered lines	Coverage
Common ancestor commit (`1653a85`)	68165	43617	63.99%
Head commit (`e243083`)	68139 (-26)	43663 (+46)	64.08% (+0.09%)

Coverage variation is the difference between the coverage for the head and common ancestor commits of the pull request branch: <coverage of head commit> - <coverage of common ancestor commit>

Diff coverage details

	Coverable lines	Covered lines	Diff coverage
Pull request (#1889)	62	0	0.00%

Diff coverage is the percentage of lines that are covered by tests out of the coverable lines that the pull request added or modified: <covered lines added or modified>/<coverable lines added or modified> * 100%

See your quality gate settings Change summary preferences

_{Codacy stopped sending the deprecated coverage status on June 5th, 2024. Learn more}

…23.Final [skip ci] Bumps [io.undertow:undertow-core](https://github.com/undertow-io/undertow) from 2.3.22.Final to 2.3.23.Final. Release notes *Sourced from [io.undertow:undertow-core's releases](https://github.com/undertow-io/undertow/releases).* > v.2.3.23.Final > -------------- > > Release [2.3.23.Final](https://issues.redhat.com/projects/UNDERTOW/versions/12475695) > Full list of Jiras: [view in Jira](https://issues.redhat.com/issues/?jql=project%20=%2012312521%20AND%20fixVersion%20=%2012475695%20ORDER%20BY%20priority%20DESC,%20key%20ASC) > > ``` > Release Notes - Undertow - Version 2.3.23.Final > ``` Commits * [`1571dd7`](undertow-io/undertow@1571dd7) Prepare 2.3.23.Final * [`f7aed07`](undertow-io/undertow@f7aed07) Merge pull request [#1889](https://redirect.github.com/undertow-io/undertow/issues/1889) from fl4via/backport-fixes\_2.3.x * [`8f86d85`](undertow-io/undertow@8f86d85) [UNDERTOW-2677] Update the Javadoc for MULTIPART\_MAX\_ENTITY\_SIZE to reflect t... * [`ee50afa`](undertow-io/undertow@ee50afa) [UNDERTOW-2690] Fix: Correct documentation post CVE-2024-3884 CVE-2024-4027 f... * [`36e27a5`](undertow-io/undertow@36e27a5) [UNDERTOW-2694] Remove build.metada file * [`e8cb858`](undertow-io/undertow@e8cb858) UNDERTOW-2663 - Unclear Error Message When Max Session Limit is Exceeded * [`fee8adf`](undertow-io/undertow@fee8adf) Merge pull request [#1865](https://redirect.github.com/undertow-io/undertow/issues/1865) from aogburn/04351015 * [`8caf70b`](undertow-io/undertow@8caf70b) Next is 2.3.23.Final * [`ef0005a`](undertow-io/undertow@ef0005a) [UNDERTOW-2677] do not override existing entity limit in MultipartParserDefin... * See full diff in [compare view](undertow-io/undertow@2.3.22.Final...2.3.23.Final) [![Dependabot compatibility score](https://dependabot-badges.githubapp.com/badges/compatibility\_score?dependency-name=io.undertow:undertow-core&package-manager=maven&previous-version=2.3.22.Final&new-version=2.3.23.Final)](https://docs.github.com/en/github/managing-security-vulnerabilities/about-dependabot-security-updates#about-compatibility-scores) Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting `@dependabot rebase`. [//]: # (dependabot-automerge-start) [//]: # (dependabot-automerge-end) --- Dependabot commands and options You can trigger Dependabot actions by commenting on this PR: - `@dependabot rebase` will rebase this PR - `@dependabot recreate` will recreate this PR, overwriting any edits that have been made to it - `@dependabot show ignore conditions` will show all of the ignore conditions of the specified dependency - `@dependabot ignore this major version` will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself) - `@dependabot ignore this minor version` will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself) - `@dependabot ignore this dependency` will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

…io with 2 updates [skip ci] Bumps the build-tools group in /studio with 2 updates: [html-webpack-plugin](https://github.com/jantimon/html-webpack-plugin) and [webpack](https://github.com/webpack/webpack). Updates `html-webpack-plugin` from 5.6.6 to 5.6.7 Changelog *Sourced from [html-webpack-plugin's changelog](https://github.com/jantimon/html-webpack-plugin/blob/main/CHANGELOG.md).* > ### [5.6.7](jantimon/html-webpack-plugin@v5.6.6...v5.6.7) (2026-04-17) > > ### Bug Fixes > > * provide module/exports in VM sandbox for Node.js 25+ compatibility ([#1889](https://redirect.github.com/jantimon/html-webpack-plugin/issues/1889)) ([c862830](jantimon/html-webpack-plugin@c862830)), closes [#1880](https://redirect.github.com/jantimon/html-webpack-plugin/issues/1880) [facebook/docusaurus#11545](https://redirect.github.com/facebook/docusaurus/issues/11545) Commits * [`fdef1b4`](jantimon/html-webpack-plugin@fdef1b4) chore(release): 5.6.7 * [`c862830`](jantimon/html-webpack-plugin@c862830) fix: provide module/exports in VM sandbox for Node.js 25+ compatibility ([#1889](https://redirect.github.com/jantimon/html-webpack-plugin/issues/1889)) * [`0d1ff98`](jantimon/html-webpack-plugin@0d1ff98) refactor: remove lodash usage where possible ([#1888](https://redirect.github.com/jantimon/html-webpack-plugin/issues/1888)) * See full diff in [compare view](jantimon/html-webpack-plugin@v5.6.6...v5.6.7) Updates `webpack` from 5.106.1 to 5.106.2 Release notes *Sourced from [webpack's releases](https://github.com/webpack/webpack/releases).* > v5.106.2 > -------- > > ### Patch Changes > > * CSS [`@import`](https://github.com/import) now inherits the parent module's exportType, so a file configured as "text" correctly creates a style tag when [`@imported`](https://github.com/imported) by a "style" parent. (by [`@xiaoxiaojx`](https://github.com/xiaoxiaojx) in [#20838](https://redirect.github.com/webpack/webpack/pull/20838)) > * Make asset modules available in JS context when referenced from both CSS and a lazily compiled JS chunk. (by [`@xiaoxiaojx`](https://github.com/xiaoxiaojx) in [#20801](https://redirect.github.com/webpack/webpack/pull/20801)) > * Include missing generator options in hash to ensure persistent cache invalidation when configuration changes (CssGenerator `exportsOnly`, JsonGenerator `JSONParse`, WebAssemblyGenerator `mangleImports`). (by [`@xiaoxiaojx`](https://github.com/xiaoxiaojx) in [#20821](https://redirect.github.com/webpack/webpack/pull/20821)) > * Fix `||` default value handling in ProgressPlugin and ManifestPlugin that incorrectly overrode user-provided falsy values (e.g. `modules: false`, `entries: false`, `entrypoints: false`). (by [`@xiaoxiaojx`](https://github.com/xiaoxiaojx) in [#20823](https://redirect.github.com/webpack/webpack/pull/20823)) > * Migrate from `mime-types` to `mime-db`. (by [`@alexander-akait`](https://github.com/alexander-akait) in [#20812](https://redirect.github.com/webpack/webpack/pull/20812)) > * Handle `@charset` at-rules in CSS modules. (by [`@alexander-akait`](https://github.com/alexander-akait) in [#20831](https://redirect.github.com/webpack/webpack/pull/20831)) > * Marked all experimental options in types. (by [`@alexander-akait`](https://github.com/alexander-akait) in [#20814](https://redirect.github.com/webpack/webpack/pull/20814)) Changelog *Sourced from [webpack's changelog](https://github.com/webpack/webpack/blob/main/CHANGELOG.md).* > 5.106.2 > ------- > > ### Patch Changes > > * CSS [`@import`](https://github.com/import) now inherits the parent module's exportType, so a file configured as "text" correctly creates a style tag when [`@imported`](https://github.com/imported) by a "style" parent. (by [`@xiaoxiaojx`](https://github.com/xiaoxiaojx) in [#20838](https://redirect.github.com/webpack/webpack/pull/20838)) > * Make asset modules available in JS context when referenced from both CSS and a lazily compiled JS chunk. (by [`@xiaoxiaojx`](https://github.com/xiaoxiaojx) in [#20801](https://redirect.github.com/webpack/webpack/pull/20801)) > * Include missing generator options in hash to ensure persistent cache invalidation when configuration changes (CssGenerator `exportsOnly`, JsonGenerator `JSONParse`, WebAssemblyGenerator `mangleImports`). (by [`@xiaoxiaojx`](https://github.com/xiaoxiaojx) in [#20821](https://redirect.github.com/webpack/webpack/pull/20821)) > * Fix `||` default value handling in ProgressPlugin and ManifestPlugin that incorrectly overrode user-provided falsy values (e.g. `modules: false`, `entries: false`, `entrypoints: false`). (by [`@xiaoxiaojx`](https://github.com/xiaoxiaojx) in [#20823](https://redirect.github.com/webpack/webpack/pull/20823)) > * Migrate from `mime-types` to `mime-db`. (by [`@alexander-akait`](https://github.com/alexander-akait) in [#20812](https://redirect.github.com/webpack/webpack/pull/20812)) > * Handle `@charset` at-rules in CSS modules. (by [`@alexander-akait`](https://github.com/alexander-akait) in [#20831](https://redirect.github.com/webpack/webpack/pull/20831)) > * Marked all experimental options in types. (by [`@alexander-akait`](https://github.com/alexander-akait) in [#20814](https://redirect.github.com/webpack/webpack/pull/20814)) Commits * [`0d7e3e0`](webpack/webpack@0d7e3e0) chore(release): new release ([#20815](https://redirect.github.com/webpack/webpack/issues/20815)) * [`d5df118`](webpack/webpack@d5df118) chore(deps): bump actions/cache in the dependencies group ([#20839](https://redirect.github.com/webpack/webpack/issues/20839)) * [`5f0874b`](webpack/webpack@5f0874b) fix: make asset modules available in JS when referenced from CSS and lazy JS ... * [`b63ab37`](webpack/webpack@b63ab37) chore(deps): bump test/test262-cases in the dependencies group ([#20792](https://redirect.github.com/webpack/webpack/issues/20792)) * [`313dfc5`](webpack/webpack@313dfc5) ci: improve time for windows ([#20840](https://redirect.github.com/webpack/webpack/issues/20840)) * [`a553f61`](webpack/webpack@a553f61) test: update test262 ([#20841](https://redirect.github.com/webpack/webpack/issues/20841)) * [`1ef747c`](webpack/webpack@1ef747c) fix: CSS [`@import`](https://github.com/import) should inherit parent's exportType over parser config ([#20838](https://redirect.github.com/webpack/webpack/issues/20838)) * [`485d4ce`](webpack/webpack@485d4ce) chore(deps): update `open-cli` ([#20834](https://redirect.github.com/webpack/webpack/issues/20834)) * [`46042b9`](webpack/webpack@46042b9) chore(deps): no outdated strip-ansi ([#20835](https://redirect.github.com/webpack/webpack/issues/20835)) * [`8c7700b`](webpack/webpack@8c7700b) fix: handle `@charset` at-rules in CSS modules * Additional commits viewable in [compare view](webpack/webpack@v5.106.1...v5.106.2) Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting `@dependabot rebase`. [//]: # (dependabot-automerge-start) [//]: # (dependabot-automerge-end) --- Dependabot commands and options You can trigger Dependabot actions by commenting on this PR: - `@dependabot rebase` will rebase this PR - `@dependabot recreate` will recreate this PR, overwriting any edits that have been made to it - `@dependabot show ignore conditions` will show all of the ignore conditions of the specified dependency - `@dependabot ignore major version` will close this group update PR and stop Dependabot creating any more for the specific dependency's major version (unless you unignore this specific dependency's major version or upgrade to it yourself) - `@dependabot ignore minor version` will close this group update PR and stop Dependabot creating any more for the specific dependency's minor version (unless you unignore this specific dependency's minor version or upgrade to it yourself) - `@dependabot ignore ` will close this group update PR and stop Dependabot creating any more for the specific dependency (unless you unignore this specific dependency or upgrade to it yourself) - `@dependabot unignore ` will remove all of the ignore conditions of the specified dependency - `@dependabot unignore ` will remove the ignore condition of the specified dependency and ignore conditions

Bumps `jline.version` from 4.1.0 to 4.1.2. Updates `org.jline:jline-terminal` from 4.1.0 to 4.1.2 Release notes *Sourced from [org.jline:jline-terminal's releases](https://github.com/jline/jline3/releases).* > JLine 4.1.2 is a patch release focused on correctness fixes across the parser, terminal I/O, and shell modules. > > Bug Fixes > --------- > > * fix: echo preserves backslash before unrecognised escape sequences ([#1901](https://redirect.github.com/jline/jline3/issues/1901), fixes [#1863](https://redirect.github.com/jline/jline3/issues/1863)) > * fix: DefaultParser preserves backslashes inside quotes ([#1902](https://redirect.github.com/jline/jline3/issues/1902), fixes [#1877](https://redirect.github.com/jline/jline3/issues/1877)) > * fix: NonBlockingInputStream keeps thread alive after EOF ([#1903](https://redirect.github.com/jline/jline3/issues/1903), fixes [#1879](https://redirect.github.com/jline/jline3/issues/1879)) > * fix: ensure cursor position after alternate screen init ([#1904](https://redirect.github.com/jline/jline3/issues/1904), fixes [#1883](https://redirect.github.com/jline/jline3/issues/1883)) > * fix: use parser for command argument splitting ([#1907](https://redirect.github.com/jline/jline3/issues/1907), fixes [#1876](https://redirect.github.com/jline/jline3/issues/1876)) > > Dependencies > ------------ > > * chore: Bump eu.maveniverse.maven.nisse:extension from 0.9.1 to 0.9.2 ([#1905](https://redirect.github.com/jline/jline3/issues/1905)) > > **Full Changelog**: <jline/jline3@4.1.1...4.1.2> > > JLine 4.1.1 is a patch release focused on stability fixes. The most notable change corrects the POSIX raw mode defaults (VMIN/VTIME) in `enterRawMode`, which could cause shell REPL sessions to hang or malfunction on certain platforms. This release also fixes a Display bug where the internal line buffer could alias or reject immutable caller-provided lists, hardens signal registration against null returns, and corrects alternate charset handling in ScreenTerminal. > > 🐛 Bug Fixes > ----------- > > * fix: Fixed Display oldLines being set as possible immutable lists. ([#1878](https://redirect.github.com/jline/jline3/pull/1878)) [`@Elec332`](https://github.com/Elec332) > * fix: use POSIX cfmakeraw defaults (VMIN=1, VTIME=0) in enterRawMode ([#1871](https://redirect.github.com/jline/jline3/pull/1871)) [`@BryanSant`](https://github.com/BryanSant) > * fix: skip null returns from signal registration in AbstractUnixSysTerminal ([#1869](https://redirect.github.com/jline/jline3/pull/1869)) [`@BryanSant`](https://github.com/BryanSant) > * fix: Fixed ScreenTerminal alt-charset ([#1867](https://redirect.github.com/jline/jline3/pull/1867)) [`@Elec332`](https://github.com/Elec332) > > 📦 Dependency updates > -------------------- > > * chore: Bump com.palantir.javaformat:palantir-java-format from 2.90.0 to 2.91.0 ([#1898](https://redirect.github.com/jline/jline3/pull/1898)) [`@dependabot`](https://github.com/dependabot) > * chore: Bump eu.maveniverse.maven.nisse:extension from 0.9.0 to 0.9.1 ([#1894](https://redirect.github.com/jline/jline3/pull/1894)) [`@dependabot`](https://github.com/dependabot) > * chore: Bump junit.version from 6.0.3 to 6.1.0 ([#1893](https://redirect.github.com/jline/jline3/pull/1893)) [`@dependabot`](https://github.com/dependabot) > * chore: Bump groovy.version from 5.0.5 to 5.0.6 ([#1865](https://redirect.github.com/jline/jline3/pull/1865)) [`@dependabot`](https://github.com/dependabot) > * chore: Bump eu.maveniverse.maven.nisse:extension from 0.8.4 to 0.9.0 ([#1887](https://redirect.github.com/jline/jline3/pull/1887)) [`@dependabot`](https://github.com/dependabot) > * chore: Bump com.diffplug.spotless:spotless-maven-plugin from 3.5.0 to 3.5.1 ([#1886](https://redirect.github.com/jline/jline3/pull/1886)) [`@dependabot`](https://github.com/dependabot) > * chore: Bump org.apache.maven.plugins:maven-enforcer-plugin from 3.6.2 to 3.6.3 ([#1889](https://redirect.github.com/jline/jline3/pull/1889)) [`@dependabot`](https://github.com/dependabot) > * chore: Bump com.diffplug.spotless:spotless-maven-plugin from 3.4.0 to 3.5.0 ([#1880](https://redirect.github.com/jline/jline3/pull/1880)) [`@dependabot`](https://github.com/dependabot) > * chore: Bump slf4j.version from 2.0.17 to 2.0.18 ([#1874](https://redirect.github.com/jline/jline3/pull/1874)) [`@dependabot`](https://github.com/dependabot) > > 👻 Maintenance > ------------- > > * refactor: Remove duplicate code in Shell impl tests ([#1868](https://redirect.github.com/jline/jline3/pull/1868)) [`@Elec332`](https://github.com/Elec332) > > New Contributors > ---------------- > > * [`@BryanSant`](https://github.com/BryanSant) made their first contribution in [#1869](https://redirect.github.com/jline/jline3/pull/1869) > > **Full Changelog**: <jline/jline3@4.1.0...4.1.1> Commits * [`9a74970`](jline/jline3@9a74970) fix: use parser for command argument splitting ([#1907](https://redirect.github.com/jline/jline3/issues/1907)) * [`c9b9315`](jline/jline3@c9b9315) chore: Bump eu.maveniverse.maven.nisse:extension from 0.9.1 to 0.9.2 ([#1905](https://redirect.github.com/jline/jline3/issues/1905)) * [`a2e0670`](jline/jline3@a2e0670) fix: ensure cursor position after alternate screen init ([#1883](https://redirect.github.com/jline/jline3/issues/1883)) ([#1904](https://redirect.github.com/jline/jline3/issues/1904)) * [`6ac5ec1`](jline/jline3@6ac5ec1) fix: NonBlockingInputStream keeps thread alive after EOF (fixes [#1879](https://redirect.github.com/jline/jline3/issues/1879)) ([#1903](https://redirect.github.com/jline/jline3/issues/1903)) * [`1ef5bed`](jline/jline3@1ef5bed) fix: DefaultParser preserves backslashes inside quotes ([#1902](https://redirect.github.com/jline/jline3/issues/1902)) * [`2825bc2`](jline/jline3@2825bc2) fix: echo preserves backslash before unrecognised escape sequences ([#1901](https://redirect.github.com/jline/jline3/issues/1901)) * [`798308f`](jline/jline3@798308f) fix: add filter-by-commitish to 3.x release drafter config * [`af5638a`](jline/jline3@af5638a) fix: avoid aliasing and immutable list issues in Display oldLines ([#1878](https://redirect.github.com/jline/jline3/issues/1878)) * [`f55dc2e`](jline/jline3@f55dc2e) Merge pull request [#1898](https://redirect.github.com/jline/jline3/issues/1898) from jline/dependabot/maven/master/com.palantir.java... * [`1f73667`](jline/jline3@1f73667) Merge pull request [#1894](https://redirect.github.com/jline/jline3/issues/1894) from jline/dependabot/maven/master/eu.maveniverse.ma... * Additional commits viewable in [compare view](jline/jline3@4.1.0...4.1.2) Updates `org.jline:jline-reader` from 4.1.0 to 4.1.2 Release notes *Sourced from [org.jline:jline-reader's releases](https://github.com/jline/jline3/releases).* > JLine 4.1.2 is a patch release focused on correctness fixes across the parser, terminal I/O, and shell modules. > > Bug Fixes > --------- > > * fix: echo preserves backslash before unrecognised escape sequences ([#1901](https://redirect.github.com/jline/jline3/issues/1901), fixes [#1863](https://redirect.github.com/jline/jline3/issues/1863)) > * fix: DefaultParser preserves backslashes inside quotes ([#1902](https://redirect.github.com/jline/jline3/issues/1902), fixes [#1877](https://redirect.github.com/jline/jline3/issues/1877)) > * fix: NonBlockingInputStream keeps thread alive after EOF ([#1903](https://redirect.github.com/jline/jline3/issues/1903), fixes [#1879](https://redirect.github.com/jline/jline3/issues/1879)) > * fix: ensure cursor position after alternate screen init ([#1904](https://redirect.github.com/jline/jline3/issues/1904), fixes [#1883](https://redirect.github.com/jline/jline3/issues/1883)) > * fix: use parser for command argument splitting ([#1907](https://redirect.github.com/jline/jline3/issues/1907), fixes [#1876](https://redirect.github.com/jline/jline3/issues/1876)) > > Dependencies > ------------ > > * chore: Bump eu.maveniverse.maven.nisse:extension from 0.9.1 to 0.9.2 ([#1905](https://redirect.github.com/jline/jline3/issues/1905)) > > **Full Changelog**: <jline/jline3@4.1.1...4.1.2> > > JLine 4.1.1 is a patch release focused on stability fixes. The most notable change corrects the POSIX raw mode defaults (VMIN/VTIME) in `enterRawMode`, which could cause shell REPL sessions to hang or malfunction on certain platforms. This release also fixes a Display bug where the internal line buffer could alias or reject immutable caller-provided lists, hardens signal registration against null returns, and corrects alternate charset handling in ScreenTerminal. > > 🐛 Bug Fixes > ----------- > > * fix: Fixed Display oldLines being set as possible immutable lists. ([#1878](https://redirect.github.com/jline/jline3/pull/1878)) [`@Elec332`](https://github.com/Elec332) > * fix: use POSIX cfmakeraw defaults (VMIN=1, VTIME=0) in enterRawMode ([#1871](https://redirect.github.com/jline/jline3/pull/1871)) [`@BryanSant`](https://github.com/BryanSant) > * fix: skip null returns from signal registration in AbstractUnixSysTerminal ([#1869](https://redirect.github.com/jline/jline3/pull/1869)) [`@BryanSant`](https://github.com/BryanSant) > * fix: Fixed ScreenTerminal alt-charset ([#1867](https://redirect.github.com/jline/jline3/pull/1867)) [`@Elec332`](https://github.com/Elec332) > > 📦 Dependency updates > -------------------- > > * chore: Bump com.palantir.javaformat:palantir-java-format from 2.90.0 to 2.91.0 ([#1898](https://redirect.github.com/jline/jline3/pull/1898)) [`@dependabot`](https://github.com/dependabot) > * chore: Bump eu.maveniverse.maven.nisse:extension from 0.9.0 to 0.9.1 ([#1894](https://redirect.github.com/jline/jline3/pull/1894)) [`@dependabot`](https://github.com/dependabot) > * chore: Bump junit.version from 6.0.3 to 6.1.0 ([#1893](https://redirect.github.com/jline/jline3/pull/1893)) [`@dependabot`](https://github.com/dependabot) > * chore: Bump groovy.version from 5.0.5 to 5.0.6 ([#1865](https://redirect.github.com/jline/jline3/pull/1865)) [`@dependabot`](https://github.com/dependabot) > * chore: Bump eu.maveniverse.maven.nisse:extension from 0.8.4 to 0.9.0 ([#1887](https://redirect.github.com/jline/jline3/pull/1887)) [`@dependabot`](https://github.com/dependabot) > * chore: Bump com.diffplug.spotless:spotless-maven-plugin from 3.5.0 to 3.5.1 ([#1886](https://redirect.github.com/jline/jline3/pull/1886)) [`@dependabot`](https://github.com/dependabot) > * chore: Bump org.apache.maven.plugins:maven-enforcer-plugin from 3.6.2 to 3.6.3 ([#1889](https://redirect.github.com/jline/jline3/pull/1889)) [`@dependabot`](https://github.com/dependabot) > * chore: Bump com.diffplug.spotless:spotless-maven-plugin from 3.4.0 to 3.5.0 ([#1880](https://redirect.github.com/jline/jline3/pull/1880)) [`@dependabot`](https://github.com/dependabot) > * chore: Bump slf4j.version from 2.0.17 to 2.0.18 ([#1874](https://redirect.github.com/jline/jline3/pull/1874)) [`@dependabot`](https://github.com/dependabot) > > 👻 Maintenance > ------------- > > * refactor: Remove duplicate code in Shell impl tests ([#1868](https://redirect.github.com/jline/jline3/pull/1868)) [`@Elec332`](https://github.com/Elec332) > > New Contributors > ---------------- > > * [`@BryanSant`](https://github.com/BryanSant) made their first contribution in [#1869](https://redirect.github.com/jline/jline3/pull/1869) > > **Full Changelog**: <jline/jline3@4.1.0...4.1.1> Commits * [`9a74970`](jline/jline3@9a74970) fix: use parser for command argument splitting ([#1907](https://redirect.github.com/jline/jline3/issues/1907)) * [`c9b9315`](jline/jline3@c9b9315) chore: Bump eu.maveniverse.maven.nisse:extension from 0.9.1 to 0.9.2 ([#1905](https://redirect.github.com/jline/jline3/issues/1905)) * [`a2e0670`](jline/jline3@a2e0670) fix: ensure cursor position after alternate screen init ([#1883](https://redirect.github.com/jline/jline3/issues/1883)) ([#1904](https://redirect.github.com/jline/jline3/issues/1904)) * [`6ac5ec1`](jline/jline3@6ac5ec1) fix: NonBlockingInputStream keeps thread alive after EOF (fixes [#1879](https://redirect.github.com/jline/jline3/issues/1879)) ([#1903](https://redirect.github.com/jline/jline3/issues/1903)) * [`1ef5bed`](jline/jline3@1ef5bed) fix: DefaultParser preserves backslashes inside quotes ([#1902](https://redirect.github.com/jline/jline3/issues/1902)) * [`2825bc2`](jline/jline3@2825bc2) fix: echo preserves backslash before unrecognised escape sequences ([#1901](https://redirect.github.com/jline/jline3/issues/1901)) * [`798308f`](jline/jline3@798308f) fix: add filter-by-commitish to 3.x release drafter config * [`af5638a`](jline/jline3@af5638a) fix: avoid aliasing and immutable list issues in Display oldLines ([#1878](https://redirect.github.com/jline/jline3/issues/1878)) * [`f55dc2e`](jline/jline3@f55dc2e) Merge pull request [#1898](https://redirect.github.com/jline/jline3/issues/1898) from jline/dependabot/maven/master/com.palantir.java... * [`1f73667`](jline/jline3@1f73667) Merge pull request [#1894](https://redirect.github.com/jline/jline3/issues/1894) from jline/dependabot/maven/master/eu.maveniverse.ma... * Additional commits viewable in [compare view](jline/jline3@4.1.0...4.1.2) Updates `org.jline:jline-terminal-jni` from 4.1.0 to 4.1.2 Release notes *Sourced from [org.jline:jline-terminal-jni's releases](https://github.com/jline/jline3/releases).* > JLine 4.1.2 is a patch release focused on correctness fixes across the parser, terminal I/O, and shell modules. > > Bug Fixes > --------- > > * fix: echo preserves backslash before unrecognised escape sequences ([#1901](https://redirect.github.com/jline/jline3/issues/1901), fixes [#1863](https://redirect.github.com/jline/jline3/issues/1863)) > * fix: DefaultParser preserves backslashes inside quotes ([#1902](https://redirect.github.com/jline/jline3/issues/1902), fixes [#1877](https://redirect.github.com/jline/jline3/issues/1877)) > * fix: NonBlockingInputStream keeps thread alive after EOF ([#1903](https://redirect.github.com/jline/jline3/issues/1903), fixes [#1879](https://redirect.github.com/jline/jline3/issues/1879)) > * fix: ensure cursor position after alternate screen init ([#1904](https://redirect.github.com/jline/jline3/issues/1904), fixes [#1883](https://redirect.github.com/jline/jline3/issues/1883)) > * fix: use parser for command argument splitting ([#1907](https://redirect.github.com/jline/jline3/issues/1907), fixes [#1876](https://redirect.github.com/jline/jline3/issues/1876)) > > Dependencies > ------------ > > * chore: Bump eu.maveniverse.maven.nisse:extension from 0.9.1 to 0.9.2 ([#1905](https://redirect.github.com/jline/jline3/issues/1905)) > > **Full Changelog**: <jline/jline3@4.1.1...4.1.2> > > JLine 4.1.1 is a patch release focused on stability fixes. The most notable change corrects the POSIX raw mode defaults (VMIN/VTIME) in `enterRawMode`, which could cause shell REPL sessions to hang or malfunction on certain platforms. This release also fixes a Display bug where the internal line buffer could alias or reject immutable caller-provided lists, hardens signal registration against null returns, and corrects alternate charset handling in ScreenTerminal. > > 🐛 Bug Fixes > ----------- > > * fix: Fixed Display oldLines being set as possible immutable lists. ([#1878](https://redirect.github.com/jline/jline3/pull/1878)) [`@Elec332`](https://github.com/Elec332) > * fix: use POSIX cfmakeraw defaults (VMIN=1, VTIME=0) in enterRawMode ([#1871](https://redirect.github.com/jline/jline3/pull/1871)) [`@BryanSant`](https://github.com/BryanSant) > * fix: skip null returns from signal registration in AbstractUnixSysTerminal ([#1869](https://redirect.github.com/jline/jline3/pull/1869)) [`@BryanSant`](https://github.com/BryanSant) > * fix: Fixed ScreenTerminal alt-charset ([#1867](https://redirect.github.com/jline/jline3/pull/1867)) [`@Elec332`](https://github.com/Elec332) > > 📦 Dependency updates > -------------------- > > * chore: Bump com.palantir.javaformat:palantir-java-format from 2.90.0 to 2.91.0 ([#1898](https://redirect.github.com/jline/jline3/pull/1898)) [`@dependabot`](https://github.com/dependabot) > * chore: Bump eu.maveniverse.maven.nisse:extension from 0.9.0 to 0.9.1 ([#1894](https://redirect.github.com/jline/jline3/pull/1894)) [`@dependabot`](https://github.com/dependabot) > * chore: Bump junit.version from 6.0.3 to 6.1.0 ([#1893](https://redirect.github.com/jline/jline3/pull/1893)) [`@dependabot`](https://github.com/dependabot) > * chore: Bump groovy.version from 5.0.5 to 5.0.6 ([#1865](https://redirect.github.com/jline/jline3/pull/1865)) [`@dependabot`](https://github.com/dependabot) > * chore: Bump eu.maveniverse.maven.nisse:extension from 0.8.4 to 0.9.0 ([#1887](https://redirect.github.com/jline/jline3/pull/1887)) [`@dependabot`](https://github.com/dependabot) > * chore: Bump com.diffplug.spotless:spotless-maven-plugin from 3.5.0 to 3.5.1 ([#1886](https://redirect.github.com/jline/jline3/pull/1886)) [`@dependabot`](https://github.com/dependabot) > * chore: Bump org.apache.maven.plugins:maven-enforcer-plugin from 3.6.2 to 3.6.3 ([#1889](https://redirect.github.com/jline/jline3/pull/1889)) [`@dependabot`](https://github.com/dependabot) > * chore: Bump com.diffplug.spotless:spotless-maven-plugin from 3.4.0 to 3.5.0 ([#1880](https://redirect.github.com/jline/jline3/pull/1880)) [`@dependabot`](https://github.com/dependabot) > * chore: Bump slf4j.version from 2.0.17 to 2.0.18 ([#1874](https://redirect.github.com/jline/jline3/pull/1874)) [`@dependabot`](https://github.com/dependabot) > > 👻 Maintenance > ------------- > > * refactor: Remove duplicate code in Shell impl tests ([#1868](https://redirect.github.com/jline/jline3/pull/1868)) [`@Elec332`](https://github.com/Elec332) > > New Contributors > ---------------- > > * [`@BryanSant`](https://github.com/BryanSant) made their first contribution in [#1869](https://redirect.github.com/jline/jline3/pull/1869) > > **Full Changelog**: <jline/jline3@4.1.0...4.1.1> Commits * [`9a74970`](jline/jline3@9a74970) fix: use parser for command argument splitting ([#1907](https://redirect.github.com/jline/jline3/issues/1907)) * [`c9b9315`](jline/jline3@c9b9315) chore: Bump eu.maveniverse.maven.nisse:extension from 0.9.1 to 0.9.2 ([#1905](https://redirect.github.com/jline/jline3/issues/1905)) * [`a2e0670`](jline/jline3@a2e0670) fix: ensure cursor position after alternate screen init ([#1883](https://redirect.github.com/jline/jline3/issues/1883)) ([#1904](https://redirect.github.com/jline/jline3/issues/1904)) * [`6ac5ec1`](jline/jline3@6ac5ec1) fix: NonBlockingInputStream keeps thread alive after EOF (fixes [#1879](https://redirect.github.com/jline/jline3/issues/1879)) ([#1903](https://redirect.github.com/jline/jline3/issues/1903)) * [`1ef5bed`](jline/jline3@1ef5bed) fix: DefaultParser preserves backslashes inside quotes ([#1902](https://redirect.github.com/jline/jline3/issues/1902)) * [`2825bc2`](jline/jline3@2825bc2) fix: echo preserves backslash before unrecognised escape sequences ([#1901](https://redirect.github.com/jline/jline3/issues/1901)) * [`798308f`](jline/jline3@798308f) fix: add filter-by-commitish to 3.x release drafter config * [`af5638a`](jline/jline3@af5638a) fix: avoid aliasing and immutable list issues in Display oldLines ([#1878](https://redirect.github.com/jline/jline3/issues/1878)) * [`f55dc2e`](jline/jline3@f55dc2e) Merge pull request [#1898](https://redirect.github.com/jline/jline3/issues/1898) from jline/dependabot/maven/master/com.palantir.java... * [`1f73667`](jline/jline3@1f73667) Merge pull request [#1894](https://redirect.github.com/jline/jline3/issues/1894) from jline/dependabot/maven/master/eu.maveniverse.ma... * Additional commits viewable in [compare view](jline/jline3@4.1.0...4.1.2) Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting `@dependabot rebase`. [//]: # (dependabot-automerge-start) [//]: # (dependabot-automerge-end) --- Dependabot commands and options You can trigger Dependabot actions by commenting on this PR: - `@dependabot rebase` will rebase this PR - `@dependabot recreate` will recreate this PR, overwriting any edits that have been made to it - `@dependabot show ignore conditions` will show all of the ignore conditions of the specified dependency - `@dependabot ignore this major version` will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself) - `@dependabot ignore this minor version` will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself) - `@dependabot ignore this dependency` will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

#1873 refactor code and add tests for CSV with big content on columns

e243083

robfrank requested a review from lvca January 3, 2025 12:10

lvca approved these changes Jan 3, 2025

View reviewed changes

lvca assigned robfrank Jan 3, 2025

lvca added the bug label Jan 3, 2025

lvca added this to the 25.1.1 milestone Jan 3, 2025

mergify Bot merged commit 0fcc8af into main Jan 3, 2025

robfrank deleted the improvement/1873-add-tests-for-big-columns branch June 20, 2025 07:41

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

#1873 refactor code and add tests for CSV with big content on columns#1889