| 123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325326327328329330331332333334335336337338339340341342343344345346347348349350351352353354355356357358359360361362363364365366367368369370371372373374375376377378379380381382383384385386387388389390391392393394395396397398399400401402403404405406407408409410411412413414415416417418419420421422423424425426427428429430431432433434435436437438439440441442443444445446447448449450451452453454455456457458459460461462463464465466467468469470471472473474475476477478479480481482483484485486487488489490491492493494495496497498499500501502503504505506507508509510511512513514515516517518519520521522523524525526527528529530531532533534535536537538539540541542543544545546547548549550551552553554555556557558559560561562563564565566567568569570571572573574575576577578579580581582583584585586587588589590591592593594595596597598599600601602603604605606607608609610611612613614615616617618619620621622623624625626627628629630631632633634635636637638639640641642643644645646647648649650651652653654655656657658659660661662663664665666667668669670671672673674675676677678679680681682683684685686687688689690691692693694695696697698699700701702703704705706707708709710711712713714715716717718719720721722723724725726727728729730731732733734735736737738739740741742743744745746747748749750751752753754755756757758759760761762763764765766767768769770771772773774775776777778779780781782783784785786787788789790791792793794795796797798799800801802803804805806807808809810811812813814815816817818819820821822823824825826827828829830831832833834835836837838839840841842843844845846847848849850851852853854855856857858859860861862863864865866867868869870871872873874875876877878879880881882883884885886887888889890891892893894895896897898899900901902903904905906907908909910911912913914915916917918919920921922923924925926927928929930931932933934935936937938939940941942943944945946947948949950951952953954955956957958959960961962963964965966967968969970971972973974975976977978979980981982983984985986987988989990991992993994995996997998999100010011002100310041005100610071008100910101011101210131014101510161017101810191020102110221023102410251026102710281029103010311032103310341035103610371038103910401041 |
- import { beforeEach, describe, it, vi, assert, expect } from 'vitest'
- import sinon from 'sinon'
- import mongodb from 'mongodb-legacy'
- const modulePath =
- '../../../../app/src/Features/Subscription/SubscriptionUpdater'
- const { ObjectId } = mongodb
- describe('SubscriptionUpdater', function () {
- beforeEach(async function (ctx) {
- ctx.recurlyPlan = { planCode: 'recurly-plan' }
- ctx.recurlySubscription = {
- uuid: '1238uoijdasjhd',
- plan: {
- plan_code: ctx.recurlyPlan.planCode,
- },
- }
- ctx.adminUser = { _id: (ctx.adminuser_id = '5208dd34438843e2db000007') }
- ctx.otherUserId = '5208dd34438842e2db000005'
- ctx.allUserIds = ['13213', 'dsadas', 'djsaiud89']
- ctx.subscription = {
- _id: '111111111111111111111111',
- admin_id: ctx.adminUser._id,
- manager_ids: [ctx.adminUser._id],
- member_ids: [],
- save: sinon.stub().resolves(),
- planCode: 'student_or_something',
- recurlySubscription_id: 'abc123def456fab789',
- }
- ctx.user_id = ctx.adminuser_id
- ctx.groupSubscription = {
- _id: '222222222222222222222222',
- admin_id: ctx.adminUser._id,
- manager_ids: [ctx.adminUser._id],
- member_ids: ctx.allUserIds,
- save: sinon.stub().resolves(),
- groupPlan: true,
- planCode: 'group_subscription',
- recurlySubscription_id: '456fab789abc123def',
- }
- ctx.betterGroupSubscription = {
- _id: '999999999999999999999999',
- admin_id: ctx.adminUser._id,
- manager_ids: [ctx.adminUser._id],
- member_ids: [ctx.otherUserId],
- save: sinon.stub().resolves(),
- groupPlan: true,
- planCode: 'better_group_subscription',
- recurlySubscription_id: '123def456fab789abc',
- }
- const subscription = ctx.subscription
- ctx.SubscriptionModel = class {
- constructor(opts) {
- // Always return our mock subscription when creating a new one
- subscription.admin_id = opts.admin_id
- subscription.manager_ids = [opts.admin_id]
- return subscription
- }
- save() {
- return Promise.resolve(subscription)
- }
- }
- ctx.SubscriptionModel.deleteOne = sinon
- .stub()
- .returns({ exec: sinon.stub().resolves() })
- ctx.SubscriptionModel.updateOne = sinon
- .stub()
- .returns({ exec: sinon.stub().resolves() })
- ctx.SubscriptionModel.findOne = sinon.stub().resolves()
- ctx.SubscriptionModel.findById = sinon.stub().resolves()
- ctx.SubscriptionModel.updateMany = sinon
- .stub()
- .returns({ exec: sinon.stub().resolves() })
- ctx.SubscriptionModel.findOneAndUpdate = sinon.stub().returns({
- exec: sinon.stub().resolves(ctx.subscription),
- })
- ctx.SSOConfigModel = class {}
- ctx.SSOConfigModel.findOne = sinon.stub().returns({
- lean: sinon.stub().returns({
- exec: sinon.stub().resolves({ enabled: true }),
- }),
- })
- ctx.SubscriptionLocator = {
- promises: {
- getUsersSubscription: sinon.stub(),
- getGroupSubscriptionMemberOf: sinon.stub(),
- getMemberSubscriptions: sinon.stub().resolves([]),
- getSubscription: sinon.stub(),
- },
- }
- ctx.SubscriptionLocator.promises.getSubscription
- .withArgs(ctx.subscription._id)
- .resolves(ctx.subscription)
- ctx.Settings = {
- defaultPlanCode: 'personal',
- defaultFeatures: { default: 'features' },
- plans: [
- ctx.recurlyPlan,
- { planCode: ctx.subscription.planCode, features: {} },
- {
- planCode: ctx.groupSubscription.planCode,
- features: {
- collaborators: 10,
- compileTimeout: 60,
- dropbox: true,
- },
- },
- {
- planCode: ctx.betterGroupSubscription.planCode,
- features: {
- collaborators: -1,
- compileTimeout: 240,
- dropbox: true,
- },
- },
- ],
- mongo: {
- options: {
- appname: 'web',
- maxPoolSize: 100,
- serverSelectionTimeoutMS: 60000,
- socketTimeoutMS: 60000,
- monitorCommands: true,
- family: 4,
- },
- url: 'mongodb://mongo/test-overleaf',
- hasSecondaries: false,
- },
- }
- ctx.UserFeaturesUpdater = {
- promises: {
- updateFeatures: sinon.stub().resolves(),
- },
- }
- ctx.ReferalFeatures = {
- promises: {
- getBonusFeatures: sinon.stub().resolves(),
- },
- }
- ctx.FeaturesUpdater = {
- promises: {
- scheduleRefreshFeatures: sinon.stub().resolves(),
- refreshFeatures: sinon.stub().resolves({}),
- },
- }
- ctx.DeletedSubscription = {
- findOneAndUpdate: sinon.stub().returns({ exec: sinon.stub().resolves() }),
- }
- ctx.AnalyticsManager = {
- recordEventForUserInBackground: sinon.stub().resolves(),
- setUserPropertyForUserInBackground: sinon.stub(),
- registerAccountMapping: sinon.stub(),
- }
- ctx.Features = {
- hasFeature: sinon.stub().returns(false),
- }
- ctx.UserAuditLogHandler = {
- promises: {
- addEntry: sinon.stub().resolves(),
- },
- }
- ctx.UserUpdater = {
- promises: {
- updateUser: sinon.stub().resolves(),
- },
- }
- vi.doMock('../../../../app/src/models/Subscription', () => ({
- Subscription: ctx.SubscriptionModel,
- }))
- vi.doMock('../../../../app/src/models/SSOConfig', () => ({
- SSOConfig: ctx.SSOConfigModel,
- }))
- vi.doMock(
- '../../../../app/src/Features/Subscription/UserFeaturesUpdater',
- () => ({
- default: ctx.UserFeaturesUpdater,
- })
- )
- vi.doMock(
- '../../../../app/src/Features/Subscription/SubscriptionLocator',
- () => ({
- default: ctx.SubscriptionLocator,
- })
- )
- vi.doMock('@overleaf/settings', () => ({
- default: ctx.Settings,
- }))
- vi.doMock('../../../../app/src/infrastructure/mongodb', () => ({
- db: {},
- ObjectId,
- }))
- vi.doMock(
- '../../../../app/src/Features/Subscription/FeaturesUpdater',
- () => ({
- default: ctx.FeaturesUpdater,
- })
- )
- vi.doMock('../../../../app/src/models/DeletedSubscription', () => ({
- DeletedSubscription: ctx.DeletedSubscription,
- }))
- vi.doMock(
- '../../../../app/src/Features/Analytics/AnalyticsManager',
- () => ({
- default: ctx.AnalyticsManager,
- })
- )
- vi.doMock(
- '../../../../app/src/Features/Analytics/AccountMappingHelper',
- () => ({
- default: (ctx.AccountMappingHelper = {
- generateSubscriptionToRecurlyMapping: sinon.stub(),
- }),
- })
- )
- vi.doMock('../../../../app/src/infrastructure/Features', () => ({
- default: ctx.Features,
- }))
- vi.doMock('../../../../app/src/Features/User/UserAuditLogHandler', () => ({
- default: ctx.UserAuditLogHandler,
- }))
- vi.doMock('../../../../app/src/Features/User/UserUpdater', () => ({
- default: ctx.UserUpdater,
- }))
- vi.doMock('../../../../app/src/infrastructure/Modules', () => ({
- default: (ctx.Modules = {
- promises: {
- hooks: {
- fire: sinon.stub().resolves(),
- },
- },
- }),
- }))
- ctx.SubscriptionUpdater = (await import(modulePath)).default
- })
- describe('updateAdmin', function () {
- it('should update the subscription admin', async function (ctx) {
- ctx.subscription.groupPlan = true
- await ctx.SubscriptionUpdater.promises.updateAdmin(
- ctx.subscription,
- ctx.otherUserId
- )
- const query = {
- _id: new ObjectId(ctx.subscription._id),
- customAccount: true,
- }
- const update = {
- $set: { admin_id: new ObjectId(ctx.otherUserId) },
- $addToSet: { manager_ids: new ObjectId(ctx.otherUserId) },
- }
- ctx.SubscriptionModel.updateOne.should.have.been.calledOnce
- ctx.SubscriptionModel.updateOne.should.have.been.calledWith(query, update)
- })
- it('should remove the manager for non-group subscriptions', async function (ctx) {
- await ctx.SubscriptionUpdater.promises.updateAdmin(
- ctx.subscription,
- ctx.otherUserId
- )
- const query = {
- _id: new ObjectId(ctx.subscription._id),
- customAccount: true,
- }
- const update = {
- $set: {
- admin_id: new ObjectId(ctx.otherUserId),
- manager_ids: [new ObjectId(ctx.otherUserId)],
- },
- }
- ctx.SubscriptionModel.updateOne.should.have.been.calledOnce
- ctx.SubscriptionModel.updateOne.should.have.been.calledWith(query, update)
- })
- })
- describe('transferSubscriptionOwnership', function () {
- it('should transfer the subscription ownership for group subscriptions', async function (ctx) {
- ctx.subscription.groupPlan = true
- ctx.subscription.paymentProvider = {
- id: 'stripe-123',
- name: 'stripe-us',
- }
- await ctx.SubscriptionUpdater.promises.transferSubscriptionOwnership(
- ctx.subscription,
- ctx.otherUserId,
- false
- )
- const query = {
- _id: new ObjectId(ctx.subscription._id),
- }
- const update = {
- $set: {
- admin_id: new ObjectId(ctx.otherUserId),
- previousPaymentProvider: ctx.subscription.paymentProvider,
- },
- $addToSet: { manager_ids: new ObjectId(ctx.otherUserId) },
- }
- ctx.SubscriptionModel.updateOne.should.have.been.calledOnce
- ctx.SubscriptionModel.updateOne.should.have.been.calledWith(query, update)
- })
- it('should transfer the subscription ownership for non-group subscriptions', async function (ctx) {
- ctx.subscription.paymentProvider = {
- id: 'stripe-123',
- name: 'stripe-us',
- }
- await ctx.SubscriptionUpdater.promises.transferSubscriptionOwnership(
- ctx.subscription,
- ctx.otherUserId,
- false
- )
- const query = {
- _id: new ObjectId(ctx.subscription._id),
- }
- const update = {
- $set: {
- admin_id: new ObjectId(ctx.otherUserId),
- manager_ids: [new ObjectId(ctx.otherUserId)],
- previousPaymentProvider: ctx.subscription.paymentProvider,
- },
- }
- ctx.SubscriptionModel.updateOne.should.have.been.calledOnce
- ctx.SubscriptionModel.updateOne.should.have.been.calledWith(query, update)
- })
- it('should clear previousPaymentProvider when clearPreviousPaymentProvider is true', async function (ctx) {
- ctx.subscription.paymentProvider = {
- id: 'stripe-123',
- name: 'stripe-us',
- }
- await ctx.SubscriptionUpdater.promises.transferSubscriptionOwnership(
- ctx.subscription,
- ctx.otherUserId,
- true
- )
- const query = {
- _id: new ObjectId(ctx.subscription._id),
- }
- const update = {
- $set: {
- admin_id: new ObjectId(ctx.otherUserId),
- manager_ids: [new ObjectId(ctx.otherUserId)],
- },
- $unset: { previousPaymentProvider: 1 },
- }
- ctx.SubscriptionModel.updateOne.should.have.been.calledOnce
- ctx.SubscriptionModel.updateOne.should.have.been.calledWith(query, update)
- })
- })
- describe('syncSubscription', function () {
- beforeEach(function (ctx) {
- ctx.SubscriptionLocator.promises.getUsersSubscription.resolves(
- ctx.subscription
- )
- })
- it('should update the subscription if the user already is admin of one', async function (ctx) {
- await ctx.SubscriptionUpdater.promises.syncSubscription(
- ctx.recurlySubscription,
- ctx.adminUser._id
- )
- ctx.SubscriptionLocator.promises.getUsersSubscription
- .calledWith(ctx.adminUser._id)
- .should.equal(true)
- })
- it('should not call updateFeatures with group subscription if recurly subscription is not expired', async function (ctx) {
- await ctx.SubscriptionUpdater.promises.syncSubscription(
- ctx.recurlySubscription,
- ctx.adminUser._id
- )
- ctx.SubscriptionLocator.promises.getUsersSubscription
- .calledWith(ctx.adminUser._id)
- .should.equal(true)
- ctx.UserFeaturesUpdater.promises.updateFeatures.called.should.equal(false)
- })
- })
- describe('updateSubscriptionFromRecurly', function () {
- afterEach(function (ctx) {
- ctx.subscription.member_ids = []
- delete ctx.subscription.paymentProvider
- })
- it('should update the subscription with token etc when not expired', async function (ctx) {
- await ctx.SubscriptionUpdater.promises.updateSubscriptionFromRecurly(
- ctx.recurlySubscription,
- ctx.subscription,
- {}
- )
- ctx.subscription.recurlySubscription_id.should.equal(
- ctx.recurlySubscription.uuid
- )
- ctx.subscription.planCode.should.equal(
- ctx.recurlySubscription.plan.plan_code
- )
- ctx.subscription.save.called.should.equal(true)
- expect(
- ctx.FeaturesUpdater.promises.scheduleRefreshFeatures
- ).to.have.been.calledWith(ctx.adminUser._id)
- })
- it('should send a recurly account mapping event', async function (ctx) {
- const createdAt = new Date().toISOString()
- ctx.AccountMappingHelper.generateSubscriptionToRecurlyMapping.returns({
- source: 'recurly',
- sourceEntity: 'subscription',
- sourceEntityId: ctx.recurlySubscription.uuid,
- target: 'v2',
- targetEntity: 'subscription',
- targetEntityId: ctx.subscription._id,
- createdAt,
- })
- await ctx.SubscriptionUpdater.promises.updateSubscriptionFromRecurly(
- ctx.recurlySubscription,
- ctx.subscription,
- {}
- )
- expect(
- ctx.AccountMappingHelper.generateSubscriptionToRecurlyMapping
- ).to.have.been.calledWith(
- ctx.subscription._id,
- ctx.recurlySubscription.uuid
- )
- expect(
- ctx.AnalyticsManager.registerAccountMapping
- ).to.have.been.calledWith({
- source: 'recurly',
- sourceEntity: 'subscription',
- sourceEntityId: ctx.recurlySubscription.uuid,
- target: 'v2',
- targetEntity: 'subscription',
- targetEntityId: ctx.subscription._id,
- createdAt,
- })
- })
- it('should not update subscription when paymentProvider service contains stripe', async function (ctx) {
- ctx.subscription.paymentProvider = {
- service: 'stripe-uk',
- }
- await ctx.SubscriptionUpdater.promises.updateSubscriptionFromRecurly(
- ctx.recurlySubscription,
- ctx.subscription,
- {}
- )
- ctx.subscription.save.called.should.equal(false)
- expect(ctx.FeaturesUpdater.promises.scheduleRefreshFeatures).to.not.have
- .been.called
- })
- it('should remove the subscription when expired', async function (ctx) {
- ctx.recurlySubscription.state = 'expired'
- await ctx.SubscriptionUpdater.promises.updateSubscriptionFromRecurly(
- ctx.recurlySubscription,
- ctx.subscription,
- {}
- )
- ctx.SubscriptionModel.deleteOne.should.have.been.calledWith({
- _id: ctx.subscription._id,
- })
- })
- it('should not remove the subscription when expired if it has Managed Users enabled', async function (ctx) {
- ctx.Features.hasFeature.withArgs('saas').returns(true)
- ctx.subscription.managedUsersEnabled = true
- ctx.recurlySubscription.state = 'expired'
- await ctx.SubscriptionUpdater.promises.updateSubscriptionFromRecurly(
- ctx.recurlySubscription,
- ctx.subscription,
- {}
- )
- ctx.SubscriptionModel.deleteOne.should.not.have.been.called
- })
- it('should not remove the subscription when expired if it has Group SSO enabled', async function (ctx) {
- ctx.Features.hasFeature.withArgs('saas').returns(true)
- ctx.subscription.ssoConfig = new ObjectId('abc123abc123abc123abc123')
- ctx.recurlySubscription.state = 'expired'
- await ctx.SubscriptionUpdater.promises.updateSubscriptionFromRecurly(
- ctx.recurlySubscription,
- ctx.subscription,
- {}
- )
- ctx.SubscriptionModel.deleteOne.should.not.have.been.called
- })
- it('should update all the users features', async function (ctx) {
- ctx.subscription.member_ids = ctx.allUserIds
- await ctx.SubscriptionUpdater.promises.updateSubscriptionFromRecurly(
- ctx.recurlySubscription,
- ctx.subscription,
- {}
- )
- expect(
- ctx.FeaturesUpdater.promises.scheduleRefreshFeatures
- ).to.have.been.calledWith(ctx.adminUser._id)
- expect(
- ctx.FeaturesUpdater.promises.scheduleRefreshFeatures
- ).to.have.been.calledWith(ctx.allUserIds[0])
- expect(
- ctx.FeaturesUpdater.promises.scheduleRefreshFeatures
- ).to.have.been.calledWith(ctx.allUserIds[1])
- expect(
- ctx.FeaturesUpdater.promises.scheduleRefreshFeatures
- ).to.have.been.calledWith(ctx.allUserIds[2])
- })
- it('should set group to true and save how many members can be added to group', async function (ctx) {
- ctx.recurlyPlan.groupPlan = true
- ctx.recurlyPlan.membersLimit = 5
- await ctx.SubscriptionUpdater.promises.updateSubscriptionFromRecurly(
- ctx.recurlySubscription,
- ctx.subscription,
- {}
- )
- ctx.subscription.membersLimit.should.equal(5)
- ctx.subscription.groupPlan.should.equal(true)
- ctx.subscription.member_ids.should.deep.equal([ctx.subscription.admin_id])
- })
- it('should delete and replace subscription when downgrading from group to individual plan', async function (ctx) {
- ctx.recurlyPlan.groupPlan = false
- await ctx.SubscriptionUpdater.promises.updateSubscriptionFromRecurly(
- ctx.recurlySubscription,
- ctx.groupSubscription,
- {}
- )
- })
- it('should not set group to true or set groupPlan', async function (ctx) {
- await ctx.SubscriptionUpdater.promises.updateSubscriptionFromRecurly(
- ctx.recurlySubscription,
- ctx.subscription,
- {}
- )
- assert.notEqual(ctx.subscription.membersLimit, 5)
- assert.notEqual(ctx.subscription.groupPlan, true)
- })
- describe('when the plan allows adding more seats', function () {
- beforeEach(function (ctx) {
- ctx.membersLimitAddOn = 'add_on1'
- ctx.recurlyPlan.groupPlan = true
- ctx.recurlyPlan.membersLimit = 5
- ctx.recurlyPlan.membersLimitAddOn = ctx.membersLimitAddOn
- })
- function expectMembersLimit(limit) {
- it('should set the membersLimit accordingly', async function (ctx) {
- await ctx.SubscriptionUpdater.promises.updateSubscriptionFromRecurly(
- ctx.recurlySubscription,
- ctx.subscription,
- {}
- )
- expect(ctx.subscription.membersLimit).to.equal(limit)
- })
- }
- describe('when the recurlySubscription does not have add ons', function () {
- beforeEach(function (ctx) {
- delete ctx.recurlySubscription.subscription_add_ons
- })
- expectMembersLimit(5)
- })
- describe('when the recurlySubscription has non-matching add ons', function () {
- beforeEach(function (ctx) {
- ctx.recurlySubscription.subscription_add_ons = [
- { add_on_code: 'add_on_99', quantity: 3 },
- ]
- })
- expectMembersLimit(5)
- })
- describe('when the recurlySubscription has a matching add on', function () {
- beforeEach(function (ctx) {
- ctx.recurlySubscription.subscription_add_ons = [
- { add_on_code: ctx.membersLimitAddOn, quantity: 10 },
- ]
- })
- expectMembersLimit(15)
- })
- // NOTE: This is unexpected, but we are going to support it anyways.
- describe('when the recurlySubscription has multiple matching add ons', function () {
- beforeEach(function (ctx) {
- ctx.recurlySubscription.subscription_add_ons = [
- { add_on_code: ctx.membersLimitAddOn, quantity: 10 },
- { add_on_code: ctx.membersLimitAddOn, quantity: 3 },
- ]
- })
- expectMembersLimit(18)
- })
- })
- })
- describe('addUserToGroup', function () {
- it('should add the user ids to the group as a set', async function (ctx) {
- ctx.SubscriptionModel.findOne = sinon
- .stub()
- .resolves(ctx.groupSubscription)
- await ctx.SubscriptionUpdater.promises.addUserToGroup(
- ctx.groupSubscription._id,
- ctx.otherUserId
- )
- const searchOps = { _id: ctx.groupSubscription._id }
- const insertOperation = {
- $addToSet: { member_ids: ctx.otherUserId },
- }
- ctx.SubscriptionModel.updateOne
- .calledWith(searchOps, insertOperation)
- .should.equal(true)
- expect(ctx.SubscriptionModel.updateOne.lastCall.args[2].session).to.exist
- sinon.assert.calledWith(
- ctx.AnalyticsManager.recordEventForUserInBackground,
- ctx.otherUserId,
- 'group-subscription-joined',
- {
- groupId: ctx.groupSubscription._id,
- subscriptionId: ctx.groupSubscription.recurlySubscription_id,
- }
- )
- })
- it('should update the users features', async function (ctx) {
- await ctx.SubscriptionUpdater.promises.addUserToGroup(
- ctx.subscription._id,
- ctx.otherUserId
- )
- ctx.FeaturesUpdater.promises.refreshFeatures
- .calledWith(ctx.otherUserId)
- .should.equal(true)
- })
- it('should set the group plan code user property to the best plan with 1 group subscription', async function (ctx) {
- ctx.SubscriptionLocator.promises.getMemberSubscriptions
- .withArgs(ctx.otherUserId)
- .resolves([ctx.groupSubscription])
- await ctx.SubscriptionUpdater.promises.addUserToGroup(
- ctx.groupSubscription._id,
- ctx.otherUserId
- )
- sinon.assert.calledWith(
- ctx.AnalyticsManager.setUserPropertyForUserInBackground,
- ctx.otherUserId,
- 'group-subscription-plan-code',
- 'group_subscription'
- )
- })
- it('should set the group plan code user property to the best plan with 2 group subscriptions', async function (ctx) {
- ctx.SubscriptionLocator.promises.getMemberSubscriptions
- .withArgs(ctx.otherUserId)
- .resolves([ctx.groupSubscription, ctx.betterGroupSubscription])
- await ctx.SubscriptionUpdater.promises.addUserToGroup(
- ctx.betterGroupSubscription._id,
- ctx.otherUserId
- )
- sinon.assert.calledWith(
- ctx.AnalyticsManager.setUserPropertyForUserInBackground,
- ctx.otherUserId,
- 'group-subscription-plan-code',
- 'better_group_subscription'
- )
- })
- it('should set the group plan code user property to the best plan with 2 group subscriptions in reverse order', async function (ctx) {
- ctx.SubscriptionLocator.promises.getMemberSubscriptions
- .withArgs(ctx.otherUserId)
- .resolves([ctx.betterGroupSubscription, ctx.groupSubscription])
- await ctx.SubscriptionUpdater.promises.addUserToGroup(
- ctx.betterGroupSubscription._id,
- ctx.otherUserId
- )
- sinon.assert.calledWith(
- ctx.AnalyticsManager.setUserPropertyForUserInBackground,
- ctx.otherUserId,
- 'group-subscription-plan-code',
- 'better_group_subscription'
- )
- })
- it('should add an entry to the user audit log when joining a group', async function (ctx) {
- await ctx.SubscriptionUpdater.promises.addUserToGroup(
- ctx.subscription._id,
- ctx.otherUserId
- )
- sinon.assert.calledWith(
- ctx.UserAuditLogHandler.promises.addEntry,
- ctx.otherUserId,
- 'join-group-subscription',
- undefined,
- undefined,
- {
- subscriptionId: ctx.subscription._id,
- }
- )
- })
- it('should add an entry to the group audit log when joining a group', async function (ctx) {
- await ctx.SubscriptionUpdater.promises.addUserToGroup(
- ctx.subscription._id,
- ctx.otherUserId,
- { ipAddress: '0:0:0:0', initiatorId: 'user123' }
- )
- expect(ctx.Modules.promises.hooks.fire).to.have.been.calledWith(
- 'addGroupAuditLogEntry',
- {
- groupId: ctx.subscription._id,
- initiatorId: 'user123',
- ipAddress: '0:0:0:0',
- operation: 'join-group',
- }
- )
- })
- })
- describe('removeUserFromGroup', function () {
- beforeEach(function (ctx) {
- ctx.fakeSubscriptions = [
- {
- _id: 'fake-id-1',
- },
- {
- _id: 'fake-id-2',
- },
- ]
- ctx.SubscriptionModel.findOne.resolves(ctx.groupSubscription)
- ctx.SubscriptionModel.findById = sinon
- .stub()
- .resolves(ctx.groupSubscription)
- ctx.SubscriptionLocator.promises.getMemberSubscriptions.resolves(
- ctx.fakeSubscriptions
- )
- })
- it('should pull the users id from the group', async function (ctx) {
- await ctx.SubscriptionUpdater.promises.removeUserFromGroup(
- ctx.subscription._id,
- ctx.otherUserId
- )
- const removeOperation = { $pull: { member_ids: ctx.otherUserId } }
- ctx.SubscriptionModel.updateOne
- .calledWith({ _id: ctx.subscription._id }, removeOperation)
- .should.equal(true)
- })
- it('should remove user enrollment if the group is managed', async function (ctx) {
- ctx.SubscriptionModel.findById.resolves({
- ...ctx.groupSubscription,
- managedUsersEnabled: true,
- })
- await ctx.SubscriptionUpdater.promises.removeUserFromGroup(
- ctx.groupSubscription._id,
- ctx.otherUserId
- )
- ctx.UserUpdater.promises.updateUser
- .calledWith(
- { _id: ctx.otherUserId },
- {
- $unset: {
- 'enrollment.managedBy': 1,
- 'enrollment.enrolledAt': 1,
- },
- }
- )
- .should.equal(true)
- })
- it('should send a group-subscription-left event', async function (ctx) {
- await ctx.SubscriptionUpdater.promises.removeUserFromGroup(
- ctx.groupSubscription._id,
- ctx.otherUserId
- )
- sinon.assert.calledWith(
- ctx.AnalyticsManager.recordEventForUserInBackground,
- ctx.otherUserId,
- 'group-subscription-left',
- {
- groupId: ctx.groupSubscription._id,
- subscriptionId: ctx.groupSubscription.recurlySubscription_id,
- }
- )
- })
- it('should set the group plan code user property when removing user from group', async function (ctx) {
- await ctx.SubscriptionUpdater.promises.removeUserFromGroup(
- ctx.subscription._id,
- ctx.otherUserId
- )
- sinon.assert.calledWith(
- ctx.AnalyticsManager.setUserPropertyForUserInBackground,
- ctx.otherUserId,
- 'group-subscription-plan-code',
- null
- )
- })
- it('should update the users features', async function (ctx) {
- await ctx.SubscriptionUpdater.promises.removeUserFromGroup(
- ctx.subscription._id,
- ctx.otherUserId
- )
- ctx.FeaturesUpdater.promises.refreshFeatures
- .calledWith(ctx.otherUserId)
- .should.equal(true)
- })
- it('should add an audit log when a user leaves a group', async function (ctx) {
- await ctx.SubscriptionUpdater.promises.removeUserFromGroup(
- ctx.subscription._id,
- ctx.otherUserId
- )
- sinon.assert.calledWith(
- ctx.UserAuditLogHandler.promises.addEntry,
- ctx.otherUserId,
- 'leave-group-subscription',
- undefined,
- undefined,
- {
- subscriptionId: ctx.subscription._id,
- }
- )
- })
- })
- describe('removeUserFromAllGroups', function () {
- beforeEach(function (ctx) {
- ctx.SubscriptionLocator.promises.getMemberSubscriptions.resolves([
- {
- _id: 'fake-id-1',
- },
- {
- _id: 'fake-id-2',
- },
- ])
- })
- it('should set the group plan code user property when removing user from all groups', async function (ctx) {
- await ctx.SubscriptionUpdater.promises.removeUserFromAllGroups(
- ctx.otherUserId
- )
- sinon.assert.calledWith(
- ctx.AnalyticsManager.setUserPropertyForUserInBackground,
- ctx.otherUserId,
- 'group-subscription-plan-code',
- null
- )
- })
- it('should pull the users id from all groups', async function (ctx) {
- await ctx.SubscriptionUpdater.promises.removeUserFromAllGroups(
- ctx.otherUserId
- )
- const filter = { _id: ['fake-id-1', 'fake-id-2'] }
- const removeOperation = { $pull: { member_ids: ctx.otherUserId } }
- sinon.assert.calledWith(
- ctx.SubscriptionModel.updateMany,
- filter,
- removeOperation
- )
- })
- it('should send a group-subscription-left event for each group', async function (ctx) {
- ctx.fakeSub1 = {
- _id: 'fake-id-1',
- groupPlan: true,
- recurlySubscription_id: 'fake-sub-1',
- }
- ctx.fakeSub2 = {
- _id: 'fake-id-2',
- groupPlan: true,
- recurlySubscription_id: 'fake-sub-2',
- }
- ctx.SubscriptionModel.findOne
- .withArgs(
- { _id: 'fake-id-1' },
- { recurlySubscription_id: 1, groupPlan: 1 }
- )
- .resolves(ctx.fakeSub1)
- .withArgs(
- { _id: 'fake-id-2' },
- { recurlySubscription_id: 1, groupPlan: 1 }
- )
- .resolves(ctx.fakeSub2)
- await ctx.SubscriptionUpdater.promises.removeUserFromAllGroups(
- ctx.otherUserId
- )
- sinon.assert.calledWith(
- ctx.AnalyticsManager.recordEventForUserInBackground,
- ctx.otherUserId,
- 'group-subscription-left',
- {
- groupId: 'fake-id-1',
- subscriptionId: 'fake-sub-1',
- }
- )
- sinon.assert.calledWith(
- ctx.AnalyticsManager.recordEventForUserInBackground,
- ctx.otherUserId,
- 'group-subscription-left',
- {
- groupId: 'fake-id-2',
- subscriptionId: 'fake-sub-2',
- }
- )
- })
- it('should add an audit log entry for each group the user leaves', async function (ctx) {
- await ctx.SubscriptionUpdater.promises.removeUserFromAllGroups(
- ctx.otherUserId
- )
- sinon.assert.calledWith(
- ctx.UserAuditLogHandler.promises.addEntry,
- ctx.otherUserId,
- 'leave-group-subscription',
- undefined,
- undefined,
- {
- subscriptionId: 'fake-id-1',
- }
- )
- sinon.assert.calledWith(
- ctx.UserAuditLogHandler.promises.addEntry,
- ctx.otherUserId,
- 'leave-group-subscription',
- undefined,
- undefined,
- {
- subscriptionId: 'fake-id-2',
- }
- )
- })
- })
- describe('deleteSubscription', function () {
- beforeEach(async function (ctx) {
- ctx.subscription = {
- _id: new ObjectId().toString(),
- mock: 'subscription',
- admin_id: new ObjectId(),
- member_ids: [new ObjectId(), new ObjectId(), new ObjectId()],
- }
- await ctx.SubscriptionUpdater.promises.deleteSubscription(
- ctx.subscription,
- {}
- )
- })
- it('should remove the subscription', function (ctx) {
- ctx.SubscriptionModel.deleteOne
- .calledWith({ _id: ctx.subscription._id })
- .should.equal(true)
- })
- it('should downgrade the admin_id', function (ctx) {
- expect(
- ctx.FeaturesUpdater.promises.scheduleRefreshFeatures
- ).to.have.been.calledWith(ctx.subscription.admin_id)
- })
- it('should downgrade all of the members', function (ctx) {
- for (const userId of ctx.subscription.member_ids) {
- expect(
- ctx.FeaturesUpdater.promises.scheduleRefreshFeatures
- ).to.have.been.calledWith(userId)
- }
- })
- })
- describe('scheduleRefreshFeatures', function () {
- it('should call upgrades feature for personal subscription from admin_id', async function (ctx) {
- ctx.subscription = {
- _id: new ObjectId().toString(),
- mock: 'subscription',
- admin_id: new ObjectId(),
- }
- await ctx.SubscriptionUpdater.promises.scheduleRefreshFeatures(
- ctx.subscription
- )
- expect(
- ctx.FeaturesUpdater.promises.scheduleRefreshFeatures
- ).to.have.been.calledOnceWith(ctx.subscription.admin_id)
- })
- it('should call upgrades feature for group subscription from admin_id and member_ids', async function (ctx) {
- ctx.subscription = {
- _id: new ObjectId().toString(),
- mock: 'subscription',
- admin_id: new ObjectId(),
- member_ids: [new ObjectId(), new ObjectId(), new ObjectId()],
- }
- await ctx.SubscriptionUpdater.promises.scheduleRefreshFeatures(
- ctx.subscription
- )
- expect(
- ctx.FeaturesUpdater.promises.scheduleRefreshFeatures.callCount
- ).to.equal(4)
- })
- })
- })
|