Skip to content

directory config & cli need integration tests #1023

Closed

Description

@nothingmuch

So far I have only done manual testing of the config & cli stuff because such tests would be boilerplate heavy and mostly testing functionality due to be removed soon.

After #915 is merged we should revisit this and consider whether or not to retain the environment variable configuration, as well as add a CLI option to specify the config file and the revisit the decision whether to use the dirs trait to find the config file.

Activity

  1. spacebear21 commented on Mar 13, 2026

    @spacebear21
    Collaborator

    The directory is now defunct in favor of payjoin-mailroom, and I think that CLI and config is comparitively lightweight enough that we can get away without testing those for now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions