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.

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

feat: support annotation @ArexMock #133

Merged
merged 1 commit into from
Apr 13, 2023
Merged

feat: support annotation @ArexMock #133

merged 1 commit into from
Apr 13, 2023

Conversation

mr3
Copy link
Collaborator

@mr3 mr3 commented Apr 11, 2023

see ArexMock.java

<!-- https://mvnrepository.com/artifact/com.arextest/arex-common -->
<dependency>
    <groupId>com.arextest</groupId>
    <artifactId>arex-common</artifactId>
    <version>0.1.4</version>
</dependency>

@mr3 mr3 added the enhancement ⭐ New feature or request label Apr 11, 2023
@mr3 mr3 requested review from YongwuHe and mengqcc April 11, 2023 13:28
@mr3 mr3 self-assigned this Apr 11, 2023
@mr3 mr3 requested review from mokdeve and lucas-myx April 11, 2023 13:30
@mr3 mr3 force-pushed the feature/arexmock branch 2 times, most recently from be7e5c3 to 4f4c6c0 Compare April 12, 2023 03:08
@mr3 mr3 force-pushed the feature/arexmock branch 3 times, most recently from 4dee4b7 to fcb1731 Compare April 12, 2023 10:55
@codecov
Copy link

codecov bot commented Apr 12, 2023

Codecov Report

Merging #133 (e1864a4) into main (0008c24) will increase coverage by 1.69%.
The diff coverage is 75.72%.

@@             Coverage Diff              @@
##               main     #133      +/-   ##
============================================
+ Coverage     42.24%   43.94%   +1.69%     
- Complexity     1006     1101      +95     
============================================
  Files           203      205       +2     
  Lines          7837     7979     +142     
  Branches        890      923      +33     
============================================
+ Hits           3311     3506     +195     
+ Misses         4206     4114      -92     
- Partials        320      359      +39     
Impacted Files Coverage Δ
.../java/io/arex/agent/bootstrap/util/ArrayUtils.java 91.66% <0.00%> (-8.34%) ⬇️
...a/io/arex/inst/dynamic/common/DynamiConstants.java 0.00% <0.00%> (ø)
...io/arex/inst/runtime/model/DynamicClassEntity.java 38.63% <41.17%> (-16.37%) ⬇️
...namic/common/listener/ListenableFutureAdapter.java 77.77% <50.00%> (-22.23%) ⬇️
.../arex/inst/dynamic/common/ExpressionParseUtil.java 64.86% <64.86%> (ø)
.../inst/cache/spring/SpringCacheInstrumentation.java 66.66% <69.23%> (ø)
.../arex/inst/cache/arex/ArexMockInstrumentation.java 75.00% <75.00%> (ø)
...inst/dynamic/common/listener/ResponseConsumer.java 71.42% <75.00%> (-28.58%) ⬇️
.../main/java/io/arex/inst/runtime/config/Config.java 75.47% <80.00%> (-3.10%) ⬇️
...rex/inst/dynamic/common/DynamicClassExtractor.java 84.49% <81.01%> (+9.24%) ⬆️
... and 4 more

... and 2 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@mr3 mr3 force-pushed the feature/arexmock branch 3 times, most recently from 3cb4cdf to ec8a6d1 Compare April 13, 2023 07:35
lucas-myx
lucas-myx previously approved these changes Apr 13, 2023
YongwuHe
YongwuHe previously approved these changes Apr 13, 2023
@sonarcloud
Copy link

sonarcloud bot commented Apr 13, 2023

SonarCloud Quality Gate failed.    Quality Gate failed

Bug B 3 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 4 Code Smells

84.2% 84.2% Coverage
0.0% 0.0% Duplication

@mr3 mr3 requested review from lucas-myx and YongwuHe April 13, 2023 08:44
@mr3 mr3 merged commit 104792f into main Apr 13, 2023
9 of 10 checks passed
@mr3 mr3 deleted the feature/arexmock branch April 13, 2023 08:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement ⭐ New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants