Skip to content

Unify test utils#42

Merged
mauriciogardini merged 16 commits intofeature/ef-pythonfrom
feature/test-output
Aug 14, 2025
Merged

Unify test utils#42
mauriciogardini merged 16 commits intofeature/ef-pythonfrom
feature/test-output

Conversation

@mauriciogardini
Copy link

@mauriciogardini mauriciogardini commented Aug 14, 2025 •
edited
Loading

Unify test utils so all tests share a single interface, that defines all the correct parameters for the Output (externals, namePolicy, etc.).

The interface is:expect(getOutput(program, [<children>])).toRenderTo(<text>)

program can be either a BasicTestRunner, defined with createEmitterFrameworkTestRunner(); or a custom Program, based in a TypeSpec schema, defined with getProgram(<schema text>).

This PR was already approved at #40, but merged incorrectly.

@mauriciogardini mauriciogardini marked this pull request as ready for review August 14, 2025 16:57
@mauriciogardini mauriciogardini changed the title Feature/test output Re-merge test-output Aug 14, 2025
@mauriciogardini mauriciogardini changed the title Re-merge test-output Unify test utils Aug 14, 2025
@mauriciogardini mauriciogardini merged commit 2006978 into feature/ef-python Aug 14, 2025
steverice pushed a commit that referenced this pull request Sep 26, 2025
@mauriciogardini @steverice
Unify test utils (#42)
221fad3 
* Add Atom

* Add typing

* Fix

* Fix

* Add typing

* Fix

* Unify test utils

* pnpm format

* Remove exports

* Fix comment

* Add TODOs
mauriciogardini added a commit that referenced this pull request Nov 10, 2025
@mauriciogardini @swatkatz
Add models and interfaces (#45)
75acb6b 
* Add alloy-js/python dependency

* Add python scaffolding

* Add starting test structure (#34)

* Add Atom

* Add initial test structure

* Add Atom

* Add initial test structure

* Add file to index

* Add python scaffolding

* Add Atom

* Add initial test structure

* Add file to index

* pnpm format

---------

Co-authored-by: swatikumar <swatikumar@pinterest.com>

* Add Atom (#36)

* Add Atom

* Fix

* Add typing (#38)

* Add Atom

* Add typing

* Fix

* Fix

* Add typing

* Fix

* Fix comment

* Add TODOs

* Unify test utils (#42)

* Add Atom

* Add typing

* Fix

* Fix

* Add typing

* Fix

* Unify test utils

* pnpm format

* Remove exports

* Fix comment

* Add TODOs

* Add models and interfaces

* Remove file

* Changes

* Missing files

* Add Literal import

* Accounting for feedback

* Handle floats in a better way

* Accounting for feedback

* Fix

* Update

* Rebase

* Fix tests

* Fix

* Add comments

* Remove typing module import

* Fix

* Fix

* Fix

* Changing docstrings

---------

Co-authored-by: swatikumar <swatikumar@pinterest.com>
Co-authored-by: Swati Kumar <swati.kumar.a@gmail.com>
mauriciogardini added a commit that referenced this pull request Nov 10, 2025
@mauriciogardini
Unify test utils (#42)
f7924f5 
* Add Atom

* Add typing

* Fix

* Fix

* Add typing

* Fix

* Unify test utils

* pnpm format

* Remove exports

* Fix comment

* Add TODOs
mauriciogardini added a commit that referenced this pull request Dec 15, 2025
@mauriciogardini
Unify test utils (#42)
16494f7 
* Add Atom

* Add typing

* Fix

* Fix

* Add typing

* Fix

* Unify test utils

* pnpm format

* Remove exports

* Fix comment

* Add TODOs
mauriciogardini added a commit that referenced this pull request Dec 15, 2025
@mauriciogardini @swatkatz
Add models and interfaces (#45)
538ce11 
* Add alloy-js/python dependency

* Add python scaffolding

* Add starting test structure (#34)

* Add Atom

* Add initial test structure

* Add Atom

* Add initial test structure

* Add file to index

* Add python scaffolding

* Add Atom

* Add initial test structure

* Add file to index

* pnpm format

---------

Co-authored-by: swatikumar <swatikumar@pinterest.com>

* Add Atom (#36)

* Add Atom

* Fix

* Add typing (#38)

* Add Atom

* Add typing

* Fix

* Fix

* Add typing

* Fix

* Fix comment

* Add TODOs

* Unify test utils (#42)

* Add Atom

* Add typing

* Fix

* Fix

* Add typing

* Fix

* Unify test utils

* pnpm format

* Remove exports

* Fix comment

* Add TODOs

* Add models and interfaces

* Remove file

* Changes

* Missing files

* Add Literal import

* Accounting for feedback

* Handle floats in a better way

* Accounting for feedback

* Fix

* Update

* Rebase

* Fix tests

* Fix

* Add comments

* Remove typing module import

* Fix

* Fix

* Fix

* Changing docstrings

---------

Co-authored-by: swatikumar <swatikumar@pinterest.com>
Co-authored-by: Swati Kumar <swati.kumar.a@gmail.com>
mauriciogardini added a commit that referenced this pull request Dec 18, 2025
@mauriciogardini
Unify test utils (#42)
4295db1 
* Add Atom

* Add typing

* Fix

* Fix

* Add typing

* Fix

* Unify test utils

* pnpm format

* Remove exports

* Fix comment

* Add TODOs
mauriciogardini added a commit that referenced this pull request Dec 18, 2025
@mauriciogardini @swatkatz
Add models and interfaces (#45)
f91a93e 
* Add alloy-js/python dependency

* Add python scaffolding

* Add starting test structure (#34)

* Add Atom

* Add initial test structure

* Add Atom

* Add initial test structure

* Add file to index

* Add python scaffolding

* Add Atom

* Add initial test structure

* Add file to index

* pnpm format

---------

Co-authored-by: swatikumar <swatikumar@pinterest.com>

* Add Atom (#36)

* Add Atom

* Fix

* Add typing (#38)

* Add Atom

* Add typing

* Fix

* Fix

* Add typing

* Fix

* Fix comment

* Add TODOs

* Unify test utils (#42)

* Add Atom

* Add typing

* Fix

* Fix

* Add typing

* Fix

* Unify test utils

* pnpm format

* Remove exports

* Fix comment

* Add TODOs

* Add models and interfaces

* Remove file

* Changes

* Missing files

* Add Literal import

* Accounting for feedback

* Handle floats in a better way

* Accounting for feedback

* Fix

* Update

* Rebase

* Fix tests

* Fix

* Add comments

* Remove typing module import

* Fix

* Fix

* Fix

* Changing docstrings

---------

Co-authored-by: swatikumar <swatikumar@pinterest.com>
Co-authored-by: Swati Kumar <swati.kumar.a@gmail.com>
mauriciogardini added a commit that referenced this pull request Dec 19, 2025
@mauriciogardini
Unify test utils (#42)
fd65b54 
* Add Atom

* Add typing

* Fix

* Fix

* Add typing

* Fix

* Unify test utils

* pnpm format

* Remove exports

* Fix comment

* Add TODOs
mauriciogardini added a commit that referenced this pull request Dec 19, 2025
@mauriciogardini @swatkatz
Add models and interfaces (#45)
9446d70 
* Add alloy-js/python dependency

* Add python scaffolding

* Add starting test structure (#34)

* Add Atom

* Add initial test structure

* Add Atom

* Add initial test structure

* Add file to index

* Add python scaffolding

* Add Atom

* Add initial test structure

* Add file to index

* pnpm format

---------

Co-authored-by: swatikumar <swatikumar@pinterest.com>

* Add Atom (#36)

* Add Atom

* Fix

* Add typing (#38)

* Add Atom

* Add typing

* Fix

* Fix

* Add typing

* Fix

* Fix comment

* Add TODOs

* Unify test utils (#42)

* Add Atom

* Add typing

* Fix

* Fix

* Add typing

* Fix

* Unify test utils

* pnpm format

* Remove exports

* Fix comment

* Add TODOs

* Add models and interfaces

* Remove file

* Changes

* Missing files

* Add Literal import

* Accounting for feedback

* Handle floats in a better way

* Accounting for feedback

* Fix

* Update

* Rebase

* Fix tests

* Fix

* Add comments

* Remove typing module import

* Fix

* Fix

* Fix

* Changing docstrings

---------

Co-authored-by: swatikumar <swatikumar@pinterest.com>
Co-authored-by: Swati Kumar <swati.kumar.a@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@mauriciogardini