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

Bug fixes #113

Merged
merged 8 commits into from
Nov 7, 2024
Merged

Bug fixes #113

merged 8 commits into from
Nov 7, 2024

Conversation

charlottekostelic
Copy link
Contributor

@charlottekostelic charlottekostelic commented Nov 4, 2024

Added

  • ignore F401 for test_bookops_worldcat.py to .flake8 file
  • types-pyyaml as dev dependency
  • type annotations to functions and fixtures missing annotations in conftests.py
  • Python 3.13 to unit-tests.yaml

Removed

  • test.mrc file and replaced with a fixture that creates MARC21 data in conftest
  • unnecessary or redundant imports in conftest
  • unused fixtures mock_400_response and mock_409_response

Fixed

  • typo in import statement in query.py
  • live_keys fixture for live tests in Github Actions

@charlottekostelic charlottekostelic changed the base branch from main to releases/v1.0.2 November 5, 2024 15:46
Copy link
Member

@klinga klinga left a comment

Choose a reason for hiding this comment

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

Nice maintenance work!

@klinga klinga merged commit 7403136 into releases/v1.0.2 Nov 7, 2024
10 checks passed
@charlottekostelic charlottekostelic deleted the bug-fixes branch November 7, 2024 16:33
charlottekostelic added a commit that referenced this pull request Dec 4, 2024
* Authorization server scope changes (#111)

* updated dependencies and version

* updated WorldcatAccessToken with scope/scopes
changes

* updated changelog and advanced docs added test

* response type for metadata api methods is Response (#112)

* Bug fixes (#113)

* added ignore flake8 F401 for test_bookops_worldcat

* added types-pyyaml as dev dependency

* added type annotations removed unnecessary imports

* deleted test.mrc as it is now redundant

* fixed typo in import statement

* exported updated dev-requirements.txt

* added 3.13 to unit-tests.yaml

* fixed live_keys fixture for GITHUB_ACTIONS

* New /search/bibs/{oclcNumber} endpoint (#115)

* added bib_search method to MetadataSession class

* added section on /search/bibs/ endpoint to docs

* fixed return type for bib_search method

* Holdings unset updates (#114)

* added cascadeDelete arg to holdings_unset methods

* updated docs to reflect holdings unset changes

* Monthly webtests (#116)

* added yaml file for monthly live test gh action

* added additional webtest for WorldcatAccessToken

* updated contributing.md with info about live tests

* added fixtures for live tests and monthly checks

* separated live tests into two classes

* renamed, simplified endpoint_params fixture

* reviewing changes to MetadataSession webtests

* moved webtests to separate files and directory

* created separate conftest for webtests

* fixed typos in conftest docstrings

* separated webtests into different classes

* updated monthly test schedule

* added additional type annotations to conftest.py

* updated live_keys fixture for posix and windows

* updated triggers in monthly-api-tests.yaml

* moved live_keys fixture to webtests/conftest.py

* changed triggers for webtest workflow

* fixed typo in conditionals

* changed live_token fixture scope to reduce calls

* added automatic retries to holdings tests

* Updated changelog and dependencies for release v1.1.0 (#117)

* updated version in __version__.py

* updated changelog for version 1.1.0

* fixed date in changelog

* updated release checklist and contribution guide

* changed package version, added python 3.13

* updated all dependencies

* added additional dependency updates to changelog

* added link to detailed code changes in changelog

* fixed failing test due to change in api response

* updated mkdocstrings due to error caused by griffe
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