| 123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287 |
- /* eslint-disable
- no-return-assign,
- */
- // TODO: This file was created by bulk-decaffeinate.
- // Fix any style issues and re-enable lint.
- /*
- * decaffeinate suggestions:
- * DS101: Remove unnecessary use of Array.from
- * DS102: Remove unnecessary code created because of implicit returns
- * Full docs: https://github.com/decaffeinate/decaffeinate/blob/master/docs/suggestions.md
- */
- const async = require('async')
- const { expect } = require('chai')
- const RealTimeClient = require('./helpers/RealTimeClient')
- const MockDocUpdaterServer = require('./helpers/MockDocUpdaterServer')
- const MockWebServer = require('./helpers/MockWebServer')
- const FixturesManager = require('./helpers/FixturesManager')
- const settings = require('@overleaf/settings')
- const redis = require('@overleaf/redis-wrapper')
- const rclient = redis.createClient(settings.redis.pubsub)
- const rclientRT = redis.createClient(settings.redis.realtime)
- const KeysRT = settings.redis.realtime.key_schema
- describe('EarlyDisconnect', function () {
- before(function (done) {
- return MockDocUpdaterServer.run(done)
- })
- describe('when the client disconnects before joinProject completes', function () {
- before(function () {
- // slow down web-api requests to force the race condition
- let joinProject
- this.actualWebAPIjoinProject = joinProject = MockWebServer.joinProject
- return (MockWebServer.joinProject = (projectId, userId, cb) =>
- setTimeout(() => joinProject(projectId, userId, cb), 300))
- })
- after(function () {
- return (MockWebServer.joinProject = this.actualWebAPIjoinProject)
- })
- beforeEach(function (done) {
- return async.series(
- [
- cb => {
- return FixturesManager.setUpProject(
- {
- privilegeLevel: 'owner',
- project: {
- name: 'Test Project',
- },
- },
- (e, { project_id: projectId, user_id: userId }) => {
- this.project_id = projectId
- this.user_id = userId
- return cb()
- }
- )
- },
- cb => {
- this.clientA = RealTimeClient.connect()
- return this.clientA.on('connectionAccepted', cb)
- },
- cb => {
- this.clientA.emit(
- 'joinProject',
- { project_id: this.project_id },
- () => {}
- )
- // disconnect before joinProject completes
- this.clientA.on('disconnect', () => cb())
- return this.clientA.disconnect()
- },
- cb => {
- // wait for joinDoc and subscribe
- return setTimeout(cb, 500)
- },
- ],
- done
- )
- })
- // we can force the race condition, there is no need to repeat too often
- return Array.from(Array.from({ length: 5 }).map((_, i) => i + 1)).map(
- attempt =>
- it(`should not subscribe to the pub/sub channel anymore (race ${attempt})`, function (done) {
- rclient.pubsub('CHANNELS', (err, resp) => {
- if (err) {
- return done(err)
- }
- expect(resp).to.not.include(`editor-events:${this.project_id}`)
- return done()
- })
- return null
- })
- )
- })
- describe('when the client disconnects before joinDoc completes', function () {
- beforeEach(function (done) {
- return async.series(
- [
- cb => {
- return FixturesManager.setUpProject(
- {
- privilegeLevel: 'owner',
- project: {
- name: 'Test Project',
- },
- },
- (e, { project_id: projectId, user_id: userId }) => {
- this.project_id = projectId
- this.user_id = userId
- return cb()
- }
- )
- },
- cb => {
- this.clientA = RealTimeClient.connect()
- return this.clientA.on('connectionAccepted', cb)
- },
- cb => {
- return this.clientA.emit(
- 'joinProject',
- { project_id: this.project_id },
- (error, project, privilegeLevel, protocolVersion) => {
- this.project = project
- this.privilegeLevel = privilegeLevel
- this.protocolVersion = protocolVersion
- return cb(error)
- }
- )
- },
- cb => {
- return FixturesManager.setUpDoc(
- this.project_id,
- { lines: this.lines, version: this.version, ops: this.ops },
- (e, { doc_id: docId }) => {
- this.doc_id = docId
- return cb(e)
- }
- )
- },
- cb => {
- this.clientA.emit('joinDoc', this.doc_id, () => {})
- // disconnect before joinDoc completes
- this.clientA.on('disconnect', () => cb())
- return this.clientA.disconnect()
- },
- cb => {
- // wait for subscribe and unsubscribe
- return setTimeout(cb, 100)
- },
- ],
- done
- )
- })
- // we can not force the race condition, so we have to try many times
- return Array.from(Array.from({ length: 20 }).map((_, i) => i + 1)).map(
- attempt =>
- it(`should not subscribe to the pub/sub channels anymore (race ${attempt})`, function (done) {
- rclient.pubsub('CHANNELS', (err, resp) => {
- if (err) {
- return done(err)
- }
- expect(resp).to.not.include(`editor-events:${this.project_id}`)
- return rclient.pubsub('CHANNELS', (err, resp) => {
- if (err) {
- return done(err)
- }
- expect(resp).to.not.include(`applied-ops:${this.doc_id}`)
- return done()
- })
- })
- return null
- })
- )
- })
- return describe('when the client disconnects before clientTracking.updatePosition starts', function () {
- beforeEach(function (done) {
- return async.series(
- [
- cb => {
- return FixturesManager.setUpProject(
- {
- privilegeLevel: 'owner',
- project: {
- name: 'Test Project',
- },
- },
- (e, { project_id: projectId, user_id: userId }) => {
- this.project_id = projectId
- this.user_id = userId
- return cb()
- }
- )
- },
- cb => {
- this.clientA = RealTimeClient.connect()
- return this.clientA.on('connectionAccepted', cb)
- },
- cb => {
- return this.clientA.emit(
- 'joinProject',
- { project_id: this.project_id },
- (error, project, privilegeLevel, protocolVersion) => {
- this.project = project
- this.privilegeLevel = privilegeLevel
- this.protocolVersion = protocolVersion
- return cb(error)
- }
- )
- },
- cb => {
- return FixturesManager.setUpDoc(
- this.project_id,
- { lines: this.lines, version: this.version, ops: this.ops },
- (e, { doc_id: docId }) => {
- this.doc_id = docId
- return cb(e)
- }
- )
- },
- cb => {
- return this.clientA.emit('joinDoc', this.doc_id, cb)
- },
- cb => {
- this.clientA.emit(
- 'clientTracking.updatePosition',
- {
- row: 42,
- column: 36,
- doc_id: this.doc_id,
- },
- () => {}
- )
- // disconnect before updateClientPosition completes
- this.clientA.on('disconnect', () => cb())
- return this.clientA.disconnect()
- },
- cb => {
- // wait for updateClientPosition
- return setTimeout(cb, 100)
- },
- ],
- done
- )
- })
- // we can not force the race condition, so we have to try many times
- return Array.from(Array.from({ length: 20 }).map((_, i) => i + 1)).map(
- attempt =>
- it(`should not show the client as connected (race ${attempt})`, function (done) {
- rclientRT.smembers(
- KeysRT.clientsInProject({ project_id: this.project_id }),
- (err, results) => {
- if (err) {
- return done(err)
- }
- expect(results).to.deep.equal([])
- return done()
- }
- )
- return null
- })
- )
- })
- })
|