Skip to content
This repository was archived by the owner on Feb 28, 2021. It is now read-only.
This repository was archived by the owner on Feb 28, 2021. It is now read-only.

Unit Testing #65

Open
Open

Description

@spiray

Hello, I think it will be very beneficial for this library to have unit tests which will automatically run on build and possibly use CI. What is the approach on this?

Activity

  1. spiray commented on Jun 8, 2018

    @spiray
    ContributorAuthor

    @MathuraMG @lm-n I wanted to get started with testing/CI as i think it will help a lot with making the contribution process smoother.

    • We will have tests
    • The CI can test each PR and then build.

    Here are my questions -

    1. Do you have any preferences on Jest Vs. Mocha?
    2. Do you have any preference on a CI platform (CircleCI, TravisCI, or Other)? If one is chosen can one of you create an account that can be used?
    3. Is there a specific part of the project that should have tests written for it first?

    Please let me know what you think. I will create a separate issue for CI.

    Thanks!

  2. MathuraMG commented on Jun 16, 2018

    @MathuraMG
    Collaborator

    @spiray - Sorry for the delay! Last week was a bit of a mess.

    1. So I don't have a ton of perspective on this. I do know that Jest os probably the more popular choice. Do you have a specific preference to one over the other?
    2. Same as the above, really. Let's maybe stick to what os being used in the other p5 projects? Its circle CI right?
    3. Let's get started with the interceptors first?

    @lm-n thoughts?

  3. MathuraMG commented on Jul 5, 2018

    @MathuraMG
    Collaborator

    hey @spiray - just wanted to follow up on this. Do the above comments make sense to start with the tests?

  4. spiray commented on Jul 5, 2018

    @spiray
    ContributorAuthor

    Yes! I am actually working on it right now. I am just setting up the environment based off of the way it is setup in the p5.js Sound library. Ill keep you updated.

  5. lm-n commented on Jul 5, 2018

    @lm-n
    Member
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions