Adding features #219
Adding features #219
Conversation
+ Property ServiceName class ServiceSelector: + operator == + operator != + Tests class Agent: + method Checks with filter + Tests
There was a problem hiding this comment.
Hi @firerain-fd, thanks for being an active contributor.
I've reviewed your changes. I see how the "ServiceName" selector is being used, but I don't see how "Service" selector is being used (or can be even used). Please provide a use case (a test case) or remove that change.
|
|
||
| public override string Encode() => Self; | ||
|
|
||
| public static Filter operator ==(ServiceSelector selector, string value) => Filters.Eq(selector, value); |
There was a problem hiding this comment.
I don't think a "service" is a valid selector that can be used for anything. Also, this is not being used in an actual test (apart from encoding tests). I think that it should be removed.
There was a problem hiding this comment.
Hi.
The 'Service' selector is used in the following api queries:
https://developer.hashicorp.com/consul/api-docs/agent/service#list-services
https://developer.hashicorp.com/consul/api-docs/catalog#retrieve-map-of-services-for-a-node
https://developer.hashicorp.com/consul/api-docs/catalog#list-services-for-node
There was a problem hiding this comment.
Ok, I see. It would be great though if you implement at least one test case which uses this APIs.
| Task<WriteResult> CheckDeregister(string checkID, CancellationToken ct = default(CancellationToken)); | ||
| Task<WriteResult> CheckRegister(AgentCheckRegistration check, CancellationToken ct = default(CancellationToken)); | ||
| Task<QueryResult<Dictionary<string, AgentCheck>>> Checks(CancellationToken ct = default(CancellationToken)); | ||
| Task<QueryResult<Dictionary<string, AgentCheck>>> Checks(Filter filter, CancellationToken ct = default(CancellationToken)); |
| } | ||
|
|
||
| [Fact] | ||
| public async Task Agent_FilterChecks() |
Fixed filtering test assertion by service identifier (duplicated string). Added filtering test assertion by service name.
| Assert.Equal(svcID2, (await _client.Agent.Services(idSelector == svcID2)).Response.Keys.Single()); | ||
| Assert.Equal(svcID1, (await _client.Agent.Services(metaSelector[uniqueMeta] == "bar1")).Response.Keys.Single()); | ||
| Assert.Equal(svcID2, (await _client.Agent.Services(metaSelector[uniqueMeta] == "bar2")).Response.Keys.Single()); | ||
| Assert.Equal(svcID1, (await _client.Agent.Services(Selectors.Service == svcID1)).Response.Keys.Single()); |
2 participants
class Selectors:
Add Property ServiceName
class ServiceSelector:
Add operator ==
Add operator !=
Add Tests
class Agent:
Add method Checks with filter
Add Tests