Skip to content

Commit ed82b89

Browse files
committed
Add tests for CLI args token resolution
- Add test Should_Pass_Source_Pat_From_Cli_Args_To_Factory to verify source PAT from CLI args is passed to factory - Add test Should_Pass_Target_Pat_From_Cli_Args_To_Factory to verify target PAT from CLI args is passed to factory - Tests prove that CLI args are properly passed through to the factory, demonstrating the bug fix works
1 parent 8964de8 commit ed82b89

File tree

2 files changed

+80
-0
lines changed

2 files changed

+80
-0
lines changed

src/OctoshiftCLI.Tests/gei/Commands/MigrateCodeScanningAlerts/MigrateCodeScanningAlertsCommandTests.cs

Lines changed: 40 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -66,4 +66,44 @@ public void Source_Pat_Should_Default_To_Target_Pat()
6666

6767
_mockCodeScanningAlertServiceFactory.Verify(m => m.Create(It.IsAny<string>(), targetToken, It.IsAny<string>(), targetToken, It.IsAny<bool>()));
6868
}
69+
70+
[Fact]
71+
public void Should_Pass_Source_Pat_From_Cli_Args_To_Factory()
72+
{
73+
var sourceToken = "source-token";
74+
var targetToken = "target-token";
75+
76+
var args = new MigrateCodeScanningAlertsCommandArgs()
77+
{
78+
SourceOrg = "source-org",
79+
SourceRepo = "source-repo",
80+
TargetOrg = "target-org",
81+
TargetRepo = "target-repo",
82+
GithubSourcePat = sourceToken,
83+
GithubTargetPat = targetToken,
84+
};
85+
86+
_command.BuildHandler(args, _serviceProvider);
87+
88+
_mockCodeScanningAlertServiceFactory.Verify(m => m.Create(It.IsAny<string>(), sourceToken, It.IsAny<string>(), targetToken, It.IsAny<bool>()));
89+
}
90+
91+
[Fact]
92+
public void Should_Pass_Target_Pat_From_Cli_Args_To_Factory()
93+
{
94+
var targetToken = "target-token";
95+
96+
var args = new MigrateCodeScanningAlertsCommandArgs()
97+
{
98+
SourceOrg = "source-org",
99+
SourceRepo = "source-repo",
100+
TargetOrg = "target-org",
101+
TargetRepo = "target-repo",
102+
GithubTargetPat = targetToken,
103+
};
104+
105+
_command.BuildHandler(args, _serviceProvider);
106+
107+
_mockCodeScanningAlertServiceFactory.Verify(m => m.Create(It.IsAny<string>(), targetToken, It.IsAny<string>(), targetToken, It.IsAny<bool>()));
108+
}
69109
}

src/OctoshiftCLI.Tests/gei/Commands/MigrateSecretAlerts/MigrateSecretAlertsCommandTests.cs

Lines changed: 40 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -66,4 +66,44 @@ public void Source_Pat_Should_Default_To_Target_Pat()
6666

6767
_mockSecretScanningAlertServiceFactory.Verify(m => m.Create(It.IsAny<string>(), targetToken, It.IsAny<string>(), targetToken, It.IsAny<bool>()));
6868
}
69+
70+
[Fact]
71+
public void Should_Pass_Source_Pat_From_Cli_Args_To_Factory()
72+
{
73+
var sourceToken = "source-token";
74+
var targetToken = "target-token";
75+
76+
var args = new MigrateSecretAlertsCommandArgs()
77+
{
78+
SourceOrg = "source-org",
79+
SourceRepo = "source-repo",
80+
TargetOrg = "target-org",
81+
TargetRepo = "target-repo",
82+
GithubSourcePat = sourceToken,
83+
GithubTargetPat = targetToken,
84+
};
85+
86+
_command.BuildHandler(args, _serviceProvider);
87+
88+
_mockSecretScanningAlertServiceFactory.Verify(m => m.Create(It.IsAny<string>(), sourceToken, It.IsAny<string>(), targetToken, It.IsAny<bool>()));
89+
}
90+
91+
[Fact]
92+
public void Should_Pass_Target_Pat_From_Cli_Args_To_Factory()
93+
{
94+
var targetToken = "target-token";
95+
96+
var args = new MigrateSecretAlertsCommandArgs()
97+
{
98+
SourceOrg = "source-org",
99+
SourceRepo = "source-repo",
100+
TargetOrg = "target-org",
101+
TargetRepo = "target-repo",
102+
GithubTargetPat = targetToken,
103+
};
104+
105+
_command.BuildHandler(args, _serviceProvider);
106+
107+
_mockSecretScanningAlertServiceFactory.Verify(m => m.Create(It.IsAny<string>(), targetToken, It.IsAny<string>(), targetToken, It.IsAny<bool>()));
108+
}
69109
}

0 commit comments

Comments
 (0)