Skip to content

Avoid throwing AuthException to the state machine#16677

Merged
JackieTien97 merged 1 commit intomasterfrom
fixNPE-1030
Oct 30, 2025
Merged

Avoid throwing AuthException to the state machine#16677
JackieTien97 merged 1 commit intomasterfrom
fixNPE-1030

Conversation

@shuwenwei
Copy link
Member

Description

Avoid throwing AuthException to the state machine

@sonarqubecloud
Copy link

@codecov
Copy link

codecov bot commented Oct 30, 2025

Codecov Report

❌ Patch coverage is 13.33333% with 39 lines in your changes missing coverage. Please review.
✅ Project coverage is 38.72%. Comparing base (57fe573) to head (2a25a80).
⚠️ Report is 7 commits behind head on master.

Files with missing lines Patch % Lines
.../iotdb/confignode/persistence/auth/AuthorInfo.java 15.00% 34 Missing ⚠️
...onfignode/persistence/auth/AuthorPlanExecutor.java 0.00% 5 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             master   #16677    +/-   ##
==========================================
  Coverage     38.72%   38.72%            
  Complexity      207      207            
==========================================
  Files          4984     4986     +2     
  Lines        330125   330322   +197     
  Branches      41960    41985    +25     
==========================================
+ Hits         127839   127926    +87     
- Misses       202286   202396   +110     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@JackieTien97 JackieTien97 merged commit f8ad340 into master Oct 30, 2025
31 checks passed
@JackieTien97 JackieTien97 deleted the fixNPE-1030 branch October 30, 2025 12:19
JackieTien97 pushed a commit that referenced this pull request Nov 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants