Skip to content

Conversation

@Croway
Copy link
Contributor

@Croway Croway commented Dec 20, 2021

No description provided.

@Croway
Copy link
Contributor Author

Croway commented Dec 20, 2021

Seems like camel-log cannot be tested due to cyclic references, like org.apache.camel:camel-management:3.15.0-SNAPSHOT --> org.apache.camel:camel-log:3.15.0-SNAPSHOT --> org.apache.camel:camel-test-junit5:3.15.0-SNAPSHOT, is there another way to test it? I'd like to use CamelTestSupport features like other components.

@davsclaus
Copy link
Contributor

You put tests inside core/camel-core for regular tests, and jmx tests in camel-management

@Croway
Copy link
Contributor Author

Croway commented Dec 20, 2021

You put tests inside core/camel-core for regular tests, and jmx tests in camel-management

I see, thanks for advice!

@UriParam(label = "formatting", enums = "Default,Tab,Fixed", defaultValue = "Default",
description = "Sets the outputs style to use.")
private DefaultExchangeFormatter.OutputStyle style = DefaultExchangeFormatter.OutputStyle.Default;
@UriParam(defaultValue = "false", description = "Log exchange body as-is (ex. log(\"${body}\").")
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lets use the same description:

If enabled only the body will be printed out

To make it stand out that all the other options are not considered then

@davsclaus davsclaus merged commit 6ef1073 into apache:main Dec 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants