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

Import pytest in density calculator GUI tests #2523

Merged
merged 1 commit into from
Jun 6, 2023
Merged

Conversation

krzywon
Copy link
Contributor

@krzywon krzywon commented Jun 5, 2023

Description

Recently, while attempting to run the GUI tests locally, pytest fails with an no module 'pytest' on line 36 of DensityCalculatorTest.py error. This fixes the error.

There is no associated issue.

How Has This Been Tested?

Run python -m pytest -v -s src/sas/qtgui from the base sasview directory and the tests run again.

Review Checklist (please remove items if they don't apply):

  • Code has been reviewed
  • Functionality has been tested
  • The introduced changes comply with SasView license (BSD 3-Clause)

Copy link
Member

@rozyczko rozyczko left a comment

Choose a reason for hiding this comment

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

nice catch.

@wpotrzebowski wpotrzebowski added the Discuss At The Call Issues to be discussed at the fortnightly call label Jun 6, 2023
@wpotrzebowski wpotrzebowski merged commit 8142559 into main Jun 6, 2023
@wpotrzebowski wpotrzebowski deleted the density-calc-test branch June 6, 2023 13:54
@butlerpd butlerpd removed the Discuss At The Call Issues to be discussed at the fortnightly call label Jul 31, 2023
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.

4 participants