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

Techdebt: allow test & lint to run for specific service #6024

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Techdebt: allow test & lint to run for specific service #6024

wants to merge 4 commits into from

Commits on Mar 7, 2023

  1. Techdebt: allow test & lint to run for specific service 

    make test SERVICE_NAME=s3
    @hans-d
    hans-d committed Mar 7, 2023
    Configuration menu
    Copy the full SHA
    02b516c View commit details
    Browse the repository at this point in the history

  2. Techdebt: allow test & lint to run for specific service 

    make test SERVICE_NAME=s3
    @hans-d
    hans-d committed Mar 7, 2023
    Configuration menu
    Copy the full SHA
    7b3a48e View commit details
    Browse the repository at this point in the history

  3. Techdebt: allow test & lint to run for specific service 

    make test SERVICE_NAME=s3
    @hans-d
    hans-d committed Mar 7, 2023
    Configuration menu
    Copy the full SHA
    4b09e3c View commit details
    Browse the repository at this point in the history

  4. mypy config to exclude, and test selection via markers 

    - mypy was configred via `files` which fails when specifying a target dir. Now turned into an exclude in the config, and adding explicit files to the Makefile when a service is selected.
- added SKIP_MYPY to be able to skip mypy check (as long as the mypy chnage is not complete)
- using conftest.py to give all the tests default markers
- the include/exclude from the makefile is now in the conftest.py marking as well
- for services, also picking up test that reference that service in its name (specific pattern)
    @hans-d
    hans-d committed Mar 7, 2023
    Configuration menu
    Copy the full SHA
    2ec997c View commit details
    Browse the repository at this point in the history