Skip to content

Fix browser tests by importing the shared tests instead of exporting#196

Merged
Goodwine merged 1 commit intomainfrom
fix-ci
Mar 26, 2026
Merged

Fix browser tests by importing the shared tests instead of exporting#196
Goodwine merged 1 commit intomainfrom
fix-ci

Conversation

@Goodwine
Copy link
Copy Markdown
Contributor

Previously we called 'export' to append main() into the test file. Dart Test package 1.29.0 now disallows this pattern. TIL that the test package is released separate from the main Dart binary!

Fixes #195

Previously we called 'export' to append main() into the test file. Dart Test package 1.29.0 now disallows this pattern. TIL that the test package is released separate from the main Dart binary!

Fixes #195

void main() {
shared_browser_tests.main();
}
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As I haven't been working with Dart, I'm not sure if this is better over void main() => shared_browser_tests.main() 🤔

I kinda like it better like this instead of an arrow function, but maybe that's my Go-brain talking lol

@Goodwine Goodwine requested a review from nex3 March 25, 2026 23:49
@Goodwine Goodwine merged commit 104cd54 into main Mar 26, 2026
17 checks passed
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.

CI is currently failing due to breaking change in dart's test package

2 participants