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

Improve testing #864

Open
edan-bainglass opened this issue Oct 16, 2024 · 3 comments
Open

Improve testing #864

edan-bainglass opened this issue Oct 16, 2024 · 3 comments
Assignees
@edan-bainglass
Labels
enhancement New feature or request

Comments

@edan-bainglass
Copy link
Member

Our tests module could likely be made more efficient, especially once (and if 🤞) #802 is merged.

A couple of ideas:

  1. Separate logic (models) and UI (widgets) testing
  2. Combine related tests with shared content in class-based pytest modules to reduce test setup overhead
@edan-bainglass edan-bainglass added the enhancement New feature or request label Oct 16, 2024
@edan-bainglass edan-bainglass self-assigned this Oct 16, 2024
@danielhollas
Copy link
Contributor

@edan-bainglass FYI in terms of speed have a look at #766, there's likely a lot of low-hanging fruit.

@edan-bainglass
Copy link
Member Author

Thanks @danielhollas. Will start addressing this soon!

@edan-bainglass edan-bainglass mentioned this issue Oct 24, 2024
Merged
@danielhollas
Copy link
Contributor

@edan-bainglass about the new fixtures released with AiiDA 2.6, here are the changes I needed to do in AWB to support them

aiidalab/aiidalab-widgets-base#582

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@edan-bainglass edan-bainglass

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@danielhollas @edan-bainglass