Skip to content

Commit

Permalink
ignore simple-multi-def-test for windows since #4270
Browse files Browse the repository at this point in the history
  • Loading branch information
soulomoon committed Jul 1, 2024
1 parent 495af1f commit 67680ea
Showing 1 changed file with 9 additions and 1 deletion.
10 changes: 9 additions & 1 deletion ghcide/test/exe/CradleTests.hs
Original file line number Diff line number Diff line change
Expand Up @@ -29,6 +29,8 @@ import Language.LSP.Protocol.Types hiding
import Language.LSP.Test
import System.FilePath
import System.IO.Extra hiding (withTempDir)
import Test.Hls.Util (EnvSpec (..), OS (..),
ignoreInEnv)
import Test.Tasty
import Test.Tasty.HUnit

Expand Down Expand Up @@ -169,7 +171,8 @@ simpleMultiTest3 variant =

-- Like simpleMultiTest but open the files in component 'a' in a separate session
simpleMultiDefTest :: FilePath -> TestTree
simpleMultiDefTest variant = testCase (multiTestName variant "def-test") $ runWithExtraFiles variant $ \dir -> do
simpleMultiDefTest variant = ignoreForWindows $ testCase testName $
runWithExtraFiles variant $ \dir -> do
let aPath = dir </> "a/A.hs"
bPath = dir </> "b/B.hs"
adoc <- liftIO $ runInDir dir $ do
Expand All @@ -184,6 +187,11 @@ simpleMultiDefTest variant = testCase (multiTestName variant "def-test") $ runWi
let fooL = mkL (adoc ^. L.uri) 2 0 2 3
checkDefs locs (pure [fooL])
expectNoMoreDiagnostics 0.5
where
testName = multiTestName variant "def-test"
ignoreForWindows
| testName == "simple-multi-def-test" = ignoreInEnv [HostOS Windows] "Test is flaky on Windows, see #4270"
| otherwise = id

multiRexportTest :: TestTree
multiRexportTest =
Expand Down

0 comments on commit 67680ea

Please sign in to comment.