Skip to content

Commit

Permalink
Merge 5c9f8fd into 7bf59df
Browse files Browse the repository at this point in the history
  • Loading branch information
pakrym-stripe committed Mar 6, 2023
2 parents 7bf59df + 5c9f8fd commit bf95503
Show file tree
Hide file tree
Showing 80 changed files with 138 additions and 161 deletions.
2 changes: 1 addition & 1 deletion test/Error.spec.ts
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
'use strict';

require('../testUtils');
require('./testUtils.js');

import * as Error from '../src/Error';
import {expect} from 'chai';
Expand Down
2 changes: 1 addition & 1 deletion test/Integration.spec.ts
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@

import * as childProcess from 'child_process';

const testUtils = require('../testUtils');
const testUtils = require('./testUtils.js');

describe('Integration test', function() {
this.timeout(50000);
Expand Down
2 changes: 1 addition & 1 deletion test/PlatformFunctions.spec.ts
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
'use strict';

require('../testUtils');
require('./testUtils.js');

import * as fs from 'fs';
import * as path from 'path';
Expand Down
4 changes: 2 additions & 2 deletions test/RequestSender.spec.js
Original file line number Diff line number Diff line change
Expand Up @@ -2,11 +2,11 @@

const {RequestSender} = require('../lib/RequestSender');

const stripe = require('../testUtils').getSpyableStripe();
const stripe = require('./testUtils.js').getSpyableStripe();
const expect = require('chai').expect;

const {HttpClientResponse} = require('../lib/net/HttpClient');
const utils = require('../testUtils');
const utils = require('./testUtils.js');
const nock = require('nock');

const {
Expand Down
4 changes: 2 additions & 2 deletions test/StripeResource.spec.js
Original file line number Diff line number Diff line change
Expand Up @@ -2,9 +2,9 @@

const nock = require('nock');

const stripe = require('../testUtils').getSpyableStripe();
const stripe = require('./testUtils.js').getSpyableStripe();
const expect = require('chai').expect;
const testUtils = require('../testUtils');
const testUtils = require('./testUtils.js');

const {StripeResource} = require('../lib/StripeResource');
const stripeMethod = StripeResource.method;
Expand Down
2 changes: 1 addition & 1 deletion test/Webhook.spec.ts
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
'use strict';

const {StripeSignatureVerificationError} = require('../lib/Error');
const {getSpyableStripe, FakeCryptoProvider} = require('../testUtils');
const {getSpyableStripe, FakeCryptoProvider} = require('./testUtils.js');
const stripe = getSpyableStripe();
import {expect} from 'chai';

Expand Down
93 changes: 59 additions & 34 deletions test/autoPagination.spec.js
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@

/* eslint-disable callback-return */

const testUtils = require('../testUtils');
const testUtils = require('./testUtils.js');
const {StripeResource} = require('../lib/StripeResource');

const {makeAutoPaginationMethods} = require('../lib/autoPagination');
Expand Down Expand Up @@ -305,42 +305,67 @@ describe('auto pagination', function() {
});

describe('async iterators', () => {
if (testUtils.envSupportsForAwait()) {
// `for await` throws a syntax error everywhere but node 10,
// so we must conditionally require it.
const forAwaitUntil = require('../testUtils/forAwait.node10')
.forAwaitUntil;

it('works with `for await` when that feature exists (user break)', () => {
const {paginator} = mockPagination(ID_PAGES, {});

return expect(
new Promise((resolve, reject) => {
forAwaitUntil(paginator, LIMIT)
.then((customers) => {
resolve(customers.map((customer) => customer.id));
})
.catch(reject);
})
).to.eventually.deep.equal(OBJECT_IDS.slice(0, LIMIT));
});

it('works with `for await` when that feature exists (end of list)', () => {
const {paginator} = mockPagination(ID_PAGES, {});
async function awaitUntil(iterator, limit) {
const items = [];
while (true) {
// eslint-disable-next-line no-await-in-loop
const {value, done} = await iterator.next();
if (done) {
break;
}
items.push(value);
if (items.length === limit) {
break;
}
if (items.length > limit) {
throw Error('Kept iterating after break.');
}
}
return items;
}

return expect(
new Promise((resolve, reject) => {
forAwaitUntil(paginator, TOTAL_OBJECTS + 1)
.then((customers) => {
resolve(customers.map((customer) => customer.id));
})
.catch(reject);
})
).to.eventually.deep.equal(OBJECT_IDS);
});
async function forAwaitUntil(iterator, limit) {
const items = [];
for await (const item of iterator) {
items.push(item);
if (items.length === limit) {
break;
}
if (items.length > limit) {
throw Error('Kept iterating after break.');
}
}
return items;
}

const awaitUntil = require('../testUtils/await.node7').awaitUntil;
it('works with `for await` when that feature exists (user break)', () => {
const {paginator} = mockPagination(ID_PAGES, {});

return expect(
new Promise((resolve, reject) => {
forAwaitUntil(paginator, LIMIT)
.then((customers) => {
resolve(customers.map((customer) => customer.id));
})
.catch(reject);
})
).to.eventually.deep.equal(OBJECT_IDS.slice(0, LIMIT));
});

it('works with `for await` when that feature exists (end of list)', () => {
const {paginator} = mockPagination(ID_PAGES, {});

return expect(
new Promise((resolve, reject) => {
forAwaitUntil(paginator, TOTAL_OBJECTS + 1)
.then((customers) => {
resolve(customers.map((customer) => customer.id));
})
.catch(reject);
})
).to.eventually.deep.equal(OBJECT_IDS);
});

it('works with `await` and a while loop when await exists', () => {
const {paginator} = mockPagination(ID_PAGES, {});

Expand Down
2 changes: 1 addition & 1 deletion test/flows.spec.ts
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
'use strict';

const testUtils = require('../testUtils');
const testUtils = require('./testUtils.js');
const chai = require('chai');
const stripe = require('../lib/stripe.node')(
testUtils.getUserStripeKey(),
Expand Down
2 changes: 1 addition & 1 deletion test/resources/Account.spec.js
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
'use strict';

const stripe = require('../../testUtils').getSpyableStripe();
const stripe = require('../testUtils.js').getSpyableStripe();
const expect = require('chai').expect;

const TEST_AUTH_KEY = 'aGN0bIwXnHdw5645VABjPdSn8nWY7G11';
Expand Down
2 changes: 1 addition & 1 deletion test/resources/AccountLinks.spec.js
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
'use strict';

const stripe = require('../../testUtils').getSpyableStripe();
const stripe = require('../testUtils.js').getSpyableStripe();
const expect = require('chai').expect;

describe('AccountLinks Resource', () => {
Expand Down
2 changes: 1 addition & 1 deletion test/resources/ApplePayDomains.spec.js
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
'use strict';

const stripe = require('../../testUtils').getSpyableStripe();
const stripe = require('../testUtils.js').getSpyableStripe();
const expect = require('chai').expect;

describe('ApplePayDomains Resource', () => {
Expand Down
2 changes: 1 addition & 1 deletion test/resources/ApplicationFees.spec.js
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
'use strict';

const stripe = require('../../testUtils').getSpyableStripe();
const stripe = require('../testUtils.js').getSpyableStripe();
const expect = require('chai').expect;

describe('ApplicationFee Resource', () => {
Expand Down
2 changes: 1 addition & 1 deletion test/resources/Balance.spec.js
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
'use strict';

const stripe = require('../../testUtils').getSpyableStripe();
const stripe = require('../testUtils.js').getSpyableStripe();
const expect = require('chai').expect;

describe('Balance Resource', () => {
Expand Down
2 changes: 1 addition & 1 deletion test/resources/BalanceTransactions.spec.js
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
'use strict';

const stripe = require('../../testUtils').getSpyableStripe();
const stripe = require('../testUtils.js').getSpyableStripe();
const expect = require('chai').expect;

describe('BalanceTransactions Resource', function() {
Expand Down
2 changes: 1 addition & 1 deletion test/resources/BillingPortal/Configurations.spec.js
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
'use strict';

const stripe = require('../../../testUtils').getSpyableStripe();
const stripe = require('../../testUtils.js').getSpyableStripe();

const expect = require('chai').expect;

Expand Down
2 changes: 1 addition & 1 deletion test/resources/BillingPortal/Sessions.spec.js
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
'use strict';

const stripe = require('../../../testUtils').getSpyableStripe();
const stripe = require('../../testUtils.js').getSpyableStripe();

const expect = require('chai').expect;

Expand Down
2 changes: 1 addition & 1 deletion test/resources/Charges.spec.js
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
'use strict';

const stripe = require('../../testUtils').getSpyableStripe();
const stripe = require('../testUtils.js').getSpyableStripe();
const expect = require('chai').expect;

describe('Charge Resource', () => {
Expand Down
2 changes: 1 addition & 1 deletion test/resources/Checkout/Sessions.spec.js
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
'use strict';

const stripe = require('../../../testUtils').getSpyableStripe();
const stripe = require('../../testUtils.js').getSpyableStripe();

const expect = require('chai').expect;

Expand Down
2 changes: 1 addition & 1 deletion test/resources/CountrySpecs.spec.js
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
'use strict';

const stripe = require('../../testUtils').getSpyableStripe();
const stripe = require('../testUtils.js').getSpyableStripe();
const expect = require('chai').expect;

describe('CountrySpecs Resource', () => {
Expand Down
2 changes: 1 addition & 1 deletion test/resources/Coupons.spec.js
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
'use strict';

const stripe = require('../../testUtils').getSpyableStripe();
const stripe = require('../testUtils.js').getSpyableStripe();
const expect = require('chai').expect;

describe('Coupons Resource', () => {
Expand Down
2 changes: 1 addition & 1 deletion test/resources/CreditNotes.spec.js
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
'use strict';

const stripe = require('../../testUtils').getSpyableStripe();
const stripe = require('../testUtils.js').getSpyableStripe();
const expect = require('chai').expect;

describe('CreditNotes Resource', () => {
Expand Down
2 changes: 1 addition & 1 deletion test/resources/Customers.spec.js
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
'use strict';

const stripe = require('../../testUtils').getSpyableStripe();
const stripe = require('../testUtils.js').getSpyableStripe();
const expect = require('chai').expect;

const TEST_AUTH_KEY = 'aGN0bIwXnHdw5645VABjPdSn8nWY7G11';
Expand Down
2 changes: 1 addition & 1 deletion test/resources/Disputes.spec.js
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
'use strict';

const stripe = require('../../testUtils').getSpyableStripe();
const stripe = require('../testUtils.js').getSpyableStripe();
const expect = require('chai').expect;

describe('Dispute Resource', () => {
Expand Down
2 changes: 1 addition & 1 deletion test/resources/EphemeralKeys.spec.js
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
'use strict';

const getSpyableStripe = require('../../testUtils').getSpyableStripe;
const getSpyableStripe = require('../testUtils.js').getSpyableStripe;
const expect = require('chai').expect;

function errorsOnNoStripeVersion(stripe) {
Expand Down
2 changes: 1 addition & 1 deletion test/resources/Events.spec.js
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
'use strict';

const stripe = require('../../testUtils').getSpyableStripe();
const stripe = require('../testUtils.js').getSpyableStripe();
const expect = require('chai').expect;

describe('Events Resource', () => {
Expand Down
2 changes: 1 addition & 1 deletion test/resources/ExchangeRates.spec.js
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
'use strict';

const stripe = require('../../testUtils').getSpyableStripe();
const stripe = require('../testUtils.js').getSpyableStripe();
const expect = require('chai').expect;

describe('ExchangeRates Resource', () => {
Expand Down
2 changes: 1 addition & 1 deletion test/resources/FileLinks.spec.js
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
'use strict';

const stripe = require('../../testUtils').getSpyableStripe();
const stripe = require('../testUtils.js').getSpyableStripe();
const expect = require('chai').expect;

describe('FileLinks Resource', () => {
Expand Down
2 changes: 1 addition & 1 deletion test/resources/Files.spec.js
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
'use strict';

const stripe = require('../../testUtils').getSpyableStripe();
const stripe = require('../testUtils.js').getSpyableStripe();
const expect = require('chai').expect;
const fs = require('fs');
const path = require('path');
Expand Down
2 changes: 1 addition & 1 deletion test/resources/Identity/VerificationReport.spec.js
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
'use strict';

const stripe = require('../../../testUtils').getSpyableStripe();
const stripe = require('../../testUtils.js').getSpyableStripe();
const expect = require('chai').expect;

describe('Identity', () => {
Expand Down
2 changes: 1 addition & 1 deletion test/resources/Identity/VerificationSession.spec.js
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
'use strict';

const stripe = require('../../../testUtils').getSpyableStripe();
const stripe = require('../../testUtils.js').getSpyableStripe();
const expect = require('chai').expect;

describe('Identity', () => {
Expand Down
2 changes: 1 addition & 1 deletion test/resources/Integration.spec.js
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@

// Resource integration tests which use stripe-mock.

const stripe = require('../../testUtils').getStripeMockClient();
const stripe = require('../testUtils.js').getStripeMockClient();
const expect = require('chai').expect;

describe('Customers Resource', () => {
Expand Down
2 changes: 1 addition & 1 deletion test/resources/InvoiceItems.spec.js
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
'use strict';

const stripe = require('../../testUtils').getSpyableStripe();
const stripe = require('../testUtils.js').getSpyableStripe();
const expect = require('chai').expect;

describe('InvoiceItems Resource', () => {
Expand Down
2 changes: 1 addition & 1 deletion test/resources/Invoices.spec.js
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
'use strict';

const stripe = require('../../testUtils').getSpyableStripe();
const stripe = require('../testUtils.js').getSpyableStripe();
const expect = require('chai').expect;

describe('Invoices Resource', () => {
Expand Down
2 changes: 1 addition & 1 deletion test/resources/Issuing/Authorization.spec.js
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
'use strict';

const stripe = require('../../../testUtils').getSpyableStripe();
const stripe = require('../../testUtils.js').getSpyableStripe();
const expect = require('chai').expect;

describe('Issuing', () => {
Expand Down
2 changes: 1 addition & 1 deletion test/resources/Issuing/Cardholders.spec.js
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
'use strict';

const stripe = require('../../../testUtils').getSpyableStripe();
const stripe = require('../../testUtils.js').getSpyableStripe();

const expect = require('chai').expect;

Expand Down
Loading

0 comments on commit bf95503

Please sign in to comment.