Skip to content

refactor: delete ~1100 redundant parse-only tests from compile.test.ts #72

@michaeldistel

Description

@michaeldistel

tests/integration/compile.test.ts lines ~537–1665 contain pure parse() smoke tests that only assert result.ast defined + errors.toHaveLength(0).

The scenarios (arithmetic, control flow, literals, variable declarations) are already fully working and covered at the parser level in tests/frontend/parser.test.ts. They were never promoted to assert anything meaningful and should be deleted.

If integration-level assertions are ever wanted for these scenarios (e.g. "addition generates x.get() + y.get()"), they belong in the relevant codegen-*.test.ts file, not here.

Sections to delete:

  • Expression Parsing Tests
  • Chained Expression Tests
  • Literal Types Tests
  • Variable Declaration Tests
  • Control Flow Parsing Tests
  • Multiple POU Tests
  • Edge Case Tests

The first ~536 lines and lines 1666+ have real integration value and should be kept.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions