Skip to content

Add testLawsGroup QuickCheck utility function - #703

Open
koslambrou wants to merge 1 commit into
masterfrom
koslambrou/qc-testlawsgroup
Open

koslambrou wants to merge 1 commit into
masterfrom
koslambrou/qc-testlawsgroup

Conversation

@koslambrou

Copy link
Copy Markdown

Description

This should be used instead of Test.QuickCheck.Classes.lawsCheckOne, which reports through Test.QuickCheck.quickCheck. That writes straight to stdout, thus it is not properly indented in the test console output. It also discards the QuickCheck Result, which means that a violated law does not fail the test suite.

Checklist

  • Commit sequence broadly makes sense and commits have useful messages
  • New tests are added if needed and existing tests are updated
  • All visible changes are prepended to the latest section of a CHANGELOG.md for the affected packages.
    New section is never added with the code changes. (See RELEASING.md)
  • When applicable, versions are updated in .cabal and CHANGELOG.md files according to the
    versioning process.
  • The version bounds in .cabal files for all affected packages are updated.
    If you change the bounds in a cabal file, that package itself must have a version increase. (See RELEASING.md)
  • Self-reviewed the diff

@koslambrou
koslambrou requested a review from lehins October 3, 2026 01:38
@koslambrou
koslambrou force-pushed the koslambrou/qc-testlawsgroup branch from 82d0124 to db88a9b Compare October 3, 2026 01:38

This branch has not been deployed

No deployments
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.

1 participant