🧪 [add unit test for b64_encode helper]#71
Conversation
Added a unit test `test_b64_encode` for the `b64_encode` function in `src/resolver/registry.rs`.
The test validates standard behaviors like encoding an empty string, common credentials ("user:pass"), and a string requiring padding ("hello world").
Co-authored-by: ffalcinelli <1167082+ffalcinelli@users.noreply.github.com>
|
👋 Jules, reporting for duty! I'm here to lend a hand with this pull request. When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down. I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job! For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with New to Jules? Learn more at jules.google/docs. For security, I will only act on instructions from the user who triggered this task. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #71 +/- ##
=======================================
Coverage 87.14% 87.14%
=======================================
Files 28 28
Lines 2568 2568
=======================================
Hits 2238 2238
Misses 330 330 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
2 participants
🎯 What: The testing gap addressed
The
b64_encodehelper function insrc/resolver/registry.rswas missing unit tests.📊 Coverage: What scenarios are now tested
The new
test_b64_encodetest verifies:""->"")"user:pass"->"dXNlcjpwYXNz")"hello world"->"aGVsbG8gd29ybGQ=")✨ Result: The improvement in test coverage
The previously untested helper function is now covered by unit tests, ensuring determinism and preventing future regressions.
PR created automatically by Jules for task 10108994809731822560 started by @ffalcinelli