Skip to content

fix: add integration-tests executable target to lakefile#8

Merged
Oppen merged 2 commits into
mainfrom
fix/lake-test-driver
Jun 10, 2026
Merged

fix: add integration-tests executable target to lakefile#8
Oppen merged 2 commits into
mainfrom
fix/lake-test-driver

Conversation

@Oppen

@Oppen Oppen commented Apr 27, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Adds a [[lean_exe]] target for Tests.IntegrationTests so the test suite can be built and run via lake exe integration-tests
  • Previously, lake test (and lake build Tests) failed or had no runnable entry point despite the test file having a main function with 33 tests

Test plan

  • lake build integration-tests — builds without errors
  • lake exe integration-tests — prints 33/33 tests passed

lake test previously errored with "no test driver configured".
Adds a [[lean_exe]] target for Tests.IntegrationTests and wires it
as the testDriver so `lake test` runs the 33-test suite.
CompletenessSpec.lean: add `omit [...] in` modifiers before the 10
private/public theorems that don't use all section variables
(NodeOps, BEq, Hashable, LawfulBEq, LawfulHashable).

Tests/IntegrationTests.lean: remove stale simp arguments
(NodeOps.children, NodeOps.replaceChildren, NodeOps.mapChildren, hlen)
from replaceChildren_children and replaceChildren_sameShape instance proofs.
@Oppen Oppen merged commit f34ebd1 into main Jun 10, 2026
@Oppen Oppen deleted the fix/lake-test-driver branch June 10, 2026 18:45
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.

2 participants