Skip to content

Commit

Permalink
Visibility
Browse files Browse the repository at this point in the history
  • Loading branch information
FriggaHel committed Jan 23, 2025
1 parent d64f2f9 commit c4a9d55
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions visual-dotnet/SauceLabs.Visual/VisualClientBase.cs
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ namespace SauceLabs.Visual
{
public abstract class VisualClientBase : IDisposable
{
protected readonly List<string> ScreenshotIds = new List<string>();
private readonly List<string> _screenshotIds = new List<string>();
internal readonly VisualApi Api;
public VisualBuild Build { get; internal set; }

Expand Down Expand Up @@ -71,7 +71,7 @@ internal async Task<string> VisualCheckBaseAsync(string name,
diffingOptions: options.DiffingOptions?.ToDiffingOptionsIn(),
baselineOverride: (options.BaselineOverride ?? BaselineOverride)?.ToBaselineOverrideIn()
))).EnsureValidResponse();
result.Result.Diffs.Nodes.ToList().ForEach(d => ScreenshotIds.Add(d.Id));
result.Result.Diffs.Nodes.ToList().ForEach(d => _screenshotIds.Add(d.Id));
return result.Result.Id;
}

Expand Down Expand Up @@ -104,7 +104,7 @@ private async Task<Dictionary<DiffStatus, int>> FetchVisualResults(string buildI

var result = (await Api.DiffForTestResult(buildId)).EnsureValidResponse();
result.Result.Nodes
.Where(n => ScreenshotIds.Contains(n.Id))
.Where(n => _screenshotIds.Contains(n.Id))
.Aggregate(dict, (counts, node) =>
{
counts[node.Status] += 1;
Expand Down

0 comments on commit c4a9d55

Please sign in to comment.