DaGeRe / peass

Tool for Performance analysis of software system
GNU Affero General Public License v3.0
10 stars 9 forks source link

Wrong module name #119

Closed amtsvch closed 1 year ago

amtsvch commented 1 year ago

In the class ModuleClassMapping, method getModuleName a module name is not defined correctly.

DaGeRe commented 1 year ago

Thanks for the issue, could you specify in which way it is not defined correctly?

amtsvch commented 1 year ago

Using getCanonicalPath instead of getAbsolutePath in the method getModuleName should be a solution.

Here an example for the module jdbc-pool in Tomcat:

DEBUG de.dagere.peass.dependency.analysis.ModuleClassMapping:46 - Module: /home/xxx/tomcat_peass/projectTemp/1/modules/jdbc-pool Name: pool

It does not concern base folders. For a baseFolder it's an empty string as expected

amtsvch commented 1 year ago

When using getCanonicalPath the module name in the above-mentioned example is "modules/jdbc-pool" as expected.