liquid-labs / liquid-labs/bash-toolkit

Rework test to be faster.

Open
#24 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
JavaScript
Stars
1
Forks
1
PR merge metrics
No merged PRs in 30d

Description

## Overview

The test is slow in part (I assume) due to constantly starting new shell processes and re-generating the imports. There's a good chance that if we generate the whole code bundle once at the beginning (which itself, would be a good test) and then execute the sub-tests by still calling `shell.exec`, but not needing to regenerate the code bundle, we could save some time.

## Deliverables

* [ ] Rework tests to generate the imported code-bundle in one go.

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Locate the test that repeatedly invokes shell.exec and regenerates the imported code-bundle, then run it to establish the current behavior and timing. Generate the bundle once and have the sub-tests reuse it; done means the existing test behavior remains covered with less process and generation overhead.

Written by the indexing model from the issue text.

Assessment

Tech stack
javascript, shell
Domain
testing-qa
Issue type
Refactor
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.