TopoToolbox / TopoToolbox/libtopotoolbox

Run tests to completion even if one property fails.

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

Nobody has claimed this yet.

test
Dominant language
C
Stars
2
Forks
11
PR merge metrics
No merged PRs in 30d

Description

We do this in the snapshot tests by updating a success flag on each test, and we should use that approach in the random DEM tests rather than the failed assert statements, which crash the program.

Contributor guide

Open the contributing guide

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

Compare the snapshot tests' success-flag handling with the random DEM tests, and locate the failed assert statements that stop execution. Change the random DEM test flow to retain failures while continuing through each property, then run the relevant test suite and confirm all properties are exercised before the final result is reported.

Written by the indexing model from the issue text.

Assessment

Tech stack
c
Domain
testing
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.