Skip to content

MMS test for volume calculation in a slab#72

Merged
dschwoerer merged 5 commits into
boutproject:masterfrom
totork:MMS_volume
Apr 23, 2026
Merged

MMS test for volume calculation in a slab#72
dschwoerer merged 5 commits into
boutproject:masterfrom
totork:MMS_volume

Conversation

@totork
Copy link
Copy Markdown
Contributor

@totork totork commented Apr 20, 2026

No description provided.

@totork totork changed the title Mms volume MMS test for volume calculation in a slab Apr 20, 2026
@dschwoerer
Copy link
Copy Markdown
Collaborator

Can you convert this actually in a pytest compatible test?
https://docs.pytest.org/en/stable/getting-started.html#create-your-first-test

@totork
Copy link
Copy Markdown
Contributor Author

totork commented Apr 22, 2026

@dschwoerer I added the pytest structure for the test.

@totork
Copy link
Copy Markdown
Contributor Author

totork commented Apr 22, 2026

I did "ruff check --fix", and it says everything is fine but it does not pass the CI.

@dschwoerer
Copy link
Copy Markdown
Collaborator

You also need to run ruff format or black on the files, to get them format. Right now there are several lines with only whitespace, probably from changing the indent level.

@totork
Copy link
Copy Markdown
Contributor Author

totork commented Apr 23, 2026

Done

Copy link
Copy Markdown
Collaborator

@dschwoerer dschwoerer left a comment

Choose a reason for hiding this comment

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

Thanks!

@dschwoerer dschwoerer merged commit 47a4466 into boutproject:master Apr 23, 2026
4 checks passed
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