Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update conftest.py to switch mapdl.tbdat to mapdl.tbdata #3362

Merged
merged 2 commits into from
Aug 19, 2024

Conversation

dts12263
Copy link
Contributor

fixing newly added mapdl.tbdat to mapdl.tbdata to fix the following

[ERROR] TestFieldComponentValueGetter.test_temp - E AttributeError: 'MapdlGrpc' object has no attribute 'tbdat'
[ERROR] TestFieldComponentValueGetter.test_pressure - E AttributeError: 'MapdlGrpc' object has no attribute 'tbdat'
[ERROR] TestFieldComponentValueGetter.test_volt - E AttributeError: 'MapdlGrpc' object has no attribute 'tbdat'
[ERROR] TestFieldComponentValueGetter.test_mag - E AttributeError: 'MapdlGrpc' object has no attribute 'tbdat'

Description

Please provide a brief description of the changes made in this pull request.

Issue linked

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

Checklist

fixing newly added mapdl.tbdat to mapdl.tbdata to fix the following

[ERROR] TestFieldComponentValueGetter.test_temp - E           AttributeError: 'MapdlGrpc' object has no attribute 'tbdat'
[ERROR] TestFieldComponentValueGetter.test_pressure - E           AttributeError: 'MapdlGrpc' object has no attribute 'tbdat'
[ERROR] TestFieldComponentValueGetter.test_volt - E           AttributeError: 'MapdlGrpc' object has no attribute 'tbdat'
[ERROR] TestFieldComponentValueGetter.test_mag - E           AttributeError: 'MapdlGrpc' object has no attribute 'tbdat'
@dts12263 dts12263 requested a review from a team as a code owner August 16, 2024 19:48
@dts12263 dts12263 requested review from germa89 and clatapie and removed request for a team August 16, 2024 19:48
@ansys-reviewer-bot
Copy link
Contributor

Thanks for opening a Pull Request. If you want to perform a review write a comment saying:

@ansys-reviewer-bot review

@dts12263
Copy link
Contributor Author

@pyansys-ci-bot

@dts12263
Copy link
Contributor Author

@pyansys-ci-bot LGTM.

Copy link

codecov bot commented Aug 16, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 87.12%. Comparing base (4d5a791) to head (54033aa).
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3362      +/-   ##
==========================================
- Coverage   87.13%   87.12%   -0.02%     
==========================================
  Files          55       55              
  Lines        9813     9813              
==========================================
- Hits         8551     8550       -1     
- Misses       1262     1263       +1     

@germa89
Copy link
Collaborator

germa89 commented Aug 19, 2024

thank you a lot @dts12263 !

@germa89 germa89 merged commit 1afdd5d into main Aug 19, 2024
64 of 66 checks passed
@germa89 germa89 deleted the dts12263-patch-1 branch August 19, 2024 16:17
@germa89
Copy link
Collaborator

germa89 commented Aug 19, 2024

@pyansys-ci-bot LGTM.

btw, you haven't been authorized yet.... but this comment will trigger

Copy link
Contributor

@pyansys-ci-bot pyansys-ci-bot left a comment

Choose a reason for hiding this comment

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

✅ Approving this PR because germa89 said so in here 😬

@germa89 germa89 mentioned this pull request Aug 19, 2024
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.

3 participants