Skip to content

Conversation

@syscordan
Copy link
Contributor

Description

Created test that is meant for checking the support of vertices. Another test is added based on the SpaceClaim tutorials.

Issue linked

Please mention the issue number or describe the problem this pull request addresses.

Checklist

  • I have tested my changes locally.
  • I have added necessary documentation or updated existing documentation.
  • I have followed the coding style guidelines of this project.
  • I have added appropriate unit tests.
  • I have reviewed my changes before submitting this pull request.
  • I have linked the issue or issues that are solved to the PR if any.
  • I have assigned this PR to myself.
  • I have added the minimum version decorator to any new backend method implemented.
  • I have made sure that the title of my PR follows Conventional commits style (e.g. feat: extrude circle to cylinder)

@syscordan syscordan self-assigned this Aug 6, 2025
@syscordan syscordan requested a review from a team as a code owner August 6, 2025 22:22
@syscordan syscordan added the testing Anything related to tests label Aug 6, 2025
@github-actions github-actions bot removed the testing Anything related to tests label Aug 6, 2025
@github-actions github-actions bot added the testing Anything related to tests label Aug 11, 2025
@codecov
Copy link

codecov bot commented Aug 11, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 94.85%. Comparing base (fd2ddc6) to head (58691d5).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2157      +/-   ##
==========================================
+ Coverage   94.81%   94.85%   +0.04%     
==========================================
  Files         141      141              
  Lines       10444    10444              
==========================================
+ Hits         9902     9907       +5     
+ Misses        542      537       -5     

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

RobPasMue
RobPasMue previously approved these changes Aug 18, 2025
Copy link
Member

@RobPasMue RobPasMue left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks @syscordan!

@RobPasMue RobPasMue enabled auto-merge (squash) August 18, 2025 06:53
@RobPasMue
Copy link
Member

@syscordan - can we check why some of the tests are failing?

@syscordan
Copy link
Contributor Author

image Found the issue, locally I was using an outdated Python so it was not failing locally. Once I upgraded to python 3.13 the test failed for me due to the combined bodies being lost after doing a unite. Still looking into why another test fails only in 25.2

@syscordan
Copy link
Contributor Author

The 2 failing checks are not due to my changes @RobPasMue so it should be okay to merge now.
image
image

@RobPasMue
Copy link
Member

The 2 failing checks are not due to my changes @RobPasMue so it should be okay to merge now.

Yes I'm aware of that no worries - good to merge on my side!

Copy link
Member

@RobPasMue RobPasMue left a comment

Choose a reason for hiding this comment

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

LGTM, let's merge!

@RobPasMue RobPasMue merged commit 5bf5f74 into main Sep 13, 2025
88 of 89 checks passed
@RobPasMue RobPasMue deleted the test/syscordan_vertex_sc_tutorials branch September 13, 2025 06:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

testing Anything related to tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants