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.

Sign up for GitHub

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

Jump to bottom

Pytestise test_atom_mapping #688

Open
wants to merge 1 commit into
base: protos
Choose a base branch
from
Open

Pytestise test_atom_mapping #688

wants to merge 1 commit into from

Conversation

oerc0122
Copy link
Collaborator

Description of work

  • Use parametrize in test_atom_mapping
  • Add some extra tests of invalid types.

Raises a misleading error on invalid input:

  • Claiming mass = -1 is not a numeric type.

Fixes
Add some extra tests.

To test
Standard tests.

@oerc0122 oerc0122 added the CI/CD Something concerning the CI/CD pipeline label Feb 28, 2025
@oerc0122 oerc0122 self-assigned this Feb 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ChiCheng45 ChiCheng45 Awaiting requested review from ChiCheng45

@MBartkowiakSTFC MBartkowiakSTFC Awaiting requested review from MBartkowiakSTFC

At least 1 approving review is required to merge this pull request.

Assignees

@oerc0122 oerc0122

Labels
CI/CD Something concerning the CI/CD pipeline
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@oerc0122