-
Notifications
You must be signed in to change notification settings - Fork 14
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
Add tomcat legacy as a test case for analysis #31
Comments
Seems to be present as tc_tomcat / Tomcate legacy application - https://github.com/konveyor/go-konveyor-tests/blob/b0615a30ee635dad62f898e67fc28ad0f828c188/analysis/tc_tomcat.go |
Test was updated with fixes of incidents line numbers and Tags. Currently expected analysis results is at https://github.com/konveyor/go-konveyor-tests/blob/main/analysis/tc_tackle_testapp_public.go @dymurray @pranavgaikwad I'd like to review&confirm if expected analysis results is correct, thanks! Open questions for the analysis results approval:
|
Re-enabled check of Analysis Tags and updated expected Tags with their Category. Updated error outputs for better readability. Related to konveyor#31 Signed-off-by: Marek Aufart <maufart@redhat.com>
|
Basic TestAppPublic analysis test (just source) is now part of TIER0 and works well. |
https://github.com/konveyor/example-applications/tree/main/example-1
The text was updated successfully, but these errors were encountered: