Skip to content

Commit

Permalink
Restore constructor
Browse files Browse the repository at this point in the history
  • Loading branch information
jameswinkler committed Jan 26, 2021
1 parent 79ca0bc commit d537617
Showing 1 changed file with 7 additions and 0 deletions.
7 changes: 7 additions & 0 deletions src/Sarif.Driver/Sdk/CommandBase.cs
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,13 @@ public abstract class CommandBase

protected IFileSystem FileSystem { get; set; }

// TODO: Removing this constructor broke the one of AbsoluteUriCommand entirely but all unit tests were passing
// Add unit tests that will break when this constructor is deleted or malfunctioning.
public CommandBase(IFileSystem fileSystem = null)
{
this.FileSystem = fileSystem ?? Sarif.FileSystem.Instance;
}

// TODO: What's the point of having a bunch of static methods in an abstract class?
// We even have a static class, "CommandUtilities" which seems like the more appropriate
// place for these to go.
Expand Down

0 comments on commit d537617

Please sign in to comment.