added IFileSystem to BuildAgentBase

arturcic · Bi0T1N · commit 207a69e29695 · 2025-03-07T18:18:56.000+01:00
diff --git a/src/GitVersion.BuildAgents.Tests/Agents/BuildServerBaseTests.cs b/src/GitVersion.BuildAgents.Tests/Agents/BuildServerBaseTests.cs
@@ -48,7 +48,7 @@ public void BuildNumberIsFullSemVer()
         writes.ShouldNotContain(x => x != null && x.StartsWith("Executing GenerateSetVersionMessage for "));
     }
 
-    private class BuildAgent(IEnvironment environment, ILog log) : BuildAgentBase(environment, log)
+    private class BuildAgent(IEnvironment environment, ILog log, IFileSystem fileSystem) : BuildAgentBase(environment, log, fileSystem)
     {
         protected override string EnvironmentVariable => throw new NotImplementedException();
 
diff --git a/src/GitVersion.BuildAgents/Agents/AppVeyor.cs b/src/GitVersion.BuildAgents/Agents/AppVeyor.cs
@@ -4,7 +4,7 @@
 
 namespace GitVersion.Agents;
 
-internal class AppVeyor(IEnvironment environment, ILog log) : BuildAgentBase(environment, log)
+internal class AppVeyor(IEnvironment environment, ILog log, IFileSystem fileSystem) : BuildAgentBase(environment, log, fileSystem)
 {
     public const string EnvironmentVariableName = "APPVEYOR";
 
diff --git a/src/GitVersion.BuildAgents/Agents/AzurePipelines.cs b/src/GitVersion.BuildAgents/Agents/AzurePipelines.cs
@@ -4,7 +4,7 @@
 
 namespace GitVersion.Agents;
 
-internal class AzurePipelines(IEnvironment environment, ILog log) : BuildAgentBase(environment, log)
+internal class AzurePipelines(IEnvironment environment, ILog log, IFileSystem fileSystem) : BuildAgentBase(environment, log, fileSystem)
 {
     public const string EnvironmentVariableName = "TF_BUILD";
 
diff --git a/src/GitVersion.BuildAgents/Agents/BitBucketPipelines.cs b/src/GitVersion.BuildAgents/Agents/BitBucketPipelines.cs
@@ -12,7 +12,7 @@ internal class BitBucketPipelines : BuildAgentBase
     private string? propertyFile;
     private string? ps1File;
 
-    public BitBucketPipelines(IEnvironment environment, ILog log) : base(environment, log)
+    public BitBucketPipelines(IEnvironment environment, ILog log, IFileSystem fileSystem) : base(environment, log, fileSystem)
     {
         WithPropertyFile("gitversion.properties");
         WithPowershellFile("gitversion.ps1");
diff --git a/src/GitVersion.BuildAgents/Agents/BuildKite.cs b/src/GitVersion.BuildAgents/Agents/BuildKite.cs
@@ -3,7 +3,7 @@
 
 namespace GitVersion.Agents;
 
-internal class BuildKite(IEnvironment environment, ILog log) : BuildAgentBase(environment, log)
+internal class BuildKite(IEnvironment environment, ILog log, IFileSystem fileSystem) : BuildAgentBase(environment, log, fileSystem)
 {
     public const string EnvironmentVariableName = "BUILDKITE";
 
diff --git a/src/GitVersion.BuildAgents/Agents/CodeBuild.cs b/src/GitVersion.BuildAgents/Agents/CodeBuild.cs
@@ -10,7 +10,7 @@ internal sealed class CodeBuild : BuildAgentBase
     public const string WebHookEnvironmentVariableName = "CODEBUILD_WEBHOOK_HEAD_REF";
     public const string SourceVersionEnvironmentVariableName = "CODEBUILD_SOURCE_VERSION";
 
-    public CodeBuild(IEnvironment environment, ILog log) : base(environment, log) => WithPropertyFile("gitversion.properties");
+    public CodeBuild(IEnvironment environment, ILog log, IFileSystem fileSystem) : base(environment, log, fileSystem) => WithPropertyFile("gitversion.properties");
 
     public void WithPropertyFile(string propertiesFileName) => this.file = propertiesFileName;
 
diff --git a/src/GitVersion.BuildAgents/Agents/ContinuaCi.cs b/src/GitVersion.BuildAgents/Agents/ContinuaCi.cs
@@ -3,7 +3,7 @@
 
 namespace GitVersion.Agents;
 
-internal class ContinuaCi(IEnvironment environment, ILog log) : BuildAgentBase(environment, log)
+internal class ContinuaCi(IEnvironment environment, ILog log, IFileSystem fileSystem) : BuildAgentBase(environment, log, fileSystem)
 {
     public const string EnvironmentVariableName = "ContinuaCI.Version";
 
diff --git a/src/GitVersion.BuildAgents/Agents/Drone.cs b/src/GitVersion.BuildAgents/Agents/Drone.cs
@@ -4,7 +4,7 @@
 
 namespace GitVersion.Agents;
 
-internal class Drone(IEnvironment environment, ILog log) : BuildAgentBase(environment, log)
+internal class Drone(IEnvironment environment, ILog log, IFileSystem fileSystem) : BuildAgentBase(environment, log, fileSystem)
 {
     public const string EnvironmentVariableName = "DRONE";
     protected override string EnvironmentVariable => EnvironmentVariableName;
diff --git a/src/GitVersion.BuildAgents/Agents/EnvRun.cs b/src/GitVersion.BuildAgents/Agents/EnvRun.cs
@@ -4,7 +4,7 @@
 
 namespace GitVersion.Agents;
 
-internal class EnvRun(IEnvironment environment, ILog log) : BuildAgentBase(environment, log)
+internal class EnvRun(IEnvironment environment, ILog log, IFileSystem fileSystem) : BuildAgentBase(environment, log, fileSystem)
 {
     public const string EnvironmentVariableName = "ENVRUN_DATABASE";
     protected override string EnvironmentVariable => EnvironmentVariableName;
diff --git a/src/GitVersion.BuildAgents/Agents/GitHubActions.cs b/src/GitVersion.BuildAgents/Agents/GitHubActions.cs
@@ -4,7 +4,7 @@
 
 namespace GitVersion.Agents;
 
-internal class GitHubActions(IEnvironment environment, ILog log) : BuildAgentBase(environment, log)
+internal class GitHubActions(IEnvironment environment, ILog log, IFileSystem fileSystem) : BuildAgentBase(environment, log, fileSystem)
 {
     // https://help.github.com/en/actions/automating-your-workflow-with-github-actions/using-environment-variables#default-environment-variables
 
diff --git a/src/GitVersion.BuildAgents/Agents/GitLabCi.cs b/src/GitVersion.BuildAgents/Agents/GitLabCi.cs
@@ -8,7 +8,7 @@ internal class GitLabCi : BuildAgentBase
     public const string EnvironmentVariableName = "GITLAB_CI";
     private string? file;
 
-    public GitLabCi(IEnvironment environment, ILog log) : base(environment, log) => WithPropertyFile("gitversion.properties");
+    public GitLabCi(IEnvironment environment, ILog log, IFileSystem fileSystem) : base(environment, log, fileSystem) => WithPropertyFile("gitversion.properties");
 
     public void WithPropertyFile(string propertiesFileName) => this.file = propertiesFileName;
 
diff --git a/src/GitVersion.BuildAgents/Agents/Jenkins.cs b/src/GitVersion.BuildAgents/Agents/Jenkins.cs
@@ -10,7 +10,7 @@ internal class Jenkins : BuildAgentBase
     private string? file;
     protected override string EnvironmentVariable => EnvironmentVariableName;
 
-    public Jenkins(IEnvironment environment, ILog log) : base(environment, log) => WithPropertyFile("gitversion.properties");
+    public Jenkins(IEnvironment environment, ILog log, IFileSystem fileSystem) : base(environment, log, fileSystem) => WithPropertyFile("gitversion.properties");
 
     public void WithPropertyFile(string propertiesFileName) => this.file = propertiesFileName;
 
diff --git a/src/GitVersion.BuildAgents/Agents/MyGet.cs b/src/GitVersion.BuildAgents/Agents/MyGet.cs
@@ -5,7 +5,7 @@
 
 namespace GitVersion.Agents;
 
-internal class MyGet(IEnvironment environment, ILog log) : BuildAgentBase(environment, log)
+internal class MyGet(IEnvironment environment, ILog log, IFileSystem fileSystem) : BuildAgentBase(environment, log, fileSystem)
 {
     public const string EnvironmentVariableName = "BuildRunner";
     protected override string EnvironmentVariable => EnvironmentVariableName;
diff --git a/src/GitVersion.BuildAgents/Agents/SpaceAutomation.cs b/src/GitVersion.BuildAgents/Agents/SpaceAutomation.cs
@@ -3,7 +3,7 @@
 
 namespace GitVersion.Agents;
 
-internal class SpaceAutomation(IEnvironment environment, ILog log) : BuildAgentBase(environment, log)
+internal class SpaceAutomation(IEnvironment environment, ILog log, IFileSystem fileSystem) : BuildAgentBase(environment, log, fileSystem)
 {
     public const string EnvironmentVariableName = "JB_SPACE_PROJECT_KEY";
 
diff --git a/src/GitVersion.BuildAgents/Agents/TeamCity.cs b/src/GitVersion.BuildAgents/Agents/TeamCity.cs
@@ -5,7 +5,7 @@
 
 namespace GitVersion.Agents;
 
-internal class TeamCity(IEnvironment environment, ILog log) : BuildAgentBase(environment, log)
+internal class TeamCity(IEnvironment environment, ILog log, IFileSystem fileSystem) : BuildAgentBase(environment, log, fileSystem)
 {
     public const string EnvironmentVariableName = "TEAMCITY_VERSION";
 
diff --git a/src/GitVersion.BuildAgents/Agents/TravisCI.cs b/src/GitVersion.BuildAgents/Agents/TravisCI.cs
@@ -3,7 +3,7 @@
 
 namespace GitVersion.Agents;
 
-internal class TravisCi(IEnvironment environment, ILog log) : BuildAgentBase(environment, log)
+internal class TravisCi(IEnvironment environment, ILog log, IFileSystem fileSystem) : BuildAgentBase(environment, log, fileSystem)
 {
     public const string EnvironmentVariableName = "TRAVIS";
     protected override string EnvironmentVariable => EnvironmentVariableName;
diff --git a/src/GitVersion.Core/Agents/BuildAgentBase.cs b/src/GitVersion.Core/Agents/BuildAgentBase.cs
@@ -4,10 +4,11 @@
 
 namespace GitVersion.Agents;
 
-internal abstract class BuildAgentBase(IEnvironment environment, ILog log) : ICurrentBuildAgent
+internal abstract class BuildAgentBase(IEnvironment environment, ILog log, IFileSystem fileSystem) : ICurrentBuildAgent
 {
     protected readonly ILog Log = log.NotNull();
-    protected IEnvironment Environment { get; } = environment.NotNull();
+    protected readonly IEnvironment Environment = environment.NotNull();
+    protected readonly IFileSystem FileSystem = fileSystem.NotNull();
 
     protected abstract string EnvironmentVariable { get; }
     public virtual bool IsDefault => false;
diff --git a/src/GitVersion.Core/Agents/LocalBuild.cs b/src/GitVersion.Core/Agents/LocalBuild.cs
@@ -3,7 +3,7 @@
 
 namespace GitVersion.Agents;
 
-internal class LocalBuild(IEnvironment environment, ILog log) : BuildAgentBase(environment, log)
+internal class LocalBuild(IEnvironment environment, ILog log, IFileSystem fileSystem) : BuildAgentBase(environment, log, fileSystem)
 {
     public override bool IsDefault => true;
 

Original file line number	Diff line number	Diff line change
`@@ -48,7 +48,7 @@ public void BuildNumberIsFullSemVer()`
`48`	`48`	`writes.ShouldNotContain(x => x != null && x.StartsWith("Executing GenerateSetVersionMessage for "));`
`49`	`49`	`}`
`50`	`50`
`51`		`- private class BuildAgent(IEnvironment environment, ILog log) : BuildAgentBase(environment, log)`
	`51`	`+ private class BuildAgent(IEnvironment environment, ILog log, IFileSystem fileSystem) : BuildAgentBase(environment, log, fileSystem)`
`52`	`52`	`{`
`53`	`53`	`protected override string EnvironmentVariable => throw new NotImplementedException();`
`54`	`54`
Original file line number	Diff line number	Diff line change
`@@ -4,7 +4,7 @@`
`4`	`4`
`5`	`5`	`namespace GitVersion.Agents;`
`6`	`6`
`7`		`-internal class AppVeyor(IEnvironment environment, ILog log) : BuildAgentBase(environment, log)`
	`7`	`+internal class AppVeyor(IEnvironment environment, ILog log, IFileSystem fileSystem) : BuildAgentBase(environment, log, fileSystem)`
`8`	`8`	`{`
`9`	`9`	`public const string EnvironmentVariableName = "APPVEYOR";`
`10`	`10`
Original file line number	Diff line number	Diff line change
`@@ -4,7 +4,7 @@`
`4`	`4`
`5`	`5`	`namespace GitVersion.Agents;`
`6`	`6`
`7`		`-internal class AzurePipelines(IEnvironment environment, ILog log) : BuildAgentBase(environment, log)`
	`7`	`+internal class AzurePipelines(IEnvironment environment, ILog log, IFileSystem fileSystem) : BuildAgentBase(environment, log, fileSystem)`
`8`	`8`	`{`
`9`	`9`	`public const string EnvironmentVariableName = "TF_BUILD";`
`10`	`10`
Original file line number	Diff line number	Diff line change
`@@ -12,7 +12,7 @@ internal class BitBucketPipelines : BuildAgentBase`
`12`	`12`	`private string? propertyFile;`
`13`	`13`	`private string? ps1File;`
`14`	`14`
`15`		`- public BitBucketPipelines(IEnvironment environment, ILog log) : base(environment, log)`
	`15`	`+ public BitBucketPipelines(IEnvironment environment, ILog log, IFileSystem fileSystem) : base(environment, log, fileSystem)`
`16`	`16`	`{`
`17`	`17`	`WithPropertyFile("gitversion.properties");`
`18`	`18`	`WithPowershellFile("gitversion.ps1");`
Original file line number	Diff line number	Diff line change
`@@ -3,7 +3,7 @@`
`3`	`3`
`4`	`4`	`namespace GitVersion.Agents;`
`5`	`5`
`6`		`-internal class BuildKite(IEnvironment environment, ILog log) : BuildAgentBase(environment, log)`
	`6`	`+internal class BuildKite(IEnvironment environment, ILog log, IFileSystem fileSystem) : BuildAgentBase(environment, log, fileSystem)`
`7`	`7`	`{`
`8`	`8`	`public const string EnvironmentVariableName = "BUILDKITE";`
`9`	`9`
Original file line number	Diff line number	Diff line change
`@@ -3,7 +3,7 @@`
`3`	`3`
`4`	`4`	`namespace GitVersion.Agents;`
`5`	`5`
`6`		`-internal class ContinuaCi(IEnvironment environment, ILog log) : BuildAgentBase(environment, log)`
	`6`	`+internal class ContinuaCi(IEnvironment environment, ILog log, IFileSystem fileSystem) : BuildAgentBase(environment, log, fileSystem)`
`7`	`7`	`{`
`8`	`8`	`public const string EnvironmentVariableName = "ContinuaCI.Version";`
`9`	`9`
Original file line number	Diff line number	Diff line change
`@@ -4,7 +4,7 @@`
`4`	`4`
`5`	`5`	`namespace GitVersion.Agents;`
`6`	`6`
`7`		`-internal class Drone(IEnvironment environment, ILog log) : BuildAgentBase(environment, log)`
	`7`	`+internal class Drone(IEnvironment environment, ILog log, IFileSystem fileSystem) : BuildAgentBase(environment, log, fileSystem)`
`8`	`8`	`{`
`9`	`9`	`public const string EnvironmentVariableName = "DRONE";`
`10`	`10`	`protected override string EnvironmentVariable => EnvironmentVariableName;`
Original file line number	Diff line number	Diff line change
`@@ -4,7 +4,7 @@`
`4`	`4`
`5`	`5`	`namespace GitVersion.Agents;`
`6`	`6`
`7`		`-internal class EnvRun(IEnvironment environment, ILog log) : BuildAgentBase(environment, log)`
	`7`	`+internal class EnvRun(IEnvironment environment, ILog log, IFileSystem fileSystem) : BuildAgentBase(environment, log, fileSystem)`
`8`	`8`	`{`
`9`	`9`	`public const string EnvironmentVariableName = "ENVRUN_DATABASE";`
`10`	`10`	`protected override string EnvironmentVariable => EnvironmentVariableName;`
Original file line number	Diff line number	Diff line change
`@@ -4,7 +4,7 @@`
`4`	`4`
`5`	`5`	`namespace GitVersion.Agents;`
`6`	`6`
`7`		`-internal class GitHubActions(IEnvironment environment, ILog log) : BuildAgentBase(environment, log)`
	`7`	`+internal class GitHubActions(IEnvironment environment, ILog log, IFileSystem fileSystem) : BuildAgentBase(environment, log, fileSystem)`
`8`	`8`	`{`
`9`	`9`	`// https://help.github.com/en/actions/automating-your-workflow-with-github-actions/using-environment-variables#default-environment-variables`
`10`	`10`
Original file line number	Diff line number	Diff line change
`@@ -5,7 +5,7 @@`
`5`	`5`
`6`	`6`	`namespace GitVersion.Agents;`
`7`	`7`
`8`		`-internal class MyGet(IEnvironment environment, ILog log) : BuildAgentBase(environment, log)`
	`8`	`+internal class MyGet(IEnvironment environment, ILog log, IFileSystem fileSystem) : BuildAgentBase(environment, log, fileSystem)`
`9`	`9`	`{`
`10`	`10`	`public const string EnvironmentVariableName = "BuildRunner";`
`11`	`11`	`protected override string EnvironmentVariable => EnvironmentVariableName;`