| 123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210 |
- 'use strict'
- const { expect } = require('chai')
- const sinon = require('sinon')
- const {
- Chunk,
- Snapshot,
- History,
- File,
- AddFileOperation,
- Change,
- } = require('overleaf-editor-core')
- const cleanup = require('./support/cleanup')
- const fixtures = require('./support/fixtures')
- const chunkBuffer = require('../../../../storage/lib/chunk_buffer')
- const chunkStore = require('../../../../storage/lib/chunk_store')
- const redisBackend = require('../../../../storage/lib/chunk_store/redis')
- const metrics = require('@overleaf/metrics')
- describe('chunk buffer', function () {
- beforeEach(cleanup.everything)
- beforeEach(fixtures.create)
- beforeEach(function () {
- sinon.spy(metrics, 'inc')
- })
- afterEach(function () {
- metrics.inc.restore()
- })
- const projectId = '123456'
- describe('loadLatest', function () {
- // Initialize project and create a test chunk
- beforeEach(async function () {
- // Initialize project in chunk store
- await chunkStore.initializeProject(projectId)
- })
- describe('with an existing chunk', function () {
- beforeEach(async function () {
- // Create a sample chunk with some content
- const snapshot = new Snapshot()
- const changes = [
- new Change(
- [new AddFileOperation('test.tex', File.fromString('Hello World'))],
- new Date(),
- []
- ),
- ]
- const history = new History(snapshot, changes)
- const chunk = new Chunk(history, 1) // startVersion 1
- // Store the chunk directly in the chunk store using create method
- // which internally calls uploadChunk
- await chunkStore.create(projectId, chunk)
- // Clear any existing cache
- await redisBackend.clearCache(projectId)
- })
- it('should load from chunk store and update cache on first access (cache miss)', async function () {
- // First access should load from chunk store and populate cache
- const firstResult = await chunkBuffer.loadLatest(projectId)
- // Verify the chunk is correct
- expect(firstResult).to.not.be.null
- expect(firstResult.getStartVersion()).to.equal(1)
- expect(firstResult.getEndVersion()).to.equal(2)
- // Verify that we got a cache miss metric
- expect(
- metrics.inc.calledWith('chunk_buffer.loadLatest', 1, {
- status: 'cache-miss',
- })
- ).to.be.true
- // Reset the metrics spy
- metrics.inc.resetHistory()
- // Second access should hit the cache
- const secondResult = await chunkBuffer.loadLatest(projectId)
- // Verify we got the same chunk
- expect(secondResult).to.not.be.null
- expect(secondResult.getStartVersion()).to.equal(1)
- expect(secondResult.getEndVersion()).to.equal(2)
- // Verify that we got a cache hit metric
- expect(
- metrics.inc.calledWith('chunk_buffer.loadLatest', 1, {
- status: 'cache-hit',
- })
- ).to.be.true
- // Verify both chunks are equivalent
- expect(secondResult.getStartVersion()).to.equal(
- firstResult.getStartVersion()
- )
- expect(secondResult.getEndVersion()).to.equal(
- firstResult.getEndVersion()
- )
- })
- it('should refresh the cache when chunk changes in the store', async function () {
- // First access to load into cache
- const firstResult = await chunkBuffer.loadLatest(projectId)
- expect(firstResult.getStartVersion()).to.equal(1)
- // Reset metrics spy
- metrics.inc.resetHistory()
- // Create a new chunk with different content
- const newSnapshot = new Snapshot()
- const newChanges = [
- new Change(
- [
- new AddFileOperation(
- 'updated.tex',
- File.fromString('Updated content')
- ),
- ],
- new Date(),
- []
- ),
- ]
- const newHistory = new History(newSnapshot, newChanges)
- const newChunk = new Chunk(newHistory, 2) // Different start version
- // Store the new chunk directly in the chunk store
- await chunkStore.create(projectId, newChunk)
- // Access again - should detect the change and refresh cache
- const secondResult = await chunkBuffer.loadLatest(projectId)
- // Verify we got the updated chunk
- expect(secondResult.getStartVersion()).to.equal(2)
- expect(secondResult.getEndVersion()).to.equal(3)
- // Verify that we got a cache miss metric (since the cached chunk was invalidated)
- expect(
- metrics.inc.calledWith('chunk_buffer.loadLatest', 1, {
- status: 'cache-miss',
- })
- ).to.be.true
- })
- it('should continue using cache when chunk in store has not changed', async function () {
- // First access to load into cache
- await chunkBuffer.loadLatest(projectId)
- // Reset metrics spy
- metrics.inc.resetHistory()
- // Access again without changing the underlying chunk
- const result = await chunkBuffer.loadLatest(projectId)
- // Verify we got the same chunk
- expect(result.getStartVersion()).to.equal(1)
- expect(result.getEndVersion()).to.equal(2)
- // Verify that we got a cache hit metric
- expect(
- metrics.inc.calledWith('chunk_buffer.loadLatest', 1, {
- status: 'cache-hit',
- })
- ).to.be.true
- })
- })
- describe('with an empty project', function () {
- it('should handle a case with empty chunks (no changes)', async function () {
- // Clear the cache
- await redisBackend.clearCache(projectId)
- // Load the initial empty chunk via buffer
- const result = await chunkBuffer.loadLatest(projectId)
- // Verify we got the empty chunk
- expect(result.getStartVersion()).to.equal(0)
- expect(result.getEndVersion()).to.equal(0) // Start equals end for empty chunks
- expect(result.history.changes.length).to.equal(0)
- // Verify cache miss metric
- expect(
- metrics.inc.calledWith('chunk_buffer.loadLatest', 1, {
- status: 'cache-miss',
- })
- ).to.be.true
- // Reset metrics
- metrics.inc.resetHistory()
- // Second access should hit the cache
- const secondResult = await chunkBuffer.loadLatest(projectId)
- // Verify we got the same empty chunk
- expect(secondResult.getStartVersion()).to.equal(0)
- expect(secondResult.getEndVersion()).to.equal(0)
- expect(secondResult.history.changes.length).to.equal(0)
- // Verify cache hit metric
- expect(
- metrics.inc.calledWith('chunk_buffer.loadLatest', 1, {
- status: 'cache-hit',
- })
- ).to.be.true
- })
- })
- })
- })
|