Autogen-8.0 #662

mickjermsurawong-stripe · 2019-02-01T00:27:03Z

This is a working branch for autogen-8.0 beta.
cc @stripe/api-libraries

mickjermsurawong-stripe · 2019-02-08T18:15:44Z

@remi-stripe the commit started with test:... should correspond to a group of resources affected by breaking changes. Happy to go through them with you to find out any issues I might miss!

mickjermsurawong-stripe · 2019-02-13T03:31:41Z

This is failing now because I rebased on the latest master 7.20 which contain tests for classes that hasn't been generated.. The PR #672 will resolve this and make a parity with 7.20

mickjermsurawong-stripe

ptal @brandur-stripe @ob-stripe
I enumerated 18 patterns of API methods encountered in stripe-java generated from sanity check here [0]
I put them in spreadsheet here and randomly manually check the implementations by myself here [1]
I made another random checks (under column in beta-8.0 PR) and annotate them on PR here for ease of review. Please feel free to spotcheck the endpoints in the spreadsheet. There are a few endpoints still to be merged.

[0] https://git.corp.stripe.com/stripe-internal/sdk-autogen-java/pull/46
[1] https://docs.google.com/spreadsheets/d/1Rv-ybR2vXbM9UC26UBOPcnTtqyopic9KTjkrunjjk-w/edit#gid=0

mickjermsurawong-stripe · 2019-03-18T18:13:57Z

src/main/java/com/stripe/model/Account.java

+   * complete.
   */
-  public PersonCollection persons(Map<String, Object> params)
+  public static Account create(Map<String, Object> params, RequestOptions options)


Create (standard): Static method to create a resource

stripe-java/src/main/java/com/stripe/model/Account.java

Lines 231 to 235 in 2aa5db5

public static Account create(Map<String, Object> params, RequestOptions options)

throws StripeException {

String url = String.format("%s%s", Stripe.getApiBase(), "/v1/accounts");

return request(ApiResource.RequestMethod.POST, url, params, Account.class, options);

}

mickjermsurawong-stripe · 2019-03-18T18:14:36Z

src/main/java/com/stripe/model/Payout.java

+   * href="#balance_object">balance object</a> details available and pending amounts by source type.
   */
-  public static PayoutCollection list(Map<String, Object> params, RequestOptions options)
+  public static Payout create(Map<String, Object> params, RequestOptions options)


Create (standard): Static method to create a resource

stripe-java/src/main/java/com/stripe/model/Payout.java

Lines 282 to 286 in 2aa5db5

public static Payout create(Map<String, Object> params, RequestOptions options)

throws StripeException {

String url = String.format("%s%s", Stripe.getApiBase(), "/v1/payouts");

return request(ApiResource.RequestMethod.POST, url, params, Payout.class, options);

}

mickjermsurawong-stripe · 2019-03-18T18:16:20Z

src/main/java/com/stripe/model/FeeRefundCollection.java

-        Stripe.getApiBase(), this.getUrl()), params, FeeRefund.class, options);
+    String url = String.format("%s%s", Stripe.getApiBase(), this.getUrl());
+    return ApiResource.request(
+        ApiResource.RequestMethod.POST, url, params, FeeRefund.class, options);


Create (standard): Instance method to create sub-resource on collection already containing ID of the parent resource.

stripe-java/src/main/java/com/stripe/model/FeeRefundCollection.java

Lines 38 to 43 in 2aa5db5

public FeeRefund create(Map<String, Object> params, RequestOptions options)

throws StripeException {

String url = String.format("%s%s", Stripe.getApiBase(), this.getUrl());

return ApiResource.request(

ApiResource.RequestMethod.POST, url, params, FeeRefund.class, options);

}

mickjermsurawong-stripe · 2019-03-18T18:17:02Z

src/main/java/com/stripe/model/UsageRecord.java

   */
-  public static UsageRecord create(Map<String, Object> params, RequestOptions options)
+  public static UsageRecord createOnSubscriptionItem(


Create (custom): Static method to create sub-resource on parent resource, taking of parent resource explicitly.
Contrast this to creating on sub-resource collection relying on the implicit parent resource.

stripe-java/src/main/java/com/stripe/model/UsageRecord.java

Lines 69 to 79 in 2aa5db5

public static UsageRecord createOnSubscriptionItem(

String subscriptionItem, Map<String, Object> params, RequestOptions options)

throws StripeException {

String url =

String.format(

"%s%s",

Stripe.getApiBase(),

String.format("/v1/subscription_items/%s/usage_records", subscriptionItem));

return request(ApiResource.RequestMethod.POST, url, params, UsageRecord.class, options);

}

}

mickjermsurawong-stripe · 2019-03-18T18:18:06Z

src/main/java/com/stripe/model/Balance.java

-    return request(RequestMethod.GET, singleClassUrl(Balance.class), params, Balance.class,
-        options);
+    String url = String.format("%s%s", Stripe.getApiBase(), "/v1/balance");
+    return request(ApiResource.RequestMethod.GET, url, params, Balance.class, options);


Retrieve (standard): Static method to retrieve a singleton-like resource, taking no ID.

stripe-java/src/main/java/com/stripe/model/Balance.java

Lines 80 to 84 in 2aa5db5

public static Balance retrieve(Map<String, Object> params, RequestOptions options)

throws StripeException {

String url = String.format("%s%s", Stripe.getApiBase(), "/v1/balance");

return request(ApiResource.RequestMethod.GET, url, params, Balance.class, options);

}

mickjermsurawong-stripe · 2019-03-18T18:33:25Z

src/main/java/com/stripe/model/BankAccount.java

+          0,
+          null);
+    }
+    return request(ApiResource.RequestMethod.POST, url, params, BankAccount.class, options);


Update (custom): Instance method to update sub-resource having business-logic impact besides mutating the data. Same behavior of relying on instance ID attributes of instance and its parent resource just just like the standard update method on sub-resource.

stripe-java/src/main/java/com/stripe/model/BankAccount.java

Lines 279 to 300 in 2aa5db5

/** Verify a specified bank account for a given customer. */

public BankAccount verify(Map<String, Object> params, RequestOptions options)

throws StripeException {

String url;

if (this.getCustomer() != null) {

url =

String.format(

"%s%s",

Stripe.getApiBase(),

String.format(

"/v1/customers/%s/sources/%s/verify", this.getCustomer(), this.getId()));

} else {

throw new InvalidRequestException(

"Unable to construct url because [customer] field(s) are all null",

null,

null,

null,

0,

null);

}

return request(ApiResource.RequestMethod.POST, url, params, BankAccount.class, options);

}

mickjermsurawong-stripe · 2019-03-18T18:34:00Z

src/main/java/com/stripe/model/WebhookEndpoint.java

+    String url =
+        String.format(
+            "%s%s", Stripe.getApiBase(), String.format("/v1/webhook_endpoints/%s", this.getId()));
+    return request(ApiResource.RequestMethod.DELETE, url, params, WebhookEndpoint.class, options);


Delete (standard): Instance method to delete a resource on itself, relying on ID attribute of the instance.

stripe-java/src/main/java/com/stripe/model/WebhookEndpoint.java

Lines 182 to 193 in 2aa5db5

/**

* You can also delete webhook endpoints via the <a

* href="https://dashboard.stripe.com/account/webhooks">webhook endpoint management</a> page of

* the Stripe dashboard.

*/

public WebhookEndpoint delete(Map<String, Object> params, RequestOptions options)

throws StripeException {

String url =

String.format(

"%s%s", Stripe.getApiBase(), String.format("/v1/webhook_endpoints/%s", this.getId()));

return request(ApiResource.RequestMethod.DELETE, url, params, WebhookEndpoint.class, options);

}

mickjermsurawong-stripe · 2019-03-18T18:34:43Z

src/main/java/com/stripe/model/Card.java

+          0,
+          null);
    }
+    return request(ApiResource.RequestMethod.DELETE, url, params, Card.class, options);


Delete (standard): Instance method to delete a sub-resource on itself, relying on ID attributes of the instance, and of its parent resource. There is validation on parent resource ID.

stripe-java/src/main/java/com/stripe/model/Card.java

Lines 334 to 364 in 2aa5db5

/**

* Delete a specified external account for a given account.

*

* <p>Delete a specified source for a given customer.

*/

public Card delete(Map<String, Object> params, RequestOptions options) throws StripeException {

String url;

if (this.getAccount() != null) {

url =

String.format(

"%s%s",

Stripe.getApiBase(),

String.format(

"/v1/accounts/%s/external_accounts/%s", this.getAccount(), this.getId()));

} else if (this.getCustomer() != null) {

url =

String.format(

"%s%s",

Stripe.getApiBase(),

String.format("/v1/customers/%s/sources/%s", this.getCustomer(), this.getId()));

} else {

throw new InvalidRequestException(

"Unable to construct url because [account, customer] field(s) are all null",

null,

null,

null,

0,

null);

}

return request(ApiResource.RequestMethod.DELETE, url, params, Card.class, options);

}

mickjermsurawong-stripe · 2019-03-18T18:35:30Z

src/main/java/com/stripe/model/Customer.java

+    String url =
+        String.format(
+            "%s%s", Stripe.getApiBase(), String.format("/v1/customers/%s/discount", this.getId()));
+    return request(ApiResource.RequestMethod.DELETE, url, params, Discount.class, options);


Delete (custom): Instance method to delete an arbitrary resource relying ID attribute of the instance.
Method name makes it distinct from standard delete method.

stripe-java/src/main/java/com/stripe/model/Customer.java

Lines 285 to 292 in 2aa5db5

/** Removes the currently applied discount on a customer. */

public Discount deleteDiscount(Map<String, Object> params, RequestOptions options)

throws StripeException {

String url =

String.format(

"%s%s", Stripe.getApiBase(), String.format("/v1/customers/%s/discount", this.getId()));

return request(ApiResource.RequestMethod.DELETE, url, params, Discount.class, options);

}

mickjermsurawong-stripe · 2019-03-18T18:36:03Z

src/main/java/com/stripe/model/Source.java

+          0,
+          null);
    }
+    return request(ApiResource.RequestMethod.DELETE, url, params, Source.class, options);


Delete (custom): Instance method to delete-like a sub-resource on itself, relying on ID attributes of the instance, and its reference to parent resource. POST should be used instead. Currently one method has this behavior.

stripe-java/src/main/java/com/stripe/model/Source.java

Lines 187 to 206 in 2aa5db5

/** Delete a specified source for a given customer. */

public Source detach(Map<String, Object> params, RequestOptions options) throws StripeException {

String url;

if (this.getCustomer() != null) {

url =

String.format(

"%s%s",

Stripe.getApiBase(),

String.format("/v1/customers/%s/sources/%s", this.getCustomer(), this.getId()));

} else {

throw new InvalidRequestException(

"Unable to construct url because [customer] field(s) are all null",

null,

null,

null,

0,

null);

}

return request(ApiResource.RequestMethod.DELETE, url, params, Source.class, options);

}

mickjermsurawong-stripe · 2019-03-18T21:25:55Z

Added permalinks there for convenience of review without clicking to view the file

tmaxwell-stripe · 2019-03-19T17:17:20Z

I looked over the snippets of code that you pulled out into comments; they look correct to me.

ob-stripe · 2019-03-19T17:36:38Z

I've spot-checked a few other random methods, particularly some non-CRUDL methods (charge capture, etc.) and nothing stood out to me either. This looks pretty great!

…d Card in `/v1/customers/...`

…al update

…Link `craeteOnAccount`

Reversal to Transfer Reversal

…n favor of `createOnSubscriptionItem`

… EphemeralKey

…03e408f22669 in master""

…on mismatch (#664) * make event data backward compat * implement versioned data * rename work * rename exception * improve deserializeUnsafeWith * add comments using static pinned version

…388f25f30d57 in master"" (#666)

)

…msurawong/subscription-schedules (#672)

test: CardTest ambiguous call with typed param

fix person

* extract common deserializer method extract common logic * implement ApiParamRequest add api param request * Add to base test lenient request params comparison for list/array int/long comparison * use List<Object> instead of Object[] in untyped deserializer * provide documentation to deserializer and api request param * reorder import statments * fix other imports in new files

* [generated] source: spec3.sdk.yaml@non-master-spec-e27a05e in mickjermsurawong/no-v-change * [generated] source: spec3.sdk.yaml@spec-82bb471 in master * [generated] source: spec3.sdk.yaml@spec-3869a7f in master * [generated] source: spec3.sdk.yaml@non-master-spec-02ef86a in mickjermsurawong/undoc-sensitive-enum

This reverts commit c3c1157.

This reverts commit a1c2175.

This reverts commit 1eb0f45.

mickjermsurawong-stripe · 2019-03-19T23:04:04Z

thank you @remi-stripe @ob-stripe @tmaxwell-stripe @brandur-stripe !
I'm self-approving here because I suspect it'll be hard for anyone to say this is good overall without looking at all the code, and it's impossible so..

I've done manual review and spot-check and surfaced the different patterns.
I made sure that no tests are unnecessarily removed, and all of them are passing now
There's breaking changes diff have been reviewed..
Updated the lib to have pin api version

rattrayalex-stripe · 2019-03-19T23:12:49Z

LGTM

I trust you, Mick 😄

mickjermsurawong-stripe changed the title ~~Autogen-8.0 beta~~ [wip] Autogen-8.0 beta Feb 1, 2019

mickjermsurawong-stripe force-pushed the autogen/8.0-beta branch from 87394b5 to 6ce8daf Compare February 7, 2019 23:15

mickjermsurawong-stripe force-pushed the autogen/8.0-beta branch from 6ce8daf to 2733fc7 Compare February 13, 2019 03:08

mickjermsurawong-stripe force-pushed the autogen/8.0-beta branch 2 times, most recently from c40db5e to 5340c29 Compare February 26, 2019 06:48

mickjermsurawong-stripe force-pushed the autogen/8.0-beta branch 2 times, most recently from ec2a54d to fa53301 Compare March 15, 2019 21:07

mickjermsurawong-stripe commented Mar 18, 2019

View reviewed changes

stripe-ci assigned brandur-stripe and ob-stripe Mar 18, 2019

mickjermsurawong-stripe changed the title ~~[wip] Autogen-8.0 beta~~ Autogen-8.0 beta Mar 18, 2019

mickjermsurawong-stripe changed the title ~~Autogen-8.0 beta~~ Autogen-8.0 Mar 18, 2019

mickjermsurawong-stripe force-pushed the autogen/8.0-beta branch from 5e2136d to 12b67a4 Compare March 19, 2019 02:13

mickjermsurawong-stripe added 12 commits March 19, 2019 12:25

remove updated resource

1011a30

remove resources

2082454

remove updated resource

9042e51

remove updated resource

58dcfb7

remove resource

0f60b50

remove resources to be regenerated

646a08d

src: remove class specific deserializers from ApiResource.java

eceb0d6

test: use PaymentSource instead of ExternalAccount for BankAccount an…

c4e8150

…d Card in `/v1/customers/...`

test: remove Charge markSafe/markFraudulent method in favor of norm…

2278630

…al update

test: remove LoginLinkCollection for create method, in favor of Login…

c322913

…Link `craeteOnAccount`

test: rename Reversal to TransferReversal

9249255

Reversal to Transfer Reversal

test: remove UsageRecord create taking ID argument in parameters, i…

b8bf8dd

…n favor of `createOnSubscriptionItem`

mickjermsurawong-stripe and others added 23 commits March 19, 2019 12:25

src/test: [version-pin] pin API_VERSION and update RequestOptions and…

cfa2028

… EphemeralKey

test: add deserialization test to OrderItem expanding parent

3ade132

"[generated] source: spec3.sdk.yaml@spec-214186957a2324edd15aba2a7d8d…

25221d4

…03e408f22669 in master""

src/test: [version-pin] remove Stripe.apiVersion

100fd0e

Interface for handling event deserialization failure due to api versi…

25f1a0d

…on mismatch (#664) * make event data backward compat * implement versioned data * rename work * rename exception * improve deserializeUnsafeWith * add comments using static pinned version

"[generated] source: spec3.sdk.yaml@spec-c6ce0ce334c19193dc22b6b63f47…

477a884

…388f25f30d57 in master"" (#666)

"[generated] source: spec3.sdk.yaml@spec-3b52a80 in master"" (#668)

d9d9e3b

[generated] source: spec3.sdk.yaml@non-master-spec-a8fea08 in HEAD (#671

c5fd147

)

[generated] source: spec3.sdk.yaml@non-master-spec-d2b863d in mickjer…

0a682d5

…msurawong/subscription-schedules (#672)

test: Subscription default source becomes PaymentSource

ae1cf97

test: typed param setup - fix ambiguous call with null casting

6be40a2

test: CardTest ambiguous call with typed param

test: LegalEntity class is removed

82ba834

fix person

test: Person remove old document expansion

cae2d4e

fix: RequestOptions use lombok equal/hashcode instead

c709bac

test: Standardization consider request param as map param

6eaf838

Revert "test: Standardization consider request param as map param"

03d5d08

This reverts commit c3c1157.

Revert "Basic primitives for typed params (#679)"

5c2fbde

This reverts commit a1c2175.

Revert "test: typed param setup - fix ambiguous call with null casting"

2dffa68

This reverts commit 1eb0f45.

[generated] source: spec3.sdk.yaml@spec-9d9decd in master (#690)

581177e

[generated] source: spec3.sdk.yaml@spec-2662e57 in master (#692)

beab17f

[generated] source: spec3.sdk.yaml@spec-436464f in master (#693)

02502bb

mickjermsurawong-stripe force-pushed the autogen/8.0-beta branch from 0e3485b to 02502bb Compare March 19, 2019 19:25

remove parts that should be done by script (#696)

b688e5b

rattrayalex-stripe added the approved label Mar 19, 2019

mickjermsurawong-stripe merged commit 80e8487 into master Mar 19, 2019

mickjermsurawong-stripe deleted the autogen/8.0-beta branch March 29, 2019 03:30

	public static Account create(Map<String, Object> params, RequestOptions options)
	throws StripeException {
	String url = String.format("%s%s", Stripe.getApiBase(), "/v1/accounts");
	return request(ApiResource.RequestMethod.POST, url, params, Account.class, options);
	}

	public static Payout create(Map<String, Object> params, RequestOptions options)
	throws StripeException {
	String url = String.format("%s%s", Stripe.getApiBase(), "/v1/payouts");
	return request(ApiResource.RequestMethod.POST, url, params, Payout.class, options);
	}

	public FeeRefund create(Map<String, Object> params, RequestOptions options)
	throws StripeException {
	String url = String.format("%s%s", Stripe.getApiBase(), this.getUrl());
	return ApiResource.request(
	ApiResource.RequestMethod.POST, url, params, FeeRefund.class, options);
	}

	public static UsageRecord createOnSubscriptionItem(
	String subscriptionItem, Map<String, Object> params, RequestOptions options)
	throws StripeException {
	String url =
	String.format(
	"%s%s",
	Stripe.getApiBase(),
	String.format("/v1/subscription_items/%s/usage_records", subscriptionItem));
	return request(ApiResource.RequestMethod.POST, url, params, UsageRecord.class, options);
	}
	}

	public static Balance retrieve(Map<String, Object> params, RequestOptions options)
	throws StripeException {
	String url = String.format("%s%s", Stripe.getApiBase(), "/v1/balance");
	return request(ApiResource.RequestMethod.GET, url, params, Balance.class, options);
	}

	/** Verify a specified bank account for a given customer. */
	public BankAccount verify(Map<String, Object> params, RequestOptions options)
	throws StripeException {
	String url;
	if (this.getCustomer() != null) {
	url =
	String.format(
	"%s%s",
	Stripe.getApiBase(),
	String.format(
	"/v1/customers/%s/sources/%s/verify", this.getCustomer(), this.getId()));
	} else {
	throw new InvalidRequestException(
	"Unable to construct url because [customer] field(s) are all null",
	null,
	null,
	null,
	0,
	null);
	}
	return request(ApiResource.RequestMethod.POST, url, params, BankAccount.class, options);
	}

	/**
	* You can also delete webhook endpoints via the <a
	* href="https://dashboard.stripe.com/account/webhooks">webhook endpoint management</a> page of
	* the Stripe dashboard.
	*/
	public WebhookEndpoint delete(Map<String, Object> params, RequestOptions options)
	throws StripeException {
	String url =
	String.format(
	"%s%s", Stripe.getApiBase(), String.format("/v1/webhook_endpoints/%s", this.getId()));
	return request(ApiResource.RequestMethod.DELETE, url, params, WebhookEndpoint.class, options);
	}

	/**
	* Delete a specified external account for a given account.
	*
	* <p>Delete a specified source for a given customer.
	*/
	public Card delete(Map<String, Object> params, RequestOptions options) throws StripeException {
	String url;
	if (this.getAccount() != null) {
	url =
	String.format(
	"%s%s",
	Stripe.getApiBase(),
	String.format(
	"/v1/accounts/%s/external_accounts/%s", this.getAccount(), this.getId()));
	} else if (this.getCustomer() != null) {
	url =
	String.format(
	"%s%s",
	Stripe.getApiBase(),
	String.format("/v1/customers/%s/sources/%s", this.getCustomer(), this.getId()));
	} else {
	throw new InvalidRequestException(
	"Unable to construct url because [account, customer] field(s) are all null",
	null,
	null,
	null,
	0,
	null);
	}
	return request(ApiResource.RequestMethod.DELETE, url, params, Card.class, options);
	}

	/** Removes the currently applied discount on a customer. */
	public Discount deleteDiscount(Map<String, Object> params, RequestOptions options)
	throws StripeException {
	String url =
	String.format(
	"%s%s", Stripe.getApiBase(), String.format("/v1/customers/%s/discount", this.getId()));
	return request(ApiResource.RequestMethod.DELETE, url, params, Discount.class, options);
	}

	/** Delete a specified source for a given customer. */
	public Source detach(Map<String, Object> params, RequestOptions options) throws StripeException {
	String url;
	if (this.getCustomer() != null) {
	url =
	String.format(
	"%s%s",
	Stripe.getApiBase(),
	String.format("/v1/customers/%s/sources/%s", this.getCustomer(), this.getId()));
	} else {
	throw new InvalidRequestException(
	"Unable to construct url because [customer] field(s) are all null",
	null,
	null,
	null,
	0,
	null);
	}
	return request(ApiResource.RequestMethod.DELETE, url, params, Source.class, options);
	}

Autogen-8.0 #662

Autogen-8.0 #662

Uh oh!

Conversation

mickjermsurawong-stripe commented Feb 1, 2019 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

mickjermsurawong-stripe commented Feb 8, 2019

Uh oh!

mickjermsurawong-stripe commented Feb 13, 2019

Uh oh!

mickjermsurawong-stripe left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

mickjermsurawong-stripe commented Mar 18, 2019

Uh oh!

tmaxwell-stripe commented Mar 19, 2019

Uh oh!

ob-stripe commented Mar 19, 2019

Uh oh!

mickjermsurawong-stripe commented Mar 19, 2019 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

rattrayalex-stripe commented Mar 19, 2019

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

mickjermsurawong-stripe commented Feb 1, 2019 •

edited

Loading

mickjermsurawong-stripe left a comment •

edited

Loading

mickjermsurawong-stripe commented Mar 19, 2019 •

edited

Loading