packages feed

tricorder-0.1.1.0: test/Unit/Tricorder/Effects/GhciSession/GhciParserSpec.hs

module Unit.Tricorder.Effects.GhciSession.GhciParserSpec (spec_GhciParser) where

import Test.Hspec

import Tricorder.BuildState (Diagnostic (..), Severity (..))
import Tricorder.Effects.GhciSession.GhciParser
    ( GhciLoad (..)
    , GhciLoading (..)
    , GhciMessage (..)
    , GhciSeverity (..)
    , LoadOutcome (..)
    , LoadResult (..)
    , Position (..)
    , collectResult
    , collectResultCustom
    , parseReload
    , parseShowModules
    , parseShowTargets
    )


spec_GhciParser :: Spec
spec_GhciParser = do
    describe "parseReload" do
        describe "clean build" testCleanBuild
        describe "with errors and warnings" testErrors
        describe "with -fhide-source-paths (no Loading items)" testHideSourcePaths
        describe "with <no location info> errors" testNoLocationInfo
        describe "with Loaded GHCi configuration" testLoadedConfig

    describe "parseShowModules" do
        describe "typical output" testShowModules
        describe "empty / blank input" testShowModulesEmpty

    describe "parseShowTargets" testShowTargets

    describe "collectResultCustom" do
        describe "<no location info> plugin load failure" testPluginLoadFailure

    describe "collectResult" do
        describe "failed load with no located error" testUnattributedFailure


--------------------------------------------------------------------------------
-- parseReload: clean build
--------------------------------------------------------------------------------

testCleanBuild :: Spec
testCleanBuild = do
    it "produces GLoading items for each compiled module" do
        let input =
                [ "[1 of 3] Compiling Tricorder.BuildState ( src/Tricorder/BuildState.hs, interpreted )"
                , "[2 of 3] Compiling Tricorder.Session    ( src/Tricorder/Session.hs, interpreted )"
                , "[3 of 3] Compiling Main                 ( app/Main.hs, interpreted )"
                , "Ok, 3 modules loaded."
                ]
        parseReload input
            `shouldBe` [ GLoading GhciLoading {index = 1, total = 3, moduleName = "Tricorder.BuildState", sourceFile = "src/Tricorder/BuildState.hs"}
                       , GLoading GhciLoading {index = 2, total = 3, moduleName = "Tricorder.Session", sourceFile = "src/Tricorder/Session.hs"}
                       , GLoading GhciLoading {index = 3, total = 3, moduleName = "Main", sourceFile = "app/Main.hs"}
                       , GSummary LoadSucceeded
                       ]

    it "handles padded module index (e.g. [ 1 of 47])" do
        let input =
                [ "[ 1 of 47] Compiling Main              ( app/Main.hs, interpreted )"
                , "Ok, 1 module loaded."
                ]
        parseReload input
            `shouldBe` [ GLoading GhciLoading {index = 1, total = 47, moduleName = "Main", sourceFile = "app/Main.hs"}
                       , GSummary LoadSucceeded
                       ]

    describe "when only summary line" $ it "returns the summary outcome" do
        parseReload ["Ok, 0 modules loaded."] `shouldBe` [GSummary LoadSucceeded]


--------------------------------------------------------------------------------
-- parseReload: errors and warnings
--------------------------------------------------------------------------------

testErrors :: Spec
testErrors = do
    it "parses a single-line error" do
        let input = ["src/Foo.hs:10:5: error: Variable not in scope: foo"]
        parseReload input
            `shouldBe` [GMessage GhciMessage {severity = GError, file = "src/Foo.hs", startPos = Position 10 5, endPos = Position 10 5, messageLines = ["src/Foo.hs:10:5: error: Variable not in scope: foo"]}]

    it "parses a warning with continuation lines" do
        let input =
                [ "src/Bar.hs:20:3: warning: [-Wunused-imports]"
                , "    Redundant import: Data.List"
                , "    Perhaps you want to remove it."
                ]
        parseReload input
            `shouldBe` [ GMessage
                            GhciMessage
                                { severity = GWarning
                                , file = "src/Bar.hs"
                                , startPos = Position 20 3
                                , endPos = Position 20 3
                                , messageLines =
                                    [ "src/Bar.hs:20:3: warning: [-Wunused-imports]"
                                    , "    Redundant import: Data.List"
                                    , "    Perhaps you want to remove it."
                                    ]
                                }
                       ]

    it "parses a span position (L:C-C2:)" do
        let input = ["src/Baz.hs:5:1-10: error: Parse error"]
        parseReload input
            `shouldBe` [GMessage GhciMessage {severity = GError, file = "src/Baz.hs", startPos = Position 5 1, endPos = Position 5 10, messageLines = ["src/Baz.hs:5:1-10: error: Parse error"]}]

    it "parses a span position ((L1,C1)-(L2,C2):)" do
        let input = ["src/Qux.hs:(3,1)-(5,20): error: Multi-line error"]
        parseReload input
            `shouldBe` [GMessage GhciMessage {severity = GError, file = "src/Qux.hs", startPos = Position 3 1, endPos = Position 5 20, messageLines = ["src/Qux.hs:(3,1)-(5,20): error: Multi-line error"]}]

    it "parses a span position with double-paren end ((L1,C1)-((L2,C2):)" do
        let input = ["src/Qux.hs:(3,1)-((5,20): error: Multi-line error"]
        parseReload input
            `shouldBe` [GMessage GhciMessage {severity = GError, file = "src/Qux.hs", startPos = Position 3 1, endPos = Position 5 20, messageLines = ["src/Qux.hs:(3,1)-((5,20): error: Multi-line error"]}]

    it "parses source-display continuation lines (pipe format)" do
        let input =
                [ "src/Foo.hs:10:5: error: Variable not in scope: foo"
                , "   |"
                , "10 | foo bar"
                , "   | ^^^"
                , "    Suggested fix: import Foo"
                ]
        parseReload input
            `shouldBe` [ GMessage
                            GhciMessage
                                { severity = GError
                                , file = "src/Foo.hs"
                                , startPos = Position 10 5
                                , endPos = Position 10 5
                                , messageLines =
                                    [ "src/Foo.hs:10:5: error: Variable not in scope: foo"
                                    , "   |"
                                    , "10 | foo bar"
                                    , "   | ^^^"
                                    , "    Suggested fix: import Foo"
                                    ]
                                }
                       ]

    it "strips ANSI codes from header for matching but stores original in glMessage" do
        let ansiHeader = "\ESC[1msrc/Foo.hs:10:5:\ESC[0m \ESC[91merror:\ESC[0m Variable not in scope: foo"
        parseReload [ansiHeader]
            `shouldBe` [GMessage GhciMessage {severity = GError, file = "src/Foo.hs", startPos = Position 10 5, endPos = Position 10 5, messageLines = [ansiHeader]}]

    it "parses a Windows drive-letter path in a diagnostic" do
        let input = ["C:\\path\\file.hs:10:5: error: Variable not in scope: foo"]
        parseReload input
            `shouldBe` [GMessage GhciMessage {severity = GError, file = "C:\\path\\file.hs", startPos = Position 10 5, endPos = Position 10 5, messageLines = ["C:\\path\\file.hs:10:5: error: Variable not in scope: foo"]}]

    it "parses mixed Loading, Message, and summary items" do
        let input =
                [ "[1 of 2] Compiling Lib ( src/Lib.hs, interpreted )"
                , "src/Lib.hs:5:1: error: Oops"
                , "[2 of 2] Compiling Main ( app/Main.hs, interpreted )"
                , "Failed, 1 module loaded."
                ]
        parseReload input
            `shouldBe` [ GLoading GhciLoading {index = 1, total = 2, moduleName = "Lib", sourceFile = "src/Lib.hs"}
                       , GMessage GhciMessage {severity = GError, file = "src/Lib.hs", startPos = Position 5 1, endPos = Position 5 1, messageLines = ["src/Lib.hs:5:1: error: Oops"]}
                       , GLoading GhciLoading {index = 2, total = 2, moduleName = "Main", sourceFile = "app/Main.hs"}
                       , GSummary LoadFailed
                       ]


--------------------------------------------------------------------------------
-- parseReload: -fhide-source-paths output
--------------------------------------------------------------------------------

testHideSourcePaths :: Spec
testHideSourcePaths = do
    describe "when source paths are hidden" $ it "produces no GLoading items" do
        let input =
                [ "src/Foo.hs:10:5: error: Variable not in scope: foo"
                , "    Perhaps you meant: 'bar'"
                , "Failed, one module failed to load."
                ]
        parseReload input
            `shouldBe` [ GMessage
                            GhciMessage
                                { severity = GError
                                , file = "src/Foo.hs"
                                , startPos = Position 10 5
                                , endPos = Position 10 5
                                , messageLines =
                                    [ "src/Foo.hs:10:5: error: Variable not in scope: foo"
                                    , "    Perhaps you meant: 'bar'"
                                    ]
                                }
                       , GSummary LoadFailed
                       ]

    describe "when all modules are already up to date" $ it "returns the summary outcome" do
        -- GHCi with -fhide-source-paths and nothing to recompile
        parseReload ["Ok, 5 modules loaded."] `shouldBe` [GSummary LoadSucceeded]


--------------------------------------------------------------------------------
-- parseReload: <no location info> errors
--------------------------------------------------------------------------------

testNoLocationInfo :: Spec
testNoLocationInfo = do
    it "handles <no location info>: error: with continuation" do
        let input =
                [ "<no location info>: error:"
                , "    Module `Tricorder.Missing' is not loaded."
                ]
        parseReload input
            `shouldBe` [ GMessage
                            GhciMessage
                                { severity = GError
                                , file = "<no location info>"
                                , startPos = Position 0 0
                                , endPos = Position 0 0
                                , messageLines =
                                    [ "<no location info>: error:"
                                    , "    Module `Tricorder.Missing' is not loaded."
                                    ]
                                }
                       ]

    it "handles <no location info>: error: with no continuation" do
        parseReload ["<no location info>: error: some error"]
            `shouldBe` [GMessage GhciMessage {severity = GError, file = "<no location info>", startPos = Position 0 0, endPos = Position 0 0, messageLines = ["<no location info>: error: some error"]}]


--------------------------------------------------------------------------------
-- parseReload: Loaded GHCi configuration
--------------------------------------------------------------------------------

testLoadedConfig :: Spec
testLoadedConfig = do
    it "parses a GHCi configuration line" do
        parseReload ["Loaded GHCi configuration from /home/user/project/.ghci"]
            `shouldBe` [GLoadConfig "/home/user/project/.ghci"]

    it "parses a Windows-style GHCi configuration path" do
        parseReload ["Loaded GHCi configuration from C:\\Users\\user\\project\\.ghci"]
            `shouldBe` [GLoadConfig "C:\\Users\\user\\project\\.ghci"]

    it "handles config line mixed with other output" do
        let input =
                [ "Loaded GHCi configuration from .ghci"
                , "[1 of 1] Compiling Main ( app/Main.hs, interpreted )"
                , "Ok, 1 module loaded."
                ]
        parseReload input
            `shouldBe` [ GLoadConfig ".ghci"
                       , GLoading GhciLoading {index = 1, total = 1, moduleName = "Main", sourceFile = "app/Main.hs"}
                       , GSummary LoadSucceeded
                       ]


--------------------------------------------------------------------------------
-- parseShowModules
--------------------------------------------------------------------------------

testShowModules :: Spec
testShowModules = do
    it "parses typical :show modules output" do
        let input =
                [ "Tricorder.BuildState     ( src/Tricorder/BuildState.hs, interpreted )"
                , "Tricorder.Session        ( src/Tricorder/Session.hs, interpreted )"
                , "Main                     ( app/Main.hs, interpreted )"
                ]
        parseShowModules input
            `shouldBe` [ ("Tricorder.BuildState", "src/Tricorder/BuildState.hs")
                       , ("Tricorder.Session", "src/Tricorder/Session.hs")
                       , ("Main", "app/Main.hs")
                       ]

    it "parses absolute paths" do
        parseShowModules ["Lib ( /home/user/project/src/Lib.hs, interpreted )"]
            `shouldBe` [("Lib", "/home/user/project/src/Lib.hs")]

    it "strips ANSI codes before parsing" do
        parseShowModules ["\ESC[1mMain\ESC[0m                     ( app/Main.hs, interpreted )"]
            `shouldBe` [("Main", "app/Main.hs")]


testShowModulesEmpty :: Spec
testShowModulesEmpty = do
    it "returns empty list for empty input" do
        parseShowModules [] `shouldBe` []

    it "returns empty list for blank lines" do
        parseShowModules ["", "   ", "\t"] `shouldBe` []

    it "skips lines without '( '" do
        parseShowModules ["just some random text"] `shouldBe` []


--------------------------------------------------------------------------------
-- parseShowTargets
--------------------------------------------------------------------------------

testShowTargets :: Spec
testShowTargets = do
    it "parses module names emitted by cabal repl --enable-multi-repl" do
        parseShowTargets
            [ "Atelier.Effects.Cache"
            , "Atelier.Effects.Chan"
            , "Paths_tricorder"
            ]
            `shouldBe` ["Atelier.Effects.Cache", "Atelier.Effects.Chan", "Paths_tricorder"]

    it "parses file-path targets emitted by plain ghci" do
        parseShowTargets ["src/Foo.hs", "test/Bar.hs"]
            `shouldBe` ["src/Foo.hs", "test/Bar.hs"]

    it "strips the leading '*' marker for the active interactive target" do
        parseShowTargets ["*Main", "Foo.Bar"] `shouldBe` ["Main", "Foo.Bar"]

    it "strips ANSI escape sequences" do
        parseShowTargets ["\ESC[1mFoo.Bar\ESC[0m"] `shouldBe` ["Foo.Bar"]

    it "skips blank and whitespace-only lines" do
        parseShowTargets ["", "   ", "\t", "Real.Target"] `shouldBe` ["Real.Target"]

    it "returns empty list for empty input" do
        parseShowTargets [] `shouldBe` []


--------------------------------------------------------------------------------
-- collectResultCustom: <no location info> plugin load failure
--------------------------------------------------------------------------------

-- | Regression test for the \"All good\" bug with home-unit GHC plugins.
--
-- Under @cabal repl --enable-multi-repl@ every unit is interpreted, so a
-- package used as a GHC plugin in the same project is not available as a
-- compiled plugin. The unit that depends on it fails to load, GHCi reports the
-- failure with @\<no location info\>@ (it has no source span), and the load
-- ends with @Failed, N modules loaded@. This output must still surface as an
-- error diagnostic — otherwise the build is silently reported as clean.
testPluginLoadFailure :: Spec
testPluginLoadFailure = do
    -- Shape of the GHCi output from `cabal repl --enable-multi-repl` when an
    -- executable loads a home-unit GHC plugin: the plugin package's modules
    -- compile, then the unit using the plugin fails with a location-less error.
    let reloadOutput =
            [ "[3 of 5] Compiling My.Plugin       ( src/My/Plugin.hs, interpreted )[plugin-pkg-1.0.0-inplace]"
            , "<no location info>: error:"
            , "    Could not load module \8216My.Plugin\8217."
            , "It is a member of the hidden package \8216plugin-pkg-1.0.0\8217."
            , "Perhaps you need to add \8216plugin-pkg\8217 to the build-depends in your .cabal file."
            , "Use -v to see a list of the files searched for."
            , ""
            , "[5 of 5] Compiling Main            ( test/Tests.hs, interpreted )[app-pkg-0.1.0.0-inplace-test]"
            , "Failed, 4 modules loaded."
            ]
        result = collectResultCustom "/project" (parseReload reloadOutput) [] []

    it "surfaces the plugin load failure as an error diagnostic" do
        map (.severity) result.diagnostics `shouldContain` [SError]

    it "carries the plugin error message in the diagnostic title" do
        map (.title) result.diagnostics
            `shouldContain` ["Could not load module \8216My.Plugin\8217."]


--------------------------------------------------------------------------------
-- collectResult: failed load with no located error
--------------------------------------------------------------------------------

-- 'collectResult' is the safety net: GHCi can end a load with @Failed, …@
-- without emitting any error that carries a source span. The build must never
-- read as clean in that case, so a synthetic error diagnostic is added.
testUnattributedFailure :: Spec
testUnattributedFailure = do
    describe "when the load failed but no error was located" $ it "adds a synthetic error" do
        let reloadOutput =
                [ "[1 of 2] Compiling Lib  ( src/Lib.hs, interpreted )"
                , "[2 of 2] Compiling Main ( app/Main.hs, interpreted )"
                , "Failed, 1 module loaded."
                ]
            result = collectResult "/project" reloadOutput [] []
        map (.severity) result.diagnostics `shouldBe` [SError]

    it "does not duplicate a failure that already produced a located error" do
        let reloadOutput =
                [ "[1 of 1] Compiling Lib ( src/Lib.hs, interpreted )"
                , "src/Lib.hs:5:1: error: Oops"
                , "Failed, 0 modules loaded."
                ]
            result = collectResult "/project" reloadOutput [] []
        -- Only the real, located diagnostic — no synthetic one appended.
        map (.file) result.diagnostics `shouldBe` ["src/Lib.hs"]

    it "adds nothing for a successful load" do
        let reloadOutput =
                [ "[1 of 1] Compiling Main ( app/Main.hs, interpreted )"
                , "Ok, 1 module loaded."
                ]
            result = collectResult "/project" reloadOutput [] []
        result.diagnostics `shouldBe` []

    describe "when 'Failed,' appears off the summary line" $ it "does not flag a clean build" do
        -- The load outcome lives on GHCi's single summary line
        -- ("Ok, …" / "Failed, …"). Output printed *during* the load — e.g. a
        -- Template Haskell splice or top-level IO run while interpreting — can
        -- contain a line that happens to begin with "Failed,". That must not be
        -- mistaken for a failed load: the summary here is "Ok," so no synthetic
        -- error belongs.
        let reloadOutput =
                [ "[1 of 1] Compiling Main ( app/Main.hs, interpreted )"
                , "Failed, retrying with fallback" -- printed by a TH splice
                , "Ok, 1 module loaded."
                ]
            result = collectResult "/project" reloadOutput [] []
        result.diagnostics `shouldBe` []