Skip to content

Refactor: Consolidate duplicate conftest fixtures (GSoC prep) #2474

@SaurabhCodesAI

Description

@SaurabhCodesAI

Hi everyone,

I am working on a GSoC proposal for the "Consolidate tests" idea from the Wiki. While looking through the tests, I noticed that conftest.py is split into three files, causing a lot of duplicate pytest fixtures.

For example, tests/unit/conftest.py and tests/integration/conftest.py both have almost identical copies of:

  • init_db (integration just adds create_test_repo)
  • choose_file_dialog
  • archive_env

As a first step before writing my full proposal (where I plan to remove nox and flatten the test folders), I would like to just do the first part of this: combining the three conftest.py files into one main tests/conftest.py file. I will pass parameters into the fixtures so that both unit and integration tests can use the same file without conflicting.

Is it okay if I open a PR for this consolidation step?

(Note: i am using AI tools to help find this boilerplate faster and navigate the codebase, but I will make sure to test and review the fixture merging manually before opening the PR).
pushing).*

@ThomasWaldmann @m3nu

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions