Skip to content

Commit 1f42b9c

Browse files
Avoid variable path combine in guardrail tests
1 parent 07e0909 commit 1f42b9c

1 file changed

Lines changed: 7 additions & 1 deletion

File tree

OfficeIMO.Tests/PackageDependencyGuardrails.cs

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -146,7 +146,13 @@ private static string GetRepositoryPath(string relativePath) {
146146

147147
var parts = NormalizeProjectPath(relativePath)
148148
.Split(new[] { '/' }, StringSplitOptions.RemoveEmptyEntries);
149-
var combinedPath = Path.GetFullPath(Path.Combine(new[] { repositoryRoot }.Concat(parts).ToArray()));
149+
var combinedPath = repositoryRoot;
150+
foreach (var part in parts) {
151+
Assert.False(Path.IsPathRooted(part), "Repository-relative path segment must not be rooted: " + relativePath);
152+
combinedPath = Path.Combine(combinedPath, part);
153+
}
154+
155+
combinedPath = Path.GetFullPath(combinedPath);
150156

151157
Assert.True(
152158
combinedPath.StartsWith(repositoryRoot, StringComparison.Ordinal),

0 commit comments

Comments
 (0)