Skip to content

added view test for architecture detection plugin#1462

Draft
jstucke wants to merge 2 commits intomasterfrom
plugin-view-tests
Draft

added view test for architecture detection plugin#1462
jstucke wants to merge 2 commits intomasterfrom
plugin-view-tests

Conversation

@jstucke
Copy link
Collaborator

@jstucke jstucke commented Mar 24, 2026

  • this PR shows how plugin view tests can be implemented using the architecture detection plugin as an example

@jstucke jstucke self-assigned this Mar 24, 2026
@jstucke jstucke force-pushed the plugin-view-tests branch 3 times, most recently from 6e81313 to 6bce562 Compare March 24, 2026 13:48
so that plugin tests can access the unit test fixtures
@jstucke jstucke force-pushed the plugin-view-tests branch from 6bce562 to 8eae218 Compare March 24, 2026 14:54
@codecov-commenter
Copy link

codecov-commenter commented Mar 24, 2026

Codecov Report

❌ Patch coverage is 97.74436% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 92.46%. Comparing base (52b8198) to head (8eae218).

Files with missing lines Patch % Lines
src/conftest.py 98.03% 2 Missing ⚠️
...itecture/test/test_plugin_cpu_architecture_view.py 96.42% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1462      +/-   ##
==========================================
+ Coverage   92.40%   92.46%   +0.05%     
==========================================
  Files         370      370              
  Lines       20829    20823       -6     
==========================================
+ Hits        19248    19254       +6     
+ Misses       1581     1569      -12     

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