Skip to content

chore: fix unit test#468

Merged
stdavis merged 4 commits intomainfrom
fix/tests
Feb 18, 2026
Merged

chore: fix unit test#468
stdavis merged 4 commits intomainfrom
fix/tests

Conversation

@stdavis
Copy link
Member

@stdavis stdavis commented Feb 18, 2026

This happened with there was a find replace from steveoh to stdavis

This happened with there was a find replace from steveoh to stdavis
Copilot AI review requested due to automatic review settings February 18, 2026 18:41
@stdavis stdavis enabled auto-merge (rebase) February 18, 2026 18:41
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates a failing unit test expectation around task parsing, and adjusts VS Code workspace configuration related to Python environment management.

Changes:

  • Update test_get_users_task_statuses expected results for the “simple” mock.
  • Add VS Code python-envs.* workspace settings (defaulting to Conda).
  • Remove the deprecated donjayamanne.python-environment-manager from recommended extensions.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 3 comments.

File Description
tests/test_task_checker.py Fixes the expected parsed task counts/user aggregation in the unit test.
.vscode/settings.json Adds Python environment manager settings that default the workspace to Conda.
.vscode/extensions.json Removes a legacy environment manager extension recommendation.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@codecov
Copy link

codecov bot commented Feb 18, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 90.45%. Comparing base (d397057) to head (2b5e945).
⚠️ Report is 66 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #468   +/-   ##
=======================================
  Coverage   90.45%   90.45%           
=======================================
  Files           2        2           
  Lines         419      419           
  Branches       77       65   -12     
=======================================
  Hits          379      379           
  Misses         33       33           
  Partials        7        7           

☔ 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.

@stdavis stdavis requested a review from jacobdadams February 18, 2026 18:48
@stdavis stdavis merged commit c1fb872 into main Feb 18, 2026
7 checks passed
@stdavis stdavis deleted the fix/tests branch February 18, 2026 18:53
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

Comments