Skip to content

Commit

Permalink
Merge pull request #124 from microsoft/vpl/add-mv-admins
Browse files Browse the repository at this point in the history
Ignore unknown commands
  • Loading branch information
vplauzon authored Mar 31, 2023
2 parents 6124ce1 + e8dad9d commit 4000ae2
Show file tree
Hide file tree
Showing 2 changed files with 20 additions and 1 deletion.
2 changes: 1 addition & 1 deletion code/DeltaKustoLib/CommandModel/CommandBase.cs
Original file line number Diff line number Diff line change
Expand Up @@ -234,7 +234,7 @@ public override int GetHashCode()
}
else
{
throw new DeltaException("Unrecognized command");
return null;
}
}

Expand Down
19 changes: 19 additions & 0 deletions code/DeltaKustoUnitTest/CommandParsing/AddMvAdminTest.cs
Original file line number Diff line number Diff line change
@@ -0,0 +1,19 @@
using DeltaKustoLib.CommandModel;
using System;
using System.Linq;
using Xunit;

namespace DeltaKustoUnitTest.CommandParsing
{
public class AddMvAdminTest : ParsingTestBase
{
[Fact]
public void AddMvAdmin()
{
var commands = Parse(
".add materialized view SampleView admins ('aaduser=imikeoein@fabrikam.com')");

Assert.Empty(commands);
}
}
}

0 comments on commit 4000ae2

Please sign in to comment.