Skip to content

test: fix failed test cases#927

Merged
NguyenHoangSon96 merged 3 commits into
masterfrom
test/fix-test-cases
Jun 25, 2026
Merged

test: fix failed test cases#927
NguyenHoangSon96 merged 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

@alespour

Copy link
Copy Markdown
Contributor

we all are 😢

@NguyenHoangSon96

Copy link
Copy Markdown
Contributor Author

we all are 😢

I used a lot credits for the refactoring python3 task.
I thought this effect only me 😲 .

@alespour alespour left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 🚀

  • parseTags() now handles the reported unescaped comma-space format from InfluxQL CSV responses.
  • The removed authorization test matches the changed server behavior around tokens.
  • The Codecov GPG key source update is the current intended fix pattern.
  • CI is green
  • Codecov reports: “All modified and coverable lines are covered by tests.”

@NguyenHoangSon96 NguyenHoangSon96 merged commit 103dfec into master Jun 25, 2026
11 checks passed
@NguyenHoangSon96 NguyenHoangSon96 deleted the test/fix-test-cases branch June 25, 2026 20:39
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.

3 participants