Skip to content

Getting rid of GetAwaiter().GetResult() calls in tests by marking them async#224

Merged
jeremydmiller merged 1 commit intoJasperFx:masterfrom
danielmarbach:NoMoreGetAwaiter
Mar 16, 2016
Merged

Getting rid of GetAwaiter().GetResult() calls in tests by marking them async#224
jeremydmiller merged 1 commit intoJasperFx:masterfrom
danielmarbach:NoMoreGetAwaiter

Conversation

@danielmarbach
Copy link
Copy Markdown
Contributor

Since the testing framework properly supports async tests we don't need to use GetAwaiter().GetResult()

@jeremydmiller jeremydmiller modified the milestone: v0.9 Mar 16, 2016
jeremydmiller added a commit that referenced this pull request Mar 16, 2016
Getting rid of GetAwaiter().GetResult() calls in tests by marking them async
@jeremydmiller jeremydmiller merged commit 522051a into JasperFx:master Mar 16, 2016
jeremydmiller added a commit that referenced this pull request Mar 23, 2026
Reproduces the user's scenario from Weasel issue #224 where a
FlatTableProjection with a mixed-case table name and unique index
fails with "relation already exists" on the second migration run.

See JasperFx/weasel#224

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
jeremydmiller added a commit that referenced this pull request Mar 23, 2026
Reproduces the user's scenario from Weasel issue #224 where a
FlatTableProjection with a mixed-case table name and unique index
fails with "relation already exists" on the second migration run.

See JasperFx/weasel#224

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
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