Skip to content

Conversation

@mmvpm
Copy link
Collaborator

@mmvpm mmvpm commented Mar 21, 2023

Description

Fixes #1986

Merged all the SARIF reports for inner classes into one.

For the code below

public class Main {
    public int len(String s) { return s.length(); }
}
class A {
    public int divide(int x) { return 1 / x; }
}
class B {
    public int index(int[] a, int i) { return a[i]; }
}

utbot displays all the results from inner classes on the Problems view:

image

How to test

Manual tests

Please, repeat the scenario from the issue #1986

Self-check list

  • I've set the proper labels for my PR (at least, for category and component).
  • PR title and description are clear and intelligible.
  • I've added enough comments to my code, particularly in hard-to-understand areas.
  • The functionality I've repaired, changed or added is covered with automated tests.
  • Manual tests have been provided optionally.
  • The documentation for the functionality I've been working on is up-to-date.

@mmvpm mmvpm added comp-sarif Issue is related to SARIF report or Problems tab ctg-bug-fix PR is fixing a bug labels Mar 21, 2023
@mmvpm mmvpm self-assigned this Mar 21, 2023
@Vassiliy-Kudryashov Vassiliy-Kudryashov enabled auto-merge (squash) March 21, 2023 09:41
@Vassiliy-Kudryashov Vassiliy-Kudryashov merged commit 3dac352 into main Mar 21, 2023
@Vassiliy-Kudryashov Vassiliy-Kudryashov deleted the ideaseeker/fix_inner_classes_inspection branch March 21, 2023 10:02
@alisevych alisevych added this to the 2023.03 Release milestone Mar 22, 2023
mmvpm added a commit that referenced this pull request Apr 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

comp-sarif Issue is related to SARIF report or Problems tab ctg-bug-fix PR is fixing a bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Missing SARIF-reports and classes on Problems view

4 participants