Skip to content

Publish cats-tests #1656

Closed
Closed

Description

@edmundnoble

cats-tests remains unpublished and undocumented (#152). This is a stumbling block for new contributors and for people like @djspiewak and myself who find ourselves writing add-on libraries for cats and are unable to write tests in a style consistent with the tests in cats.

cats-tests has never been published according to Git blame, even in the cats artifact, contrary to statements in #1184.

Are there any issues with publishing cats-tests?

Activity

  1. kailuowang commented on May 10, 2017

    @kailuowang
    Contributor

    My 2 cents:

    1. we need to consolidate all the packaging related issues. I think @BennyHill summarize the situation (questions to answer) pretty well in this comment
    2. I totally agree that we need better documentation on how to write law tests in cats style. However, as I found out in mainecoon, you probably don't need the majority of the test utils in cats.tests. I just needed to copy one class declaration from cats.tests to be able to write tests consistent to cats, examples. The majority of the test machinery is in the discipline package in the laws module. I kind of feel that the discipline is good enough as a test-kits for cats.

    P.S. alleycats also has tests consistent to cats, it just needed to copy this file out.

  2. ceedubs commented on Sep 21, 2017

    @ceedubs
    Contributor

    @kailuowang was this resolved by #1758?

  3. kailuowang commented on Sep 21, 2017

    @kailuowang
    Contributor

    yes, closing.

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

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions