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

Update test suite to remove unittest dependency in favor of pytest (feature/pytest) #69

Closed
wants to merge 6 commits into from

Conversation

lbeckman314
Copy link
Member

Overview

This PR includes an initial update of test suit from unitest-style to pytest-style format

lbeckman314 and others added 6 commits November 1, 2024 13:02
* feat: support Python 3.7+; drop Python 2.7,<3.7

* feat: pass through json serialization options

* docs: add advanced usage examples

* feat: spec-compliant routes, with legacy support

* fix: remove debug messages

* test: increase unit test coverage (#52)

* test: increase unit test coverage

* Update tests.yml

---------

Co-authored-by: Liam Beckman <lbeckman314@gmail.com>

* docs: browsable API reference on GitHub Pages (#49)

* docs: browsable API reference on GitHub Pages

* Update tests.yml

---------

Co-authored-by: Liam Beckman <lbeckman314@gmail.com>

* Minor release testing updates

* Update support for Service Info (#55)

- Fix version to match PyPi release (#59)

* Add integration test with Funnel

* Add initial TES integration test (Funnel)

* Add Tmate debug session

* Update Funnel installation

* Re-add all unit tests

* Fix Funnel download issue with correct rc version

* Add latest stable Python to test matrix

* Fix unit tests not reaching Funnel

* Increase test coverage

* Minor linting fix

* Update raised exception for invalid server response

* Update README

* Update README.md

* Exclude tests from packages (#58)

* Add in 3 fields missing in the models for tes.

---------

Co-authored-by: Alex Kanitz <alexander.kanitz@alumni.ethz.ch>
Co-authored-by: Kyle Ellrott <kellrott@gmail.com>
Co-authored-by: Ben Beasley <code@musicinmybrain.net>
Co-authored-by: Venkat Malladi <vmalladi@microsoft.com>
(not to be confused with `from __future__ import …`)

Co-authored-by: Liam Beckman <lbeckman314@gmail.com>
@lbeckman314
Copy link
Member Author

Closing in favor of #69

@lbeckman314 lbeckman314 closed this Nov 4, 2024
@lbeckman314 lbeckman314 deleted the feature/pytest branch November 4, 2024 21:01
@lbeckman314 lbeckman314 restored the feature/pytest branch November 4, 2024 21:01
@lbeckman314 lbeckman314 reopened this Nov 4, 2024
@lbeckman314 lbeckman314 changed the base branch from master to develop November 4, 2024 21:05
@lbeckman314 lbeckman314 closed this Nov 4, 2024
@lbeckman314 lbeckman314 deleted the feature/pytest branch November 4, 2024 21:09
@lbeckman314
Copy link
Member Author

Closing in favor of PR #70

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