Skip to content

Adding additional smoketests for new features#21790

Draft
Benjin wants to merge 5 commits intomainfrom
dev/benjin/smoketests
Draft

Adding additional smoketests for new features#21790
Benjin wants to merge 5 commits intomainfrom
dev/benjin/smoketests

Conversation

@Benjin
Copy link
Copy Markdown
Contributor

@Benjin Benjin commented Mar 31, 2026

Description

Adding basic E2E/smoketests for backup/restore, dacpac dialog, database management, and schema compare

Code Changes Checklist

  • New or updated unit tests added
  • All existing tests pass (npm run test)
  • Code follows contributing guidelines
  • Telemetry/logging updated if relevant
  • No regressions or UX breakage

Reviewers: Please read our reviewer guidelines

@github-actions
Copy link
Copy Markdown

github-actions bot commented Mar 31, 2026

PR Changes

Category Target Branch PR Branch Difference
vscode-mssql VSIX 77253 KB 6398 KB 🟢 -70855 KB ( -91% )
sql-database-projects VSIX 6174 KB 6119 KB ⚪ -55 KB ( 0% )
data-workspace VSIX 535 KB 535 KB ⚪ 0 KB ( 0% )
keymap VSIX 7 KB 7 KB ⚪ 0 KB ( 0% )

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Mar 31, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 73.34%. Comparing base (7ba53ee) to head (156b11c).
⚠️ Report is 4 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main   #21790      +/-   ##
==========================================
- Coverage   73.42%   73.34%   -0.08%     
==========================================
  Files         331      323       -8     
  Lines      100234    99748     -486     
  Branches     5720     5708      -12     
==========================================
- Hits        73599    73165     -434     
+ Misses      26635    26583      -52     

see 8 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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