dan/Cosmos
1
0
Fork 0
mirror of https://github.com/danbulant/Cosmos synced 2026-05-27 05:52:11 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

Merge pull request #755 from KingLuigi4932/patch-3

Browse source 
Fixing WriteAllLines issue
This commit is contained in:
fanoI 2017-09-02 20:54:15 +02:00 committed by GitHub
commit cf08368c9e
2 changed files with 17 additions and 24 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

26
Tests/Cosmos.Kernel.Tests.Fat/Kernel.cs
View file

@ -617,20 +617,18 @@ namespace Cosmos.Kernel.Tests.Fat
mDebugger.Send("END TEST");
mDebugger.Send("");
}
// Generic methods on arrays don't work, see https://github.com/CosmosOS/Cosmos/issues/583
//
//string[] contents = { "One", "Two", "Three" };
//File.WriteAllLines(@"0:\test3.txt", contents);
//mDebugger.Send("Text written");
//mDebugger.Send("Now reading with ReadAllLines()");
//string[] readLines = File.ReadAllLines(@"0:\test3.txt");
//mDebugger.Send("Contents retrieved after writing");
//for (int i = 0; i < readLines.Length; i++)
//{
// mDebugger.Send(readLines[i]);
//}
//Assert.IsTrue(StringArrayAreEquals(contents, readLines), "Contents of test3.txt was written incorrectly!");
string[] contents = { "One", "Two", "Three" };
File.WriteAllLines(@"0:\test3.txt", contents);
mDebugger.Send("Text written");
mDebugger.Send("Now reading with ReadAllLines()");
string[] readLines = File.ReadAllLines(@"0:\test3.txt");
mDebugger.Send("Contents retrieved after writing");
for (int i = 0; i < readLines.Length; i++)
{
mDebugger.Send(readLines[i]);
}
Assert.IsTrue(StringArrayAreEquals(contents, readLines), "Contents of test3.txt was written incorrectly!");
#if false
// TODO maybe the more correct test is to implement ReadAllLines and then check that two arrays are equals
var xContents = File.ReadAllText(@"0:\test3.txt");

15
source/Cosmos.System2_Plugs/System/IO/FileImpl.cs
View file

@ -119,19 +119,14 @@ namespace Cosmos.System_Plugs.System.IO
return result;
}
public static void WriteAllLines(string aFile, IEnumerable<string> contents)
public static void WriteAllLines(string aFile, string[] contents)
{
string text = null;
foreach (var line in contents)
{
text = string.Concat(text, line, Environment.NewLine);
}
string text = String.Join(Environment.NewLine, contents);
Global.mFileSystemDebugger.SendInternal("Writing contents");
Global.mFileSystemDebugger.SendInternal(text);
WriteAllText(aFile, text);
}