Issues/86 v2 #87

lf-jason-jia · 2023-09-15T15:54:09Z

In this PR, we built the v2 repository client library. Here are some areas we worked on:

Generate code from the new v2 schema.
Update the manual code accordingly.
Update integration and unit tests accordingly.

…rt of .NET Standard

…ts since the API names have changed

…the API

lf-jason-jia added 30 commits September 11, 2023 16:39

No longer need most of the overrides

a13eb6e

Remove more overrides

aa3e02e

Generate clients, lots to fix

6fc19d1

Prefer list

af705cd

Manual fix to generated code due to ComponentModel.Annotations not pa…

1368072

…rt of .NET Standard

Server session is gone for good

765e540

Remove unnecessary long names

b605f24

Update attributes APIs

10d9517

Update entries APIs

5d2d6ea

Update APIs for field definitions

b57b1e6

Update APIs for link definitions

675c280

Update APIs for listing repositories

d3952b9

Update APIs for search

28ba91a

Update APIs for tag definitions

77f3fc7

WIP: update APIs for template definitions

3503709

Finish updating APIs for template definitions

b0ade44

No longer needed

033d5d9

No need to use so many fully qualified names

a0d8fbf

Again, no need for fully qualified name

00b7521

Use shorter names

adc81f0

Use null propagation

e45b4fb

Fix CreateEntry method

14c66d4

Add missing API

99daa99

Fix test util methods and use simple import for entry creation

1b64aa1

Server session is gone

6e084ff

Update parameter naming for entries API

214a54b

Update tests for attributes APIs

50ea7ed

Update tests for audit reason APIs

11192d4

For tests of attribtues APIs, rename test classes names and their tes…

e177ba7

…ts since the API names have changed

Rename tests for audit reasons since their names have changed

2302dcc

lf-jason-jia added 20 commits September 14, 2023 15:56

Test no need to support .net core 3

d028012

Simplify test naming

5debc1e

ImportEntryRequest must have its Name field filled

0046fd7

Problem details have changed

53c7876

FieldToUpdate needs its Name field set

189b337

Fix set link test

916e331

LFS behavior has changed, use a more robust check

097ea84

Pass in repository ID instead of task ID

7d1f0f6

Non-null check is consistent with the JS client

3a19927

Should pass in repository ID instead

7292eb0

Update task cancellation usage

a6f667c

We already check equality using Linq

ad08f89

Update task cancellation test since the original test no longer fits …

5a6bdb5

…the API

There's no server session anymore, so remove its tests

35319b1

There's no point unit testing auto generated implementation

838b475

Remove tests since related class no longer exists

06324ab

Again, let's no to test auto-generated implementation

20c3a21

Remove unused import

6295858

Remove tests since CreateEntryResult no longer exists

bea808b

Remove the test since it's testing .NET HTTP client

d38da1d

samadpaydar3 previously approved these changes Sep 15, 2023

View reviewed changes

Use the repository that other client libraries use

3321fff

lf-jason-jia dismissed samadpaydar3’s stale review via 3321fff September 15, 2023 20:51

samadpaydar3 previously approved these changes Sep 15, 2023

View reviewed changes

Do not run on production

4032034

lf-jason-jia dismissed samadpaydar3’s stale review via 4032034 September 15, 2023 20:56

samadpaydar3 approved these changes Sep 15, 2023

View reviewed changes

lf-jason-jia merged commit b9dbe46 into 2.x Sep 15, 2023
5 of 6 checks passed

lf-jason-jia deleted the issues/86-v2 branch September 15, 2023 21:04

lf-jason-jia mentioned this pull request Sep 15, 2023

Make clients for v2 #86

Closed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Issues/86 v2 #87

Issues/86 v2 #87

lf-jason-jia commented Sep 15, 2023

Issues/86 v2 #87

Issues/86 v2 #87

Conversation

lf-jason-jia commented Sep 15, 2023