BAEL-7881: Introduction to SootUp #18106
Conversation
|
|
||
| public class AnalyzeUnitTest { | ||
| @Test | ||
| void whenAnalyzingTheJVM_thenWeCanListClasses() { |
There was a problem hiding this comment.
We follow https://google.github.io/styleguide/javaguide.html#s5.3-camel-case
| void whenAnalyzingTheJVM_thenWeCanListClasses() { | |
| void whenAnalyzingTheJvm_thenWeCanListClasses() { |
|
|
||
| JavaView view = new JavaView(inputLocation); | ||
|
|
||
| assertTrue(view.getClasses().size() > 0); |
There was a problem hiding this comment.
For complex assertions we use AssertJ
| StmtGraph<?> stmtGraph = methodBody.getStmtGraph(); | ||
| List<Stmt> stmts = stmtGraph.getStmts(); | ||
| for (Stmt stmt : stmts) { | ||
| System.out.println(stmt); |
There was a problem hiding this comment.
Can we assert here instead of printing?
There was a problem hiding this comment.
We can, but I was concerned that this would be a bit flaky. It's possible that the exact format of the statements - or even the number of them - would change between versions of SootUp, and maybe between versions of Java.
However, I've updated it as you said so if you want to keep it then it's there :)
|
|
||
| private void someMethod(String name) { | ||
| var capitald = name.toUpperCase(); | ||
| System.out.println("Hello, " + capitald); |
There was a problem hiding this comment.
We have a guideline to use SLF4J not System.out for print statements
There was a problem hiding this comment.
This method isn't being used. It's purely here so that it can be analyzed. I just needed to do something with the capitals variable was all.
| @Test | ||
| void whenAnalyzingThisClass_thenWeCanListMethodsByName() { | ||
| Path javaFile = Path.of("src/test/java/com/baeldung/sootup/MethodUnitTest.java"); | ||
| AnalysisInputLocation inputLocation = new OTFCompileAnalysisInputLocation(javaFile); | ||
|
|
||
| JavaView view = new JavaView(inputLocation); | ||
|
|
||
| IdentifierFactory identifierFactory = view.getIdentifierFactory(); | ||
| ClassType javaClass = identifierFactory.getClassType("com.baeldung.sootup.MethodUnitTest"); | ||
|
|
||
| SootClass sootClass = view.getClassOrThrow(javaClass); | ||
| Set<? extends SootMethod> method = sootClass.getMethodsByName("someMethod"); | ||
| assertEquals(1, method.size()); | ||
| } | ||
|
|
||
| private void someMethod(String name) {} |
There was a problem hiding this comment.
Can we show multiple overloaded methods with the same name listed here perhaps?
No description provided.