CAMEL-19970 - Avoid IllegalArgumentException for file without

[apupier]

extension in same folder

when using JBang and that we let auto-detect routes, the files without extensions were added as potential routes and, as expected, no RouteLoader were found. An IllegalArgumentExeption was thrown at this time.

Note that the regression is a side effect. Previously it was part of Routes but there wasn't pre-parsing and so there was no error (at least when doing an export)

Description

Target

• I checked that the commit is targeting the correct branch (note that Camel 3 uses camel-3.x, whereas Camel 4 uses the main branch)

Tracking

• If this is a large change, bug fix, or code improvement, I checked there is a JIRA issue filed for the change (usually before you start working on it).

Apache Camel coding standards and style

• I checked that each commit in the pull request has a meaningful subject line and body.

• I have run mvn clean install -DskipTests locally and I have committed all auto-generated changes

[apupier]

not found a way to write a test easily.

[apupier]

Given that it is a regression in Camel 3.21, I think we should backport it to 4.0.x and 3.21.x and 3.x if and when the PR is accepted

[davsclaus]

This should be done in the skipFile method instead, and I think its easier to just check if there is no file extension then return true to skip the file.

[davsclaus]

Ah can you move that logic after the // skip dirs check as essentially almost all dirs are named without a dot / extension.