Skip to content
This repository was archived by the owner on Apr 16, 2026. It is now read-only.

schemaview.py: tests for checking SchemaView equality - #474

Merged
ialarmedalien merged 2 commits into
mainfrom
schemaview_eq_tests
Oct 16, 2025
Merged

ialarmedalien merged 2 commits into
mainfrom
schemaview_eq_tests

Conversation

@ialarmedalien

Copy link
Copy Markdown
Collaborator

Tests for the beloved SchemaView __eq__ function.

@codecov

codecov Bot commented Oct 16, 2025 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 77.83%. Comparing base (3c42fc7) to head (ae9b463).
⚠️ Report is 3 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #474      +/-   ##
==========================================
+ Coverage   77.63%   77.83%   +0.19%     
==========================================
  Files          52       52              
  Lines        4512     4512              
  Branches      984      984              
==========================================
+ Hits         3503     3512       +9     
+ Misses        788      781       -7     
+ Partials      221      219       -2     

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

@Silvanoc Silvanoc 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

@ialarmedalien
ialarmedalien merged commit 5af475d into main Oct 16, 2025
17 checks passed
@ialarmedalien
ialarmedalien deleted the schemaview_eq_tests branch October 16, 2025 15:34
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

enhancement New feature or request testing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants