Skip to content

Commit

Permalink
Quotes proto files argument
Browse files Browse the repository at this point in the history
  • Loading branch information
MihaMarkic committed Jul 25, 2018
1 parent ebd521e commit 63d74b0
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 4 deletions.
2 changes: 1 addition & 1 deletion src/Cake.ProtobufTools.Tests/Protoc/ProtocAliasesTest.cs
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ public void WhenCSharpAndOFileAreSet_CommandLineIsCorrect()
};

var actual = fixture.Run();
Assert.That(actual.Args, Is.EqualTo("-oFILE --csharp_out=\"some_folder\" definitions.proto"));
Assert.That(actual.Args, Is.EqualTo("-oFILE --csharp_out=\"some_folder\" \"definitions.proto\""));
}
}
}
2 changes: 1 addition & 1 deletion src/Cake.ProtobufTools/ArgumentsBuilderExtension.cs
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@ public static void AppendAll<TSettings>(this ProcessArgumentBuilder builder, TSe
{
foreach (string argument in arguments)
{
builder.Append(argument);
builder.AppendQuoted(argument);
}
}
}
Expand Down
4 changes: 2 additions & 2 deletions src/Cake.ProtobufTools/ProtobufTools`1.cs
Original file line number Diff line number Diff line change
Expand Up @@ -43,10 +43,10 @@ public void Run(ProtocSettings settings, string[] additional)
Run(settings, GetArguments(settings, additional));
}

private ProcessArgumentBuilder GetArguments(ProtocSettings settings, string[] containers)
private ProcessArgumentBuilder GetArguments(ProtocSettings settings, string[] files)
{
var builder = new ProcessArgumentBuilder();
builder.AppendAll(settings, containers);
builder.AppendAll(settings, files);
return builder;
}

Expand Down

0 comments on commit 63d74b0

Please sign in to comment.