Add more tests and improve fix

Author: alfonsogarciacaro

src/fsharp/CompileOps.fs

@@ -4962,38 +4962,32 @@ module private ScriptPreprocessClosure =
             match closureDirective with 
             | ClosedSourceFile _ as csf -> [csf]
             | SourceFile(filename,m,source) ->
-                let filename = FileSystem.GetFullPathShim(filename)
-                if observedSources.HaveSeen(filename) then [] 
-                else     
-                    observedSources.SetSeen(filename)
-                    
-                    let errors = ref []
-                    let warnings = ref [] 
-                    let errorLogger = 
-                         { new ErrorLogger("FindClosure") with 
-                               member x.ErrorSinkImpl(e) = errors := e :: !errors
-                               member x.WarnSinkImpl(e) = warnings := e :: !warnings
-                               member x.ErrorCount = (!errors).Length }                        
-
-                    use unwindEL = PushErrorLoggerPhaseUntilUnwind (fun _ -> errorLogger)
-                    let pathOfMetaCommandSource = Path.GetDirectoryName(filename)
-                    match ParseScriptText(filename,source,!tcConfig,codeContext,lexResourceManager,errorLogger) with 
-                    | Some(input) ->                    
-                        let tcConfigResult, noWarns = ApplyMetaCommandsFromInputToTcConfigAndGatherNoWarn !tcConfig (input,pathOfMetaCommandSource)
-                        tcConfig := tcConfigResult
+                let errors = ref []
+                let warnings = ref [] 
+                let errorLogger = 
+                        { new ErrorLogger("FindClosure") with 
+                            member x.ErrorSinkImpl(e) = errors := e :: !errors
+                            member x.WarnSinkImpl(e) = warnings := e :: !warnings
+                            member x.ErrorCount = (!errors).Length }                        
+
+                use unwindEL = PushErrorLoggerPhaseUntilUnwind (fun _ -> errorLogger)
+                let pathOfMetaCommandSource = Path.GetDirectoryName(filename)
+                match ParseScriptText(filename,source,!tcConfig,codeContext,lexResourceManager,errorLogger) with 
+                | Some(input) ->                    
+                    let tcConfigResult, noWarns = ApplyMetaCommandsFromInputToTcConfigAndGatherNoWarn !tcConfig (input,pathOfMetaCommandSource)
+                    tcConfig := tcConfigResult
 
-                        let AddFileIfNotSeen(m,filename) = 
-                            if observedSources.HaveSeen(filename) then []
-                            else
-                                if IsScript(filename) then SourceFileOfFilename(filename,m,tcConfigResult.inputCodePage)
-                                else 
-                                    observedSources.SetSeen(filename)
-                                    [ClosedSourceFile(filename,m,None,[],[],[])] // Don't traverse into .fs leafs.
+                    let AddFileIfNotSeen(m,filename) = 
+                        if observedSources.HaveSeen(filename) then []
+                        else
+                            observedSources.SetSeen(filename)
+                            if IsScript(filename) then SourceFileOfFilename(filename,m,tcConfigResult.inputCodePage)
+                            else [ClosedSourceFile(filename,m,None,[],[],[])] // Don't traverse into .fs leafs.
 
-                        let loadedSources = (!tcConfig).GetAvailableLoadedSources() |> List.rev |> List.map AddFileIfNotSeen |> List.concat
-                        (loadedSources |> List.map FindClosure |> List.concat)
-                            @  [ClosedSourceFile(filename,m,Some(input),!errors,!warnings,!noWarns)]
-                    | None -> [ClosedSourceFile(filename,m,None,!errors,!warnings,[])]
+                    let loadedSources = (!tcConfig).GetAvailableLoadedSources() |> List.map AddFileIfNotSeen |> List.concat
+                    (loadedSources |> List.map FindClosure |> List.concat)
+                        @  [ClosedSourceFile(filename,m,Some(input),!errors,!warnings,!noWarns)]
+                | None -> [ClosedSourceFile(filename,m,None,!errors,!warnings,[])]
 
         closureDirectives |> List.map FindClosure |> List.concat, !tcConfig
 
@@ -5011,7 +5005,7 @@ module private ScriptPreprocessClosure =
         let sourceFiles = ref []
         let sourceInputs = ref []
         let globalNoWarns = ref []
-        for directive in closureDirectives do
+        for directive in List.rev closureDirectives do
             match directive with 
             | ClosedSourceFile(filename,m,input,_,_,noWarns) -> 
                 let filename = FileSystem.GetFullPathShim(filename)
@@ -5054,8 +5048,8 @@ module private ScriptPreprocessClosure =
         let rootWarnings = rootWarnings |> List.filter isRootRange
 
         let result : LoadClosure = 
-            { SourceFiles = List.groupByFirst !sourceFiles |> List.rev
-              References = List.groupByFirst references |> List.rev
+            { SourceFiles = List.groupByFirst !sourceFiles
+              References = List.groupByFirst references
               UnresolvedReferences = unresolvedReferences
               Inputs = !sourceInputs
               NoWarns = List.groupByFirst !globalNoWarns

tests/service/ProjectOptionsTests.fs

@@ -420,15 +420,20 @@ let ``Project file parsing -- report files``() =
 
 [<Test>]
 let ``Test ProjectFileNames order for GetProjectOptionsFromScript`` () = // See #594
-    let scriptPath = __SOURCE_DIRECTORY__ + @"/data/ScriptProject/Main.fsx"
-    let scriptSource = File.ReadAllText scriptPath
-    let projOpts =
-        checker.GetProjectOptionsFromScript(scriptPath, scriptSource)
-        |> Async.RunSynchronously
-    projOpts.ProjectFileNames
-    |> Array.map Path.GetFileNameWithoutExtension
-    |> (=) [|"BaseLib"; "Lib1"; "Lib2"; "Main"|]
-    |> shouldEqual true
+    let test scriptName expected =
+        let scriptPath = __SOURCE_DIRECTORY__ + @"/data/ScriptProject/" + scriptName + ".fsx"
+        let scriptSource = File.ReadAllText scriptPath
+        let projOpts =
+            checker.GetProjectOptionsFromScript(scriptPath, scriptSource)
+            |> Async.RunSynchronously
+        projOpts.ProjectFileNames
+        |> Array.map Path.GetFileNameWithoutExtension
+        |> (=) expected
+        |> shouldEqual true
+    test "Main1" [|"BaseLib1"; "Lib1"; "Lib2"; "Main1"|]
+    test "Main2" [|"BaseLib1"; "Lib1"; "Lib2"; "Lib3"; "Main2"|]
+    test "Main3" [|"Lib3"; "Lib4"; "Main3"|]
+    test "Main4" [|"BaseLib2"; "Lib5"; "BaseLib1"; "Lib1"; "Lib2"; "Main4"|]
 
 #endif
 

tests/service/data/ScriptProject/BaseLib.fs renamed to tests/service/data/ScriptProject/BaseLib1.fs

@@ -1,3 +1,3 @@
-module BaseLib
+module BaseLib1
 
 let add2 x = x + 2

tests/service/data/ScriptProject/BaseLib2.fs

@@ -0,0 +1,3 @@
+module BaseLib2
+
+let add10 x = x + 10

tests/service/data/ScriptProject/Lib1.fsx

@@ -1,2 +1,2 @@
-#load "BaseLib.fs"
-let add3 = BaseLib.add2 >> ((+) 1)
+#load "BaseLib1.fs"
+let add3 = BaseLib1.add2 >> ((+) 1)

tests/service/data/ScriptProject/Lib2.fsx

@@ -1,2 +1,2 @@
-#load "BaseLib.fs"
-let add4 = BaseLib.add2 >> ((+) 2)
+#load "BaseLib1.fs"
+let add4 = BaseLib1.add2 >> ((+) 2)

tests/service/data/ScriptProject/Lib3.fs

@@ -0,0 +1,3 @@
+module Lib3
+
+let add6 = ((+) 6)

tests/service/data/ScriptProject/Lib4.fs

@@ -0,0 +1,3 @@
+module Lib4
+
+let add8 = ((+) 8)

tests/service/data/ScriptProject/Lib5.fsx

@@ -0,0 +1,2 @@
+#load "BaseLib2.fs"
+let add12 = BaseLib2.add10 >> ((+) 2)