ml5js/ml5-library

[devops] TravisCI tests fail more often than not - Need to simplify testing and mock network requests

Open

#637 opened on Oct 16, 2019

View on GitHub
 (5 comments) (0 reactions) (0 assignees)JavaScript (6,136 stars) (905 forks)batch import
bugdevopshelp wantedhigh priority

Description

Dear ml5 community,

I'm submitting a new issue. Please see the details below.

→ Step 1: Describe the issue 📝

Did you find a bug? Want to suggest an idea for feature?

  • Want to suggest an idea

Currently the TravisCI tests fail more often than they pass. This has more to do with the fetching of images for analysis, the instantiation of models making big requests to external pretrained models from google or github servers, etc.

It would be wonderful if we can get this under control so that we can also start making a comprehensive set of tests for our features.

Contributor guide

Tech stack
javascript
Domain
devopstesting
Issue type
test
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
3
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
3-5 days
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
stale
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
clear
Prerequisites
JavaScript testingTravisCI configurationHTTP mockingml5 basics
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
55
Research direction
First, examine the existing test suite in the repository to identify which tests make external network requests (e.g., fetching images or model files). Look for opportunities to mock these requests using libraries like nock or sinon. Review the TravisCI configuration file (.travis.yml) to understand the current build process. Consider simplifying the tests by using local fixtures or pre downloaded models to avoid network dependencies. Check issue comments for any specific suggestions or prior attempts.