Browse Source

[libraries] align test setup (#29009)

* [libraries] move all the unit tests into test/unit

* [settings] remove empty test file

* [mongo-utils] remove test setup as there are no tests

* [libraries] ensure that all libraries with tests have test/setup.js

* [monorepo] remove test step from Cloud build for libraries without tests

GitOrigin-RevId: b9a0a8a3330db7d8646c7a272b03fbb8c5e44bc0
Jakob Ackermann 10 tháng trước cách đây
mục cha
commit
0ca3813a00
41 tập tin đã thay đổi với 75 bổ sung81 xóa
  1. 1 0
      libraries/metrics/test/setup.js
  2. 0 11
      libraries/mongo-utils/test/setup.js
  3. 1 0
      libraries/o-error/test/setup.js
  4. 1 1
      libraries/o-error/test/unit/o-error-util.test.js
  5. 1 1
      libraries/o-error/test/unit/o-error.test.js
  6. 1 1
      libraries/o-error/test/unit/support/index.js
  7. 1 1
      libraries/object-persistor/.mocharc.json
  8. 0 0
      libraries/object-persistor/test/setup.js
  9. 1 0
      libraries/overleaf-editor-core/test/setup.js
  10. 3 3
      libraries/overleaf-editor-core/test/unit/add_comment_operation.test.js
  11. 1 1
      libraries/overleaf-editor-core/test/unit/change.test.js
  12. 2 2
      libraries/overleaf-editor-core/test/unit/comment.test.js
  13. 3 3
      libraries/overleaf-editor-core/test/unit/comments_list.test.js
  14. 4 4
      libraries/overleaf-editor-core/test/unit/delete_comment_operation.test.js
  15. 3 3
      libraries/overleaf-editor-core/test/unit/doc_updater_compatible_ranges.test.js
  16. 2 2
      libraries/overleaf-editor-core/test/unit/edit_file_operation.test.js
  17. 9 9
      libraries/overleaf-editor-core/test/unit/edit_operation.test.js
  18. 1 1
      libraries/overleaf-editor-core/test/unit/file.test.js
  19. 1 1
      libraries/overleaf-editor-core/test/unit/file_map.test.js
  20. 3 3
      libraries/overleaf-editor-core/test/unit/hash_file_data.test.js
  21. 1 1
      libraries/overleaf-editor-core/test/unit/history.test.js
  22. 2 2
      libraries/overleaf-editor-core/test/unit/hollow_string_file_data.test.js
  23. 1 1
      libraries/overleaf-editor-core/test/unit/label.test.js
  24. 3 3
      libraries/overleaf-editor-core/test/unit/lazy_string_file_data.test.js
  25. 1 1
      libraries/overleaf-editor-core/test/unit/move_file_operation.test.js
  26. 2 2
      libraries/overleaf-editor-core/test/unit/operation.test.js
  27. 1 1
      libraries/overleaf-editor-core/test/unit/range.test.js
  28. 1 1
      libraries/overleaf-editor-core/test/unit/safe_pathname.test.js
  29. 3 3
      libraries/overleaf-editor-core/test/unit/scan_op.test.js
  30. 1 1
      libraries/overleaf-editor-core/test/unit/snapshot.test.js
  31. 2 2
      libraries/overleaf-editor-core/test/unit/string_file_data.test.js
  32. 0 0
      libraries/overleaf-editor-core/test/unit/support/fake_blob_store.js
  33. 0 0
      libraries/overleaf-editor-core/test/unit/support/random.js
  34. 3 3
      libraries/overleaf-editor-core/test/unit/support/random_text_operation.js
  35. 5 5
      libraries/overleaf-editor-core/test/unit/text_operation.test.js
  36. 3 3
      libraries/overleaf-editor-core/test/unit/tracked_change.test.js
  37. 4 4
      libraries/overleaf-editor-core/test/unit/tracked_change_list.test.js
  38. 1 0
      libraries/ranges-tracker/test/setup.js
  39. 0 1
      libraries/settings/test/notests.js
  40. 1 0
      libraries/stream-utils/test/setup.js
  41. 1 0
      libraries/validation-tools/test/setup.js

+ 1 - 0
libraries/metrics/test/setup.js

@@ -0,0 +1 @@
+// no setup yet

+ 0 - 11
libraries/mongo-utils/test/setup.js

@@ -1,11 +0,0 @@
-const chai = require('chai')
-const sinonChai = require('sinon-chai')
-const SandboxedModule = require('sandboxed-module')
-
-// Chai configuration
-chai.should()
-chai.use(sinonChai)
-
-SandboxedModule.configure({
-  globals: { Buffer, JSON, console, process },
-})

+ 1 - 0
libraries/o-error/test/setup.js

@@ -0,0 +1 @@
+// no setup yet

+ 1 - 1
libraries/o-error/test/o-error-util.test.js → libraries/o-error/test/unit/o-error-util.test.js

@@ -1,7 +1,7 @@
 const { expect } = require('chai')
 const { promisify } = require('node:util')
 
-const OError = require('..')
+const OError = require('../..')
 
 const {
   expectError,

+ 1 - 1
libraries/o-error/test/o-error.test.js → libraries/o-error/test/unit/o-error.test.js

@@ -1,6 +1,6 @@
 const { expect } = require('chai')
 
-const OError = require('..')
+const OError = require('../..')
 const {
   expectError,
   expectFullStackWithoutStackFramesToEqual,

+ 1 - 1
libraries/o-error/test/support/index.js → libraries/o-error/test/unit/support/index.js

@@ -1,6 +1,6 @@
 const { expect } = require('chai')
 
-const OError = require('../..')
+const OError = require('../../..')
 
 /**
  * @param {Error} e

+ 1 - 1
libraries/object-persistor/.mocharc.json

@@ -2,5 +2,5 @@
   "ui": "bdd",
   "recursive": "true",
   "reporter": "spec",
-  "require": "./test/Init"
+  "require": "./test/setup"
 }

+ 0 - 0
libraries/object-persistor/test/Init.js → libraries/object-persistor/test/setup.js


+ 1 - 0
libraries/overleaf-editor-core/test/setup.js

@@ -0,0 +1 @@
+// no setup yet

+ 3 - 3
libraries/overleaf-editor-core/test/add_comment_operation.test.js → libraries/overleaf-editor-core/test/unit/add_comment_operation.test.js

@@ -1,8 +1,8 @@
 // @ts-check
 const { expect } = require('chai')
-const { AddCommentOperation, DeleteCommentOperation } = require('..')
-const Range = require('../lib/range')
-const StringFileData = require('../lib/file_data/string_file_data')
+const { AddCommentOperation, DeleteCommentOperation } = require('../..')
+const Range = require('../../lib/range')
+const StringFileData = require('../../lib/file_data/string_file_data')
 
 describe('AddCommentOperation', function () {
   it('constructs an AddCommentOperation fromJSON', function () {

+ 1 - 1
libraries/overleaf-editor-core/test/change.test.js → libraries/overleaf-editor-core/test/unit/change.test.js

@@ -9,7 +9,7 @@ const {
   Snapshot,
   Origin,
   RestoreFileOrigin,
-} = require('..')
+} = require('../..')
 
 describe('Change', function () {
   describe('findBlobHashes', function () {

+ 2 - 2
libraries/overleaf-editor-core/test/comment.test.js → libraries/overleaf-editor-core/test/unit/comment.test.js

@@ -2,8 +2,8 @@
 'use strict'
 
 const { expect } = require('chai')
-const Comment = require('../lib/comment')
-const Range = require('../lib/range')
+const Comment = require('../../lib/comment')
+const Range = require('../../lib/range')
 
 describe('Comment', function () {
   it('should move ranges to the right of insert', function () {

+ 3 - 3
libraries/overleaf-editor-core/test/comments_list.test.js → libraries/overleaf-editor-core/test/unit/comments_list.test.js

@@ -2,9 +2,9 @@
 'use strict'
 
 const { expect } = require('chai')
-const CommentList = require('../lib/file_data/comment_list')
-const Comment = require('../lib/comment')
-const Range = require('../lib/range')
+const CommentList = require('../../lib/file_data/comment_list')
+const Comment = require('../../lib/comment')
+const Range = require('../../lib/range')
 
 describe('commentList', function () {
   it('checks if toRaw() returns a correct comment list', function () {

+ 4 - 4
libraries/overleaf-editor-core/test/delete_comment_operation.test.js → libraries/overleaf-editor-core/test/unit/delete_comment_operation.test.js

@@ -1,9 +1,9 @@
 // @ts-check
 const { expect } = require('chai')
-const { AddCommentOperation, DeleteCommentOperation } = require('..')
-const Comment = require('../lib/comment')
-const StringFileData = require('../lib/file_data/string_file_data')
-const Range = require('../lib/range')
+const { AddCommentOperation, DeleteCommentOperation } = require('../..')
+const Comment = require('../../lib/comment')
+const StringFileData = require('../../lib/file_data/string_file_data')
+const Range = require('../../lib/range')
 
 describe('DeleteCommentOperation', function () {
   it('constructs an DeleteCommentOperation fromJSON', function () {

+ 3 - 3
libraries/overleaf-editor-core/test/doc_updater_compatible_ranges.test.js → libraries/overleaf-editor-core/test/unit/doc_updater_compatible_ranges.test.js

@@ -3,9 +3,9 @@
 const { expect } = require('chai')
 const {
   getDocUpdaterCompatibleRanges,
-} = require('../lib/doc_updater_compatible_ranges.js')
-const StringFileData = require('../lib/file_data/string_file_data.js')
-const File = require('../lib/file.js')
+} = require('../../lib/doc_updater_compatible_ranges.js')
+const StringFileData = require('../../lib/file_data/string_file_data.js')
+const File = require('../../lib/file.js')
 
 describe('getDocUpdaterCompatibleRanges', function () {
   describe('with tracked deletes', function () {

+ 2 - 2
libraries/overleaf-editor-core/test/edit_file_operation.test.js → libraries/overleaf-editor-core/test/unit/edit_file_operation.test.js

@@ -3,8 +3,8 @@
 
 const { expect } = require('chai')
 
-const ot = require('..')
-const EditOperationBuilder = require('../lib/operation/edit_operation_builder')
+const ot = require('../..')
+const EditOperationBuilder = require('../../lib/operation/edit_operation_builder')
 const File = ot.File
 const Operation = ot.Operation
 

+ 9 - 9
libraries/overleaf-editor-core/test/edit_operation.test.js → libraries/overleaf-editor-core/test/unit/edit_operation.test.js

@@ -1,15 +1,15 @@
 const { expect } = require('chai')
-const EditOperationBuilder = require('../lib/operation/edit_operation_builder')
-const TextOperation = require('../lib/operation/text_operation')
-const EditOperationTransformer = require('../lib/operation/edit_operation_transformer')
-const EditOperation = require('../lib/operation/edit_operation')
+const EditOperationBuilder = require('../../lib/operation/edit_operation_builder')
+const TextOperation = require('../../lib/operation/text_operation')
+const EditOperationTransformer = require('../../lib/operation/edit_operation_transformer')
+const EditOperation = require('../../lib/operation/edit_operation')
 const randomTextOperation = require('./support/random_text_operation')
 const random = require('./support/random')
-const AddCommentOperation = require('../lib/operation/add_comment_operation')
-const DeleteCommentOperation = require('../lib/operation/delete_comment_operation')
-const SetCommentStateOperation = require('../lib/operation/set_comment_state_operation')
-const Range = require('../lib/range')
-const EditNoOperation = require('../lib/operation/edit_no_operation')
+const AddCommentOperation = require('../../lib/operation/add_comment_operation')
+const DeleteCommentOperation = require('../../lib/operation/delete_comment_operation')
+const SetCommentStateOperation = require('../../lib/operation/set_comment_state_operation')
+const Range = require('../../lib/range')
+const EditNoOperation = require('../../lib/operation/edit_no_operation')
 
 describe('EditOperation', function () {
   it('Cannot be instantiated', function () {

+ 1 - 1
libraries/overleaf-editor-core/test/file.test.js → libraries/overleaf-editor-core/test/unit/file.test.js

@@ -2,7 +2,7 @@
 
 const { expect } = require('chai')
 const FakeBlobStore = require('./support/fake_blob_store')
-const ot = require('..')
+const ot = require('../..')
 const File = ot.File
 
 describe('File', function () {

+ 1 - 1
libraries/overleaf-editor-core/test/file_map.test.js → libraries/overleaf-editor-core/test/unit/file_map.test.js

@@ -3,7 +3,7 @@
 const { expect } = require('chai')
 const _ = require('lodash')
 
-const ot = require('..')
+const ot = require('../..')
 const File = ot.File
 const FileMap = ot.FileMap
 

+ 3 - 3
libraries/overleaf-editor-core/test/hash_file_data.test.js → libraries/overleaf-editor-core/test/unit/hash_file_data.test.js

@@ -1,8 +1,8 @@
-const HashFileData = require('../lib/file_data/hash_file_data')
+const HashFileData = require('../../lib/file_data/hash_file_data')
 const { expect } = require('chai')
-const StringFileData = require('../lib/file_data/string_file_data')
+const StringFileData = require('../../lib/file_data/string_file_data')
 const sinon = require('sinon')
-const Blob = require('../lib/blob')
+const Blob = require('../../lib/blob')
 
 describe('HashFileData', function () {
   beforeEach(function () {

+ 1 - 1
libraries/overleaf-editor-core/test/history.test.js → libraries/overleaf-editor-core/test/unit/history.test.js

@@ -1,7 +1,7 @@
 'use strict'
 
 const { expect } = require('chai')
-const core = require('..')
+const core = require('../..')
 const Change = core.Change
 const File = core.File
 const History = core.History

+ 2 - 2
libraries/overleaf-editor-core/test/hollow_string_file_data.test.js → libraries/overleaf-editor-core/test/unit/hollow_string_file_data.test.js

@@ -1,8 +1,8 @@
 'use strict'
 
 const { expect } = require('chai')
-const ot = require('..')
-const HollowStringFileData = require('../lib/file_data/hollow_string_file_data')
+const ot = require('../..')
+const HollowStringFileData = require('../../lib/file_data/hollow_string_file_data')
 const TextOperation = ot.TextOperation
 
 describe('HollowStringFileData', function () {

+ 1 - 1
libraries/overleaf-editor-core/test/label.test.js → libraries/overleaf-editor-core/test/unit/label.test.js

@@ -1,7 +1,7 @@
 'use strict'
 
 const { expect } = require('chai')
-const ot = require('..')
+const ot = require('../..')
 const Label = ot.Label
 
 describe('Label', function () {

+ 3 - 3
libraries/overleaf-editor-core/test/lazy_string_file_data.test.js → libraries/overleaf-editor-core/test/unit/lazy_string_file_data.test.js

@@ -5,11 +5,11 @@ const _ = require('lodash')
 const { expect } = require('chai')
 const sinon = require('sinon')
 
-const ot = require('..')
+const ot = require('../..')
 const File = ot.File
 const TextOperation = ot.TextOperation
-const LazyStringFileData = require('../lib/file_data/lazy_string_file_data')
-const EagerStringFileData = require('../lib/file_data/string_file_data')
+const LazyStringFileData = require('../../lib/file_data/lazy_string_file_data')
+const EagerStringFileData = require('../../lib/file_data/string_file_data')
 
 describe('LazyStringFileData', function () {
   beforeEach(function () {

+ 1 - 1
libraries/overleaf-editor-core/test/move_file_operation.test.js → libraries/overleaf-editor-core/test/unit/move_file_operation.test.js

@@ -1,7 +1,7 @@
 'use strict'
 
 const { expect } = require('chai')
-const ot = require('..')
+const ot = require('../..')
 const File = ot.File
 const MoveFileOperation = ot.MoveFileOperation
 const Snapshot = ot.Snapshot

+ 2 - 2
libraries/overleaf-editor-core/test/operation.test.js → libraries/overleaf-editor-core/test/unit/operation.test.js

@@ -3,8 +3,8 @@
 const _ = require('lodash')
 const { expect } = require('chai')
 
-const ot = require('..')
-const StringFileData = require('../lib/file_data/string_file_data')
+const ot = require('../..')
+const StringFileData = require('../../lib/file_data/string_file_data')
 const File = ot.File
 const AddFileOperation = ot.AddFileOperation
 const MoveFileOperation = ot.MoveFileOperation

+ 1 - 1
libraries/overleaf-editor-core/test/range.test.js → libraries/overleaf-editor-core/test/unit/range.test.js

@@ -1,7 +1,7 @@
 'use strict'
 
 const { expect } = require('chai')
-const Range = require('../lib/range')
+const Range = require('../../lib/range')
 
 describe('Range', function () {
   it('should create a range', function () {

+ 1 - 1
libraries/overleaf-editor-core/test/safe_pathname.test.js → libraries/overleaf-editor-core/test/unit/safe_pathname.test.js

@@ -1,7 +1,7 @@
 'use strict'
 
 const { expect } = require('chai')
-const ot = require('..')
+const ot = require('../..')
 const safePathname = ot.safePathname
 
 describe('safePathname', function () {

+ 3 - 3
libraries/overleaf-editor-core/test/scan_op.test.js → libraries/overleaf-editor-core/test/unit/scan_op.test.js

@@ -5,9 +5,9 @@ const {
   ScanOp,
   InsertOp,
   RemoveOp,
-} = require('../lib/operation/scan_op')
-const { UnprocessableError, ApplyError } = require('../lib/errors')
-const TrackingProps = require('../lib/file_data/tracking_props')
+} = require('../../lib/operation/scan_op')
+const { UnprocessableError, ApplyError } = require('../../lib/errors')
+const TrackingProps = require('../../lib/file_data/tracking_props')
 
 describe('ScanOp', function () {
   describe('fromJSON', function () {

+ 1 - 1
libraries/overleaf-editor-core/test/snapshot.test.js → libraries/overleaf-editor-core/test/unit/snapshot.test.js

@@ -7,7 +7,7 @@ const {
   TextOperation,
   Change,
   EditFileOperation,
-} = require('..')
+} = require('../..')
 
 describe('Snapshot', function () {
   describe('findBlobHashes', function () {

+ 2 - 2
libraries/overleaf-editor-core/test/string_file_data.test.js → libraries/overleaf-editor-core/test/unit/string_file_data.test.js

@@ -4,8 +4,8 @@
 const { expect } = require('chai')
 const _ = require('lodash')
 
-const ot = require('..')
-const StringFileData = require('../lib/file_data/string_file_data')
+const ot = require('../..')
+const StringFileData = require('../../lib/file_data/string_file_data')
 const TextOperation = ot.TextOperation
 
 describe('StringFileData', function () {

+ 0 - 0
libraries/overleaf-editor-core/test/support/fake_blob_store.js → libraries/overleaf-editor-core/test/unit/support/fake_blob_store.js


+ 0 - 0
libraries/overleaf-editor-core/test/support/random.js → libraries/overleaf-editor-core/test/unit/support/random.js


+ 3 - 3
libraries/overleaf-editor-core/test/support/random_text_operation.js → libraries/overleaf-editor-core/test/unit/support/random_text_operation.js

@@ -1,6 +1,6 @@
-const TrackingProps = require('../../lib/file_data/tracking_props')
-const ClearTrackingProps = require('../../lib/file_data/clear_tracking_props')
-const TextOperation = require('../../lib/operation/text_operation')
+const TrackingProps = require('../../../lib/file_data/tracking_props')
+const ClearTrackingProps = require('../../../lib/file_data/clear_tracking_props')
+const TextOperation = require('../../../lib/operation/text_operation')
 const random = require('./random')
 
 /**

+ 5 - 5
libraries/overleaf-editor-core/test/text_operation.test.js → libraries/overleaf-editor-core/test/unit/text_operation.test.js

@@ -10,12 +10,12 @@ const { expect } = require('chai')
 const random = require('./support/random')
 const randomOperation = require('./support/random_text_operation')
 
-const ot = require('..')
+const ot = require('../..')
 const TextOperation = ot.TextOperation
-const StringFileData = require('../lib/file_data/string_file_data')
-const { RetainOp, InsertOp, RemoveOp } = require('../lib/operation/scan_op')
-const TrackingProps = require('../lib/file_data/tracking_props')
-const ClearTrackingProps = require('../lib/file_data/clear_tracking_props')
+const StringFileData = require('../../lib/file_data/string_file_data')
+const { RetainOp, InsertOp, RemoveOp } = require('../../lib/operation/scan_op')
+const TrackingProps = require('../../lib/file_data/tracking_props')
+const ClearTrackingProps = require('../../lib/file_data/clear_tracking_props')
 
 describe('TextOperation', function () {
   const numTrials = 500

+ 3 - 3
libraries/overleaf-editor-core/test/tracked_change.test.js → libraries/overleaf-editor-core/test/unit/tracked_change.test.js

@@ -1,7 +1,7 @@
 // @ts-check
-const TrackedChange = require('../lib/file_data/tracked_change')
-const Range = require('../lib/range')
-const TrackingProps = require('../lib/file_data/tracking_props')
+const TrackedChange = require('../../lib/file_data/tracked_change')
+const Range = require('../../lib/range')
+const TrackingProps = require('../../lib/file_data/tracking_props')
 const { expect } = require('chai')
 
 describe('TrackedChange', function () {

+ 4 - 4
libraries/overleaf-editor-core/test/tracked_change_list.test.js → libraries/overleaf-editor-core/test/unit/tracked_change_list.test.js

@@ -1,9 +1,9 @@
 // @ts-check
-const TrackedChangeList = require('../lib/file_data/tracked_change_list')
-const TrackingProps = require('../lib/file_data/tracking_props')
-const ClearTrackingProps = require('../lib/file_data/clear_tracking_props')
+const TrackedChangeList = require('../../lib/file_data/tracked_change_list')
+const TrackingProps = require('../../lib/file_data/tracking_props')
+const ClearTrackingProps = require('../../lib/file_data/clear_tracking_props')
 const { expect } = require('chai')
-/** @import { TrackedChangeRawData } from '../lib/types' */
+/** @import { TrackedChangeRawData } from '../../lib/types' */
 
 describe('TrackedChangeList', function () {
   describe('applyInsert', function () {

+ 1 - 0
libraries/ranges-tracker/test/setup.js

@@ -0,0 +1 @@
+// no setup yet

+ 0 - 1
libraries/settings/test/notests.js

@@ -1 +0,0 @@
-// There are no tests yet

+ 1 - 0
libraries/stream-utils/test/setup.js

@@ -0,0 +1 @@
+// no setup yet

+ 1 - 0
libraries/validation-tools/test/setup.js

@@ -0,0 +1 @@
+// no setup yet