mobile-dev-inc/maestro

`maestro record` doesn't respect the `--no-ansi` flag

Open

#1,009 opened on Apr 21, 2023

View on GitHub
 (3 comments) (0 reactions) (0 assignees)Kotlin (306 forks)batch import
buggood first issuemaestro cli

Repository metrics

Stars
 (6,111 stars)
PR merge metrics
 (Avg merge 12d 2h) (39 merged PRs in 30d)

Description

While using maestro record, I noticed that it seems to ignore the --no-ansi option. When using maestro test, it works just fine. Currently using Maestro version 1.26.1.

Command that doesn't work;

maestro --no-ansi record {flow path}

Command that does work;

maestro --no-ansi test {flow path}

https://user-images.githubusercontent.com/68027680/233732432-d2217c7e-5f0f-40e8-9b71-d58386cb9f43.mp4

Contributor guide

Research direction
Compare how `maestro test` and `maestro record` handle the ` no ansi` flag in the codebase. Check CLI argument parsing and find where ansi output is suppressed or not.
Tech stack
kotlin
Domain
mobiletestingcli
Issue type
Bug
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
2
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
1-3 hours
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
Active
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
Clear
Prerequisites
Gitcommand line
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
70