Skip to content

Commit

Permalink
fix: activity expression should ignore casing, json function should w…
Browse files Browse the repository at this point in the history
…ork on strings only
  • Loading branch information
arjendev committed Aug 31, 2023
1 parent 8190083 commit 6119308
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@

<PropertyGroup>
<PackageId>AzureDataFactory.TestingFramework</PackageId>
<PackageVersion>0.1.1-alpha</PackageVersion>
<PackageVersion>0.1.2-alpha</PackageVersion>
<Authors>arjendev</Authors>
<GeneratePackageOnBuild>true</GeneratePackageOnBuild>
</PropertyGroup>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ public ActivityExpression(string expression) : base(expression)
public TType Evaluate<TType>(PipelineRunState state)
{
var (activityName, fields) = GetActivityNameAndFields();
var activity = state.PipelineActivityResults.SingleOrDefault(x => x.Name == activityName) ??
var activity = state.PipelineActivityResults.SingleOrDefault(x => string.Equals(x.Name, activityName, StringComparison.CurrentCultureIgnoreCase)) ??
throw new ActivityNotFoundException(activityName);
if (activity.Status == null)
throw new ActivityNotEvaluatedException(activity.Name);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@ private static string Trim(string text, string trimArgument)
return text.Trim(trimArgument[0]);
}

private static object Json(object argument)
private static string Json(string argument)
{
return argument;
}
Expand Down

0 comments on commit 6119308

Please sign in to comment.