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

DO NOT MERGE - simple GUI test case #2

Closed
wants to merge 35 commits into from
Closed

DO NOT MERGE - simple GUI test case #2

wants to merge 35 commits into from

Conversation

GenevieveBuckley
Copy link
Owner

@GenevieveBuckley GenevieveBuckley commented Oct 4, 2023 •
edited
Loading

DO NOT MERGE - test case for computational-cell-analytics#223

@GenevieveBuckley
Switch to pytest (previously used unittest)
984a7a4
@GenevieveBuckley
Testing CI, remove pytest-qt for now
ed1bb00
@GenevieveBuckley
Is micromamba faster for CI than miniconda
adc161a
@GenevieveBuckley
Update test.yaml (more minimal micromamba setup config)
1cbd132
@GenevieveBuckley
Update environment_cpu.yaml (put name at top for micromamba CI install)
59a8c4f
@GenevieveBuckley
Update environment_gpu.yaml (put name up top to avoid micromamba CI s…
3cfc777 
…etup problems)
@GenevieveBuckley
Fix filename pointing to environment_cpu.yaml
55a7813
@GenevieveBuckley
Generate code coverage report for CI tests
246f642 
Code coverage report is automatically uploaded to codecov.io
https://about.codecov.io/
@GenevieveBuckley
Update test.yaml (install pytest-cov for code coverage reports)
6045f6a
@GenevieveBuckley
Fix code coverage call in CI
fee3c6f
@GenevieveBuckley
Update pyproject.toml to include pytest-cov default arguments
f86fcf8
@GenevieveBuckley
Update test.yaml (pytest-cov default arguments now in pyproject.toml …
1fea5c9 
…settings)
@GenevieveBuckley
Update pyproject.toml (wrap pytest adopts args in quote string)
b0b57e5
@GenevieveBuckley
Update pyproject.toml (rename code coverage output file)
dbbc441
@GenevieveBuckley
Update test.yaml (try experimental codecov@v4 github action)
682f1e5
@GenevieveBuckley
Update test.yaml (explicitly specify location of coverage.xml file)
8b50d4f
@GenevieveBuckley
Update test.yaml (reduce duplicate CI runs)
0cf6803
@GenevieveBuckley
Update pyproject.toml (fix TOML formatting)
f943015
@GenevieveBuckley
Update test.yaml - more verbose output for debugging
cda309f
@GenevieveBuckley
Update pyproject.toml - fix pytest configuration
22d0064
@GenevieveBuckley
Update test.yaml - less verbose output in github actions
f5d34d4
@GenevieveBuckley
Update test.yaml - use latest github checkout action
fa08a7e
@GenevieveBuckley
Update pyproject.toml - update pytest config for more info printed to…
79b0605 
… terminal
@GenevieveBuckley
Update .gitignore - do not commit code coverage reports
0b1820b
@GenevieveBuckley
Update .gitignore - ignore generated code coverage files
8e6c2fc
@GenevieveBuckley
Setup Qt libraries for github actions workflows
36d4c42
@codecov-commenter
Copy link

codecov-commenter commented Oct 4, 2023 •
edited
Loading

Codecov Report

❗ No coverage uploaded for pull request base (dev@8af8aa0). Click here to learn what that means.
The diff coverage is n/a.

@@          Coverage Diff           @@
##             dev       #2   +/-   ##
======================================
  Coverage       ?   23.12%           
======================================
  Files          ?       30           
  Lines          ?     3965           
  Branches       ?        0           
======================================
  Hits           ?      917           
  Misses         ?     3048           
  Partials       ?        0

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@pep8speaks
Copy link

pep8speaks commented Oct 4, 2023 •
edited
Loading

Hello @GenevieveBuckley! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

Line 22:80: E501 line too long (80 > 79 characters)

Comment last updated at 2023-10-04 07:14:33 UTC

@GenevieveBuckley
Copy link
Owner Author

Now that computational-cell-analytics#223 is merged, we can close this test branch.

@GenevieveBuckley GenevieveBuckley deleted the simple-gui-test branch October 25, 2023 05:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@GenevieveBuckley @codecov-commenter @pep8speaks