Skip to content

Latest commit

 

History

History
145 lines (97 loc) · 5.74 KB

CONTRIBUTING.rst

File metadata and controls

145 lines (97 loc) · 5.74 KB

Contributing

Contributions are welcome, and they are greatly appreciated! Every little bit helps, and credit will always be given.

You can contribute in many ways:

Types of Contributions

Report Bugs

Report bugs at https://github.com/lichess-org/berserk/issues.

Fix Bugs

Look through the GitHub issues for bugs.

Implement Missing Endpoints

You can run the check-endpoints.py script (requites yaml to be installed, pip3 install pyyaml) in the root of the project to get a list of endpoints that are still missing.

Write Documentation

berserk could always use more documentation, whether as part of the official berserk docs, in docstrings, or even on the web in blog posts, articles, and such.

Submit Feedback

The best way to send feedback is to file an issue at https://github.com/lichess-org/berserk/issues.

If you are proposing a feature:

  • Explain in detail how it would work.
  • Keep the scope as narrow as possible, to make it easier to implement.
  • Remember that this is a volunteer-driven project, and that contributions are welcome :)

Get Started!

  • Install poetry (pip3 install poetry)

  • Setup dependencies by running poetry install --with dev or make setup

  • Optional: Add pre-commit hook to test and format your change before commiting: cp hooks/pre-commit .git/hooks/pre-commit

  • Start editing the code

  • To test your changes, run poetry shell to activate the poetry environment, open a python interpreter (python3), and import the library to test your changes:

    >>> import berserk
    >>> client = berserk.Client()
    >>> client.users.my_new_method()

For a PR to be merged, it needs to pass the CI, you can reproduce most of them locally (commands assume being in the root directory of this repo):

  • To run tests, use poetry run pytest or make test
  • To run type checking, use poetry run pyright berserk or make typecheck
  • To format the code, use poetry run black berserk tests or make format
  • To check doc generation use poetry run sphinx-build -b html docs _build -EW or make docs
    • You can then open _build/index.html in your browser or use python3 -m http.server --directory _build to serve it locally
    • Alternatively, run make servedocs to automatically build and serve them on http://localhost:8000

Writing Tests

We use requests-mock and pytest-recording / vcrpy to test http requests.

Using requests-mock

requests-mock can be used to manually mock and test simple http requests:

import requests_mock
from berserk import Client

def test_correct_speed_params(self):
    """The test verify that speeds parameter are passed correctly in query params"""
    with requests_mock.Mocker() as m:
        m.get(
            "https://explorer.lichess.ovh/lichess?variant=standard&speeds=rapid%2Cclassical",
            json={"white":1212,"draws":160,"black":1406},
        )
        res = Client().opening_explorer.get_lichess_games(speeds=["rapid", "classical"])

Mocking should only be used to test client-side logic.

Using pytest-recording / vcrpy

pytest-recording (which internally uses vcrpy) can be used to record and replay real http requests:

import pytest

from berserk import Client, OpeningStatistic

from utils import validate, skip_if_older_3_dot_10

@skip_if_older_3_dot_10
@pytest.mark.vcr # <---- this tells pytest-recording to record/mock requests made in this test
def test_result(self):
    """Verify that the response matches the typed-dict"""
    res = Client().opening_explorer.get_lichess_games(
        variant="standard",
        speeds=["blitz", "rapid", "classical"],
        ratings=["2200", "2500"],
        position="rnbqkbnr/ppp2ppp/8/3pp3/4P3/2NP4/PPP2PPP/R1BQKBNR b KQkq - 0 1",
    )
    validate(OpeningStatistic, res)

This should be used to test server-side behavior.

To record new requests, run make test_record. This will run all tests and record new requests made in annotated methods in a cassettes directory next to the test. Note that this will not overwrite existing captures, so you need to delete them manually if you want to re-record them.

When running tests regularly (e.g. with make test), the recorded requests will be replayed instead of making real http requests.

⚠️ Do not record sensitive information (tokens). See the Filtering information documentation.

Deploying

A reminder for the maintainers on how to deploy.

You need a PyPI account with access to the berserk package and have an API token with the corresponding access configured for poetry (see https://python-poetry.org/docs/repositories/#configuring-credentials):

Make sure all your changes are committed (including an entry in CHANGELOG.rst) and you set the version in pyproject.toml correctly.

Then run make publish and tag the release on git: git tag v1.2.3 && git push --tags