Skip to content

chore: add test-verbose Makefile target for real-time test output#2835

Merged
crivetimihai merged 4 commits intomainfrom
chore/issue-2665-test-verbose-cleanup
Feb 11, 2026
Merged

chore: add test-verbose Makefile target for real-time test output#2835
crivetimihai merged 4 commits intomainfrom
chore/issue-2665-test-verbose-cleanup

Conversation

@shoummu1
Copy link
Copy Markdown
Collaborator

Description

Adds a new test-verbose Makefile target for sequential test execution with real-time test name visibility, addressing developer needs during debugging sessions.

Related Issue

Closes: #2665

Changes

  • Added test-verbose target for sequential execution (--maxfail=1 -v --tb=short)
  • Updated Makefile help text to document the new target
  • Added test-verbose to .PHONY declaration
  • Existing make test behavior remains unchanged for CI/CD compatibility

Usage

# Fast parallel execution (existing behavior)
make test

# Sequential with real-time test names (new)
make test-verbose

Testing

  • Verified test-verbose runs sequentially and displays test names in real-time
  • Confirmed test target unchanged (16 workers, parallel execution)
  • Help text displays correctly with make help

@shoummu1 shoummu1 marked this pull request as ready for review February 11, 2026 10:15
@shoummu1 shoummu1 force-pushed the chore/issue-2665-test-verbose-cleanup branch from 395fd3c to 0843fad Compare February 11, 2026 10:48
shoummu1 and others added 4 commits February 11, 2026 11:16
…output

Signed-off-by: Shoumi <shoumimukherjee@gmail.com>
Signed-off-by: Shoumi <shoumimukherjee@gmail.com>
Signed-off-by: Shoumi <shoumimukherjee@gmail.com>
Run all tests to completion showing failures inline in real-time,
rather than stopping at the first failure.

Signed-off-by: Mihai Criveti <crivetimihai@gmail.com>
@crivetimihai crivetimihai force-pushed the chore/issue-2665-test-verbose-cleanup branch from 0843fad to 259e2c4 Compare February 11, 2026 11:18
@crivetimihai
Copy link
Copy Markdown
Member

Thanks for the contribution! Rebased onto latest main and made one adjustment:

  • Changed --maxfail=1 to --maxfail=0 so all tests run to completion instead of stopping at the first failure
  • Added --instafail flag to the pytest command — this is what actually activates the pytest-instafail dependency you added, showing failure tracebacks inline as they happen rather than at the end

Without --instafail, the plugin was installed but unused. And with --maxfail=1, --instafail would have been redundant since the run stops immediately anyway.

@crivetimihai crivetimihai self-assigned this Feb 11, 2026
@crivetimihai crivetimihai merged commit cb27109 into main Feb 11, 2026
47 checks passed
@crivetimihai crivetimihai deleted the chore/issue-2665-test-verbose-cleanup branch February 11, 2026 11:20
ja8zyjits pushed a commit that referenced this pull request Feb 13, 2026
)

* add test-verbose target for sequential test execution with real-time output

Signed-off-by: Shoumi <shoumimukherjee@gmail.com>

* additional enhancements

Signed-off-by: Shoumi <shoumimukherjee@gmail.com>

* minor fix

Signed-off-by: Shoumi <shoumimukherjee@gmail.com>

* fix: use --instafail and --maxfail=0 in test-verbose target

Run all tests to completion showing failures inline in real-time,
rather than stopping at the first failure.

Signed-off-by: Mihai Criveti <crivetimihai@gmail.com>

---------

Signed-off-by: Shoumi <shoumimukherjee@gmail.com>
Signed-off-by: Mihai Criveti <crivetimihai@gmail.com>
Co-authored-by: Mihai Criveti <crivetimihai@gmail.com>
kcostell06 pushed a commit to kcostell06/mcp-context-forge that referenced this pull request Feb 24, 2026
…M#2835)

* add test-verbose target for sequential test execution with real-time output

Signed-off-by: Shoumi <shoumimukherjee@gmail.com>

* additional enhancements

Signed-off-by: Shoumi <shoumimukherjee@gmail.com>

* minor fix

Signed-off-by: Shoumi <shoumimukherjee@gmail.com>

* fix: use --instafail and --maxfail=0 in test-verbose target

Run all tests to completion showing failures inline in real-time,
rather than stopping at the first failure.

Signed-off-by: Mihai Criveti <crivetimihai@gmail.com>

---------

Signed-off-by: Shoumi <shoumimukherjee@gmail.com>
Signed-off-by: Mihai Criveti <crivetimihai@gmail.com>
Co-authored-by: Mihai Criveti <crivetimihai@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[CHORE][PYTEST]: Add verbose test output option for real-time test name visibility

2 participants