Skip to content

Commit eb72509

Browse files
authored
feat: Project added to headers for every request (#10)
part of Visual-Regression-Tracker/Visual-Regression-Tracker#322
1 parent f8bc883 commit eb72509

File tree

3 files changed

+21
-9
lines changed

3 files changed

+21
-9
lines changed

VisualRegressionTracker/ApiClient.cs

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -9,10 +9,13 @@ public partial class ApiClient
99
public ApiClient(string baseUrl) : this(baseUrl, new HttpClient()) {}
1010

1111
public string ApiKey { get; set; }
12+
public string Project { get; set; }
13+
1214

1315
partial void PrepareRequest(HttpClient client, HttpRequestMessage request, string url)
1416
{
1517
request.Headers.Add("apiKey", new[] { ApiKey });
18+
request.Headers.Add("project", new[] { Project });
1619
}
1720
}
1821
}

VisualRegressionTracker/VisualRegressionTracker.cs

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,8 @@ public VisualRegressionTracker(Config config, HttpClient httpClient)
5252
httpClient ?? new HttpClient()
5353
)
5454
{
55-
ApiKey = this.config.ApiKey
55+
ApiKey = this.config.ApiKey,
56+
Project = this.config.Project
5657
};
5758
}
5859

tests/VisualRegressionTracker.Tests/VisualRegressionTrackerTests.cs

Lines changed: 16 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -70,8 +70,10 @@ public async Task Start()
7070
Assert.Equal(buildDto.Id, vrt.BuildId);
7171

7272
mock.VerifyRequest(1, req => {
73-
req.Headers.TryGetValues("apiKey", out var values);
74-
Assert.Equal(new[] {config.ApiKey}, values);
73+
req.Headers.TryGetValues("apiKey", out var apiKey);
74+
Assert.Equal(new[] {config.ApiKey}, apiKey);
75+
req.Headers.TryGetValues("project", out var project);
76+
Assert.Equal(new[] {config.Project}, project);
7577
});
7678
}
7779

@@ -99,8 +101,10 @@ public async Task Start_CancellationToken()
99101
Assert.Equal(buildDto.Id, vrt.BuildId);
100102

101103
mock.VerifyRequest(1, req => {
102-
req.Headers.TryGetValues("apiKey", out var values);
103-
Assert.Equal(new[] {config.ApiKey}, values);
104+
req.Headers.TryGetValues("apiKey", out var apiKey);
105+
Assert.Equal(new[] {config.ApiKey}, apiKey);
106+
req.Headers.TryGetValues("project", out var project);
107+
Assert.Equal(new[] {config.Project}, project);
104108
});
105109
}
106110

@@ -168,8 +172,10 @@ public async Task Stop()
168172
Assert.Null(vrt.ProjectId);
169173

170174
mock.VerifyRequest(1, req => {
171-
req.Headers.TryGetValues("apiKey", out var values);
172-
Assert.Equal(new[] {config.ApiKey}, values);
175+
req.Headers.TryGetValues("apiKey", out var apiKey);
176+
Assert.Equal(new[] {config.ApiKey}, apiKey);
177+
req.Headers.TryGetValues("project", out var project);
178+
Assert.Equal(new[] {config.Project}, project);
173179
});
174180
}
175181

@@ -191,8 +197,10 @@ public async Task Stop_CancellationToken()
191197
Assert.Null(vrt.ProjectId);
192198

193199
mock.VerifyRequest(1, req => {
194-
req.Headers.TryGetValues("apiKey", out var values);
195-
Assert.Equal(new[] {config.ApiKey}, values);
200+
req.Headers.TryGetValues("apiKey", out var apiKey);
201+
Assert.Equal(new[] {config.ApiKey}, apiKey);
202+
req.Headers.TryGetValues("project", out var project);
203+
Assert.Equal(new[] {config.Project}, project);
196204
});
197205
}
198206

0 commit comments

Comments
 (0)