Skip to content

Implement tests for src#2865

Open
leolost2605 wants to merge 2 commits into
mainfrom
leolost/src-tests
Open

Implement tests for src#2865
leolost2605 wants to merge 2 commits into
mainfrom
leolost/src-tests

Conversation

@leolost2605

Copy link
Copy Markdown
Member

Add a TestRunner where we register the test types instead of doing it manually in main.
Add a simple test for the transitionbuilder that makes sure it gets freed even it's not run.

@leolost2605
leolost2605 requested a review from a team July 8, 2026 13:16
@lenemter

lenemter commented Jul 8, 2026

Copy link
Copy Markdown
Member

TBH TransitionBuilder should be moved to the library, as it doesn't seem to use anything from Gala itself

I have a branch with tests for gala here: lenemter#3, but it still needs a lot of work to figure out how to test things like the dock

@leolost2605

leolost2605 commented Jul 8, 2026

Copy link
Copy Markdown
Member Author

TBH TransitionBuilder should be moved to the library, as it doesn't seem to use anything from Gala itself

Yeah ig it was just an easy thing to test for an initial implementation. We could do that in a follow up?

I have a branch with tests for gala here: lenemter#3, but it still needs a lot of work to figure out how to test things like the dock

I had a quick look and there seems to be a lot of unrelated things? This PR just tries to get the initial infrastructure in place. Does it look like a similar direction or would you do it differently?

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