Closed DavidBakerEffendi closed 2 years ago
Merging #241 (fbc83a8) into master (51af989) will decrease coverage by
1.45%
. The diff coverage is84.09%
.
@@ Coverage Diff @@
## master #241 +/- ##
==========================================
- Coverage 80.97% 79.51% -1.46%
==========================================
Files 32 28 -4
Lines 2302 1850 -452
Branches 145 119 -26
==========================================
- Hits 1864 1471 -393
+ Misses 438 379 -59
Impacted Files | Coverage Δ | |
---|---|---|
...c/main/scala/com/github/plume/oss/Jimple2Cpg.scala | 94.84% <ø> (-0.11%) |
:arrow_down: |
...lume/oss/passes/PlumeForkJoinParallelCpgPass.scala | 64.10% <ø> (ø) |
|
.../plume/oss/passes/base/PlumeContainsEdgePass.scala | 100.00% <ø> (ø) |
|
...om/github/plume/oss/passes/base/PlumeCpgPass.scala | 81.98% <ø> (ø) |
|
.../oss/passes/callgraph/PlumeDynamicCallLinker.scala | 82.75% <ø> (ø) |
|
.../oss/passes/controlflow/PlumeCfgCreationPass.scala | 100.00% <ø> (ø) |
|
...ntrolflow/cfgdominator/PlumeCfgDominatorPass.scala | 100.00% <ø> (ø) |
|
...sses/controlflow/codepencegraph/PlumeCdgPass.scala | 100.00% <ø> (ø) |
|
...b/plume/oss/passes/incremental/PlumeDiffPass.scala | 100.00% <ø> (ø) |
|
...b/plume/oss/passes/incremental/PlumeHashPass.scala | 71.42% <ø> (ø) |
|
... and 8 more |
Added
JavaSrc2Cpg
, specificallyOperatorTests
Changed
jimple2cpg
instead of duplicating work here.IdentityStmt
since they end up duplicating parameters as locals.ParallelCpgPass
forConcurrentWriterCpgPass
Fixed
this
object as the object the method is actually invoked with.code
properties now include the object invoking the method.Local
nodes were duplicated for each time they were referencedNewCall(Operators.assignment)
did not havemethodFullName = Operators.assignment
and thus messing up data flow pathsRelated issues:
None
Reviewer
@DavidBakerEffendi