Skip to content

Fix architecture test failures for apache5.x #6140

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged

Conversation

joviegas
Copy link
Contributor

Motivation and Context

  • Architecture test were failing since we had some warning logs in Apache5 which were migrated from Apach4.x

Modifications

  • Removed Disabling UriNormalization since Apache5.x disabled by default and added test cases
  • Added suprresion for valid WARN cases similar to Apache 4.x

Testing

  • Added Junit
  • Locally ran architecture tests

Screenshots (if appropriate)

License

  • I confirm that this pull request can be released under the Apache 2 license

@joviegas joviegas requested a review from a team as a code owner May 29, 2025 15:56
@joviegas joviegas force-pushed the joviegas/apache-5-achitecture-test-fix branch from 64c98a7 to 9e8fc15 Compare May 29, 2025 16:55
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
64.1% Coverage on New Code (required ≥ 80%)
23.9% Duplication on New Code (required ≤ 3%)
C Reliability Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

@joviegas joviegas merged commit fbfb70f into feature/master/apache5x May 29, 2025
21 of 22 checks passed
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.

2 participants