Alright, we can leave it for now

On Thursday, March 7, 2013, dschenkelman wrote:

> In test/ScriptCs.Core.Tests/ScriptExecutorTests.cs:
>
> > @@ -187,6 +187,118 @@ public void ShouldExecuteScriptReturnedFromFileProcessorInSessionWhenExecuteIsIn
> >                  preProcessor.Verify(fs => fs.ProcessFile(Path.Combine(currentDirectory, scriptName)), Times.Once());
> >                  session.Verify(s => s.Execute(code), Times.Once());
> >              }
> > +
> > +            [Fact]
> > +            public void ShouldCopyFilesInPathIfLastWriteTimeDiffersFromLastWriteTimeOfFileInBin()
> > +            {
> > +                // arrange
> > +                var fileSystem = new Mock<IFileSystem>();
>
> Actually they are exactly the same except for the dates and the asserts.
> In this case I would prefer to have them separate as it only involves a bit
> of setup, but provides better readability IMHO. I can definitely refactor
> this if you believe it would be better.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/scriptcs/scriptcs/pull/66/files#r3279655>
> .
>