Skip to content

No unit tests/CI #83

Closed
Closed

Description

@sgreene570

This would be a nice touch for contributors!

Activity

  1. josegonzalez commented on Feb 19, 2018

    @josegonzalez
    Owner

    @sgreene570 pull requests welcome!

  2. sgreene570 commented on Feb 20, 2018

    @sgreene570
    ContributorAuthor

    @josegonzalez working on setting travis up. Trying to figure out who to use as a test user and what to do for that user's access key.

  3. josegonzalez commented on Feb 20, 2018

    @josegonzalez
    Owner

    A better method would probably be to just mock out the api response calls.

  4. Iamrodos commented on Nov 17, 2025

    @Iamrodos
    Contributor

    Looks like this has been addressed. Lint workflow was added a long time ago and some unit tests have just been added. My suggested tests for PR #449 mock the api response calls. Without a token you can't do a live test from CI as you only get 60 per hour.

    Can this now be closed?

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

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions