Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Response abstraction #4438

Merged
merged 45 commits into from
Oct 18, 2024
Merged
Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
45 commits
Select commit Hold shift + click to select a range
40ecf45
init
live1206 Sep 13, 2024
7da2225
Merge branch 'main' into response-abstraction
live1206 Sep 13, 2024
d170d95
cleanup
live1206 Sep 13, 2024
88a4620
clean up
live1206 Sep 13, 2024
0c8c172
Merge branch 'main' into response-abstraction
live1206 Sep 14, 2024
634c06a
wip
live1206 Sep 18, 2024
5bbcd8a
update FromValue expression
live1206 Sep 18, 2024
93932af
rename
live1206 Sep 19, 2024
f1a9a3d
restructure
live1206 Sep 19, 2024
1169baf
revert type change
live1206 Sep 19, 2024
9752376
Merge branch 'main' into response-abstraction
live1206 Sep 19, 2024
f062d77
make static methods abstract
live1206 Sep 23, 2024
d1889c2
Merge branch 'response-abstraction' of https://github.com/live1206/ty…
live1206 Sep 23, 2024
9eb3b66
update Type to CSharpType
live1206 Sep 23, 2024
56e0f8a
make ErrorResultDefinition public
live1206 Sep 25, 2024
8f6e642
update
live1206 Sep 26, 2024
47a6427
Merge branch 'main' into response-abstraction
live1206 Sep 26, 2024
4a94206
Add more abstraction to make Azure plugin work
live1206 Sep 30, 2024
37295c0
regen tests
live1206 Sep 30, 2024
49ee47d
Merge branch 'main' into response-abstraction
live1206 Sep 30, 2024
eb32597
fix typo
live1206 Sep 30, 2024
c29e8c0
update
live1206 Sep 30, 2024
eb98948
Merge branch 'main' into response-abstraction
live1206 Oct 8, 2024
fcf7ee5
add interfaces
live1206 Oct 9, 2024
109e93e
revert changes for serialization
live1206 Oct 9, 2024
f8c7bbb
revert test change
live1206 Oct 9, 2024
b0d4fca
Merge branch 'main' into response-abstraction
live1206 Oct 9, 2024
a038db4
fix
live1206 Oct 9, 2024
f83dd0b
cleanup
live1206 Oct 9, 2024
a5a379f
make test pass
live1206 Oct 9, 2024
aa27a8d
Merge branch 'main' into response-abstraction
live1206 Oct 10, 2024
100328f
minor
live1206 Oct 10, 2024
14f87d0
Merge branch 'main' into response-abstraction
live1206 Oct 10, 2024
48449f4
Merge branch 'main' into response-abstraction
live1206 Oct 11, 2024
456f13a
resolve comments
live1206 Oct 11, 2024
71bf272
Merge branch 'main' into response-abstraction
live1206 Oct 11, 2024
0cfe54f
Add IExpressionApi<T>
live1206 Oct 14, 2024
1189b18
add constraint
live1206 Oct 14, 2024
cb6c50e
Add ParameterName to IHttpRequestOptionsApi
live1206 Oct 16, 2024
486104c
Add tests for abstraction and simplify the implementation
live1206 Oct 17, 2024
aba79be
Merge branch 'main' into response-abstraction
live1206 Oct 17, 2024
c8a8735
fix typo
live1206 Oct 17, 2024
e3c067a
more typo
live1206 Oct 17, 2024
55637ee
fix typo in test data file name
live1206 Oct 17, 2024
5f192fb
Merge branch 'main' into response-abstraction
live1206 Oct 18, 2024
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@ public static ParameterProvider ClientOptions(CSharpType clientOptionsType)
=> new("options", $"The options for configuring the client.", clientOptionsType.WithNullable(true), initializationValue: New.Instance(clientOptionsType.WithNullable(true)));
public static readonly ParameterProvider KeyAuth = new("keyCredential", $"The token credential to copy", ClientModelPlugin.Instance.TypeFactory.KeyCredentialType);
public static readonly ParameterProvider MatchConditionsParameter = new("matchConditions", $"The content to send as the request conditions of the request.", ClientModelPlugin.Instance.TypeFactory.MatchConditionsType, DefaultOf(ClientModelPlugin.Instance.TypeFactory.MatchConditionsType));
public static readonly ParameterProvider RequestOptions = new("options", $"The request options, which can override default behaviors of the client pipeline on a per-call basis.", ClientModelPlugin.Instance.TypeFactory.HttpRequestOptionsApi.HttpRequestOptionsType);
public static readonly ParameterProvider RequestOptions = new(ClientModelPlugin.Instance.TypeFactory.HttpRequestOptionsApi.ParameterName, $"The request options, which can override default behaviors of the client pipeline on a per-call basis.", ClientModelPlugin.Instance.TypeFactory.HttpRequestOptionsApi.HttpRequestOptionsType);
public static readonly ParameterProvider RequestContent = new("content", $"The content to send as the body of the request.", ClientModelPlugin.Instance.TypeFactory.RequestContentApi.RequestContentType, location: ParameterLocation.Body) { Validation = ParameterValidationType.AssertNotNull };

// Known header parameters
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -22,5 +22,6 @@ public HttpRequestOptionsApi(CSharpType type, ValueExpression original) : base(t
public abstract HttpRequestOptionsApi ToExpression();

public abstract CSharpType HttpRequestOptionsType { get; }
public abstract string ParameterName { get; }
live1206 marked this conversation as resolved.
Show resolved Hide resolved
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -8,5 +8,7 @@ namespace Microsoft.Generator.CSharp.ClientModel.Providers
public interface IHttpRequestOptionsApi : IExpressionApi<HttpRequestOptionsApi>
{
CSharpType HttpRequestOptionsType { get; }
live1206 marked this conversation as resolved.
Show resolved Hide resolved

string ParameterName { get; }
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,8 @@ private RequestOptionsProvider() : base(typeof(RequestOptions), Empty)

public override CSharpType HttpRequestOptionsType => typeof(RequestOptions);

public override string ParameterName => "options";

public override ValueExpression ErrorOptions()
=> Original.NullConditional().Property(nameof(RequestOptions.ErrorOptions));

Expand Down
Loading