Skip to content

test: fix failed test cases#927

Open
NguyenHoangSon96 wants to merge 3 commits into
masterfrom
test/fix-test-cases
Open

test: fix failed test cases#927
NguyenHoangSon96 wants to merge 3 commits into
masterfrom
test/fix-test-cases

Conversation

@NguyenHoangSon96

@NguyenHoangSon96 NguyenHoangSon96 commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

Closes #

Proposed Changes

  • Modify parseTags() function to make it works with case like this "host=A,location=vancouver, BC,model, uid=droid, C3PO,region=west" - tags with space ' ' .
  • Remove findAuthorizationsByID() test case because Influxdb2 has changed the way they handle tokens. The same was done for C# v2 some weeks ago.
  • Fix gpg donwload script. The same was done for other clients weeks ago.

Checklist

  • CHANGELOG.md updated
  • Rebased/mergeable
  • A test has been added if appropriate
  • mvn test completes successfully
  • Commit messages are conventional
  • Sign CLA (if not already signed)

@NguyenHoangSon96 NguyenHoangSon96 self-assigned this Jun 24, 2026
@codecov-commenter

codecov-commenter commented Jun 24, 2026

Copy link
Copy Markdown

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 88.85%. Comparing base (b431276) to head (609bd17).
❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@            Coverage Diff            @@
##             master     #927   +/-   ##
=========================================
  Coverage     88.84%   88.85%           
+ Complexity      791      789    -2     
=========================================
  Files           174      174           
  Lines          7282     7285    +3     
  Branches        421      422    +1     
=========================================
+ Hits           6470     6473    +3     
  Misses          680      680           
  Partials        132      132           

☔ View full report in Codecov by Harness.
📢 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.

@NguyenHoangSon96

NguyenHoangSon96 commented Jun 24, 2026

Copy link
Copy Markdown
Contributor Author

@alespour @karel-rehor
I can't use Copilot AI review. I'm out of credits for this month 🤓
Screenshot 2026-06-24 at 23 41 33

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