Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Unit tests passing on both Windows and Wine #1016

Merged
merged 2 commits into from
Nov 13, 2019

Conversation

rvl
Copy link
Contributor

@rvl rvl commented Nov 13, 2019

Relates to #703.

Overview

Fixes/disables some of the last tests that were failing on Windows for Microsoft reasons.

@rvl rvl self-assigned this Nov 13, 2019
@@ -47,6 +47,7 @@ spec = describe "API Server" $ do
res `shouldBe` Left ListenErrorOperationNotPermitted

it "handles port in use" $ do
skipOnWindows "Windows permits listening on same port multiple times"
Copy link
Member

Choose a reason for hiding this comment

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

What... ?

@KtorZ
Copy link
Member

KtorZ commented Nov 13, 2019

bors r+

iohk-bors bot added a commit that referenced this pull request Nov 13, 2019
1016: Unit tests passing on both Windows and Wine r=KtorZ a=rvl

Relates to #703. 

# Overview

Fixes/disables some of the last tests that were failing on Windows for Microsoft reasons.


1017: Update stylish-haskell to latest (v0.9.4.4) r=KtorZ a=rvl

Updates stylish-haskell to the latest.

I mistakenly thought that the latest version in a Stackage LTS would be recent and downgraded it before.

To test:

```
nix-shell --run "stylish-haskell --version"
```


Co-authored-by: Rodney Lorrimar <[email protected]>
@iohk-bors
Copy link
Contributor

iohk-bors bot commented Nov 13, 2019

Build succeeded

@iohk-bors iohk-bors bot merged commit 8937f69 into master Nov 13, 2019
@rvl rvl deleted the rvl/703/windows-unit-tests branch November 14, 2019 00:00
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