Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Jetbrains | Add unit tests and integration tests (AST-80090, AST-80471) #304

Merged
merged 57 commits into from
Jan 23, 2025
Merged

Jetbrains | Add unit tests and integration tests (AST-80090, AST-80471) #304

merged 57 commits into from
Jan 23, 2025

Conversation

AlvoBen
Copy link
Contributor

@AlvoBen AlvoBen commented Jan 20, 2025 •
edited
Loading

By submitting a PR to this repository, you agree to the terms within the Checkmarx Code of Conduct. Please see the contributing guidelines for how to create and submit a high-quality PR for this repo.

Description

Added unit tests infra and tests, and increased integration tests coverage

References

https://checkmarx.atlassian.net/browse/AST-80471
https://checkmarx.atlassian.net/browse/AST-80090

Testing

Added unit tests and integration tests

Checklist

  • I have added documentation for new/changed functionality in this PR (if applicable).
  • All active GitHub checks for tests, formatting, and security are passing
  • The correct base branch is being used

@github-actions GitHub Actions
Copy link

github-actions bot commented Jan 20, 2025 •
edited
Loading

Logo
Checkmarx One – Scan Summary & Details – 68741f60-cf68-4c2a-9b60-4fedd41e1e1f

Great job, no security vulnerabilities found in this Pull Request

BenAlvo1 added 24 commits January 20, 2025 14:50
BenAlvo1 added 25 commits January 21, 2025 14:12
@BenAlvo1
add Selection Tests
0b5ed56
@BenAlvo1
Add scan tests
1acfd93
@BenAlvo1
Add results tree tests
0e138f2
@BenAlvo1
Add component tests
ba9b255
@BenAlvo1
Add component tests
7a0c2c8
@BenAlvo1
Add filter tests
ca8d24e
@BenAlvo1
Add filter tests
6f7bca3
@BenAlvo1
Add filter tests
6a91d7c
@BenAlvo1
Add filter tests
a38a7a4
@BenAlvo1
Add filter tests
6bbc661
@BenAlvo1
Add filter tests
997d8ca
@BenAlvo1
Add resultsNode tests
33ae9c9
@BenAlvo1
Add resultsNode tests
f500b3b
@BenAlvo1
Add resultsNode tests
dc583f9
@BenAlvo1
Add resultsNode tests
ab30ad6
@BenAlvo1
Add resultsNode tests
7ed115b
@BenAlvo1
remove unnecessary functions
078fcac
@BenAlvo1
add different coverage report configuration for unit and integration …
a57cafa 
…tests
@BenAlvo1
exclude getters setters and toString methids from coverage
56ddeb7
@BenAlvo1
move ResultGetState to obj package
ba947c7
@BenAlvo1
increase integration test coverage
82e1a4f
@BenAlvo1
increase integration test coverage
4921e23
@BenAlvo1
increase integration test coverage
3c69186
@BenAlvo1
increase integration test coverage
f9e5a29
@BenAlvo1
increase integration test coverage
a83b361
@AlvoBen AlvoBen changed the title add unit tests platform Jetbrains | Add unit tests and integration tests (AST-80090, AST-80471) Jan 22, 2025
@AlvoBen AlvoBen merged commit 36d5d3e into main Jan 23, 2025
7 checks passed
@AlvoBen AlvoBen deleted the other/benalvo/add-unit-tests branch January 23, 2025 13:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@elchananarb elchananarb elchananarb approved these changes

@greensd4 greensd4 Awaiting requested review from greensd4 greensd4 is a code owner

@miryamfoiferCX miryamfoiferCX Awaiting requested review from miryamfoiferCX miryamfoiferCX is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@AlvoBen @elchananarb @BenAlvo1