Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

ci: disposing unit should not emit error #2529

Merged
merged 13 commits into from
Jun 15, 2024
Merged

ci: disposing unit should not emit error #2529

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
9b6d6f5
ci: should throw no error on disposing unit
wzhudev Jun 15, 2024
47c2ce0
ci: trigger error
wzhudev Jun 15, 2024
0dbb76a
chore: fix code style
wzhudev Jun 15, 2024
1d8ef9f
fix: change logic of permission menu
ybzky Jun 15, 2024
bdc7010
fix: fix init value
ybzky Jun 15, 2024
2508e75
chore: change export
ybzky Jun 15, 2024
2d98c60
fix: bugfix
ybzky Jun 15, 2024
dc6b0a9
fix: fix disposing sequence
wzhudev Jun 15, 2024
29fe779
chore: remove redundant code
wzhudev Jun 15, 2024
f5ee3ad
fix: clear sheet permission when dispose unit
ybzky Jun 15, 2024
f88d0dc
fix: fix time sequence error
wzhudev Jun 15, 2024
f9a6163
fix: clear permission point when dispose unit
ybzky Jun 15, 2024
3a50452
fix: fix test
ybzky Jun 15, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
fix: fix test
  • Loading branch information
@ybzky
ybzky committed Jun 15, 2024
commit 3a50452a802765afd4f59dd5b2bf8db0d647beef
3 changes: 2 additions & 1 deletion packages/facade/src/apis/__tests__/create-test-bed.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,7 @@ import {
} from '@univerjs/core';
import { FormulaDataModel, FunctionService, IFunctionService, LexerTreeBuilder } from '@univerjs/engine-formula';
import { ISocketService, WebSocketService } from '@univerjs/network';
import { SelectionManagerService, SheetInterceptorService, WorkbookPermissionService, WorksheetPermissionService, WorksheetProtectionPointModel, WorksheetProtectionRuleModel } from '@univerjs/sheets';
import { RangeProtectionRuleModel, SelectionManagerService, SheetInterceptorService, WorkbookPermissionService, WorksheetPermissionService, WorksheetProtectionPointModel, WorksheetProtectionRuleModel } from '@univerjs/sheets';
import {
DescriptionService,
FormulaCustomFunctionService,
Expand Down Expand Up @@ -151,6 +151,7 @@ export function createFacadeTestBed(workbookData?: IWorkbookData, dependencies?:
injector.add([WorksheetPermissionService]);
injector.add([WorkbookPermissionService]);
injector.add([WorksheetProtectionPointModel]);
injector.add([RangeProtectionRuleModel]);
injector.add([IAuthzIoService, { useClass: AuthzIoLocalService }]);
injector.add([WorksheetProtectionRuleModel]);

Expand Down
3 changes: 2 additions & 1 deletion packages/sheets-formula/src/commands/operations/__tests__/create-command-test-bed.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@
import type { IWorkbookData } from '@univerjs/core';
import { AuthzIoLocalService, IAuthzIoService, ILogService, IUniverInstanceService, LocaleType, LogLevel, Plugin, Univer, UniverInstanceType } from '@univerjs/core';
import { LexerTreeBuilder } from '@univerjs/engine-formula';
import { SelectionManagerService, SheetInterceptorService, WorkbookPermissionService, WorksheetPermissionService, WorksheetProtectionPointModel, WorksheetProtectionRuleModel } from '@univerjs/sheets';
import { RangeProtectionRuleModel, SelectionManagerService, SheetInterceptorService, WorkbookPermissionService, WorksheetPermissionService, WorksheetProtectionPointModel, WorksheetProtectionRuleModel } from '@univerjs/sheets';
import type { Dependency } from '@wendellhu/redi';
import { Inject, Injector } from '@wendellhu/redi';
import { EditorService, IEditorService } from '@univerjs/ui';
Expand Down Expand Up @@ -80,6 +80,7 @@ export function createCommandTestBed(workbookData?: IWorkbookData, dependencies?
injector.add([WorksheetPermissionService]);
injector.add([WorksheetProtectionPointModel]);
injector.add([WorkbookPermissionService]);
injector.add([RangeProtectionRuleModel]);
injector.add([IAuthzIoService, { useClass: AuthzIoLocalService }]);
injector.add([WorksheetProtectionRuleModel]);

Expand Down
2 changes: 2 additions & 0 deletions packages/sheets/src/commands/commands/__tests__/create-command-test-bed.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -36,6 +36,7 @@ import { BorderStyleManagerService } from '../../../services/border-style-manage
import { SelectionManagerService } from '../../../services/selection-manager.service';
import { SheetInterceptorService } from '../../../services/sheet-interceptor/sheet-interceptor.service';
import { WorksheetProtectionPointModel, WorksheetProtectionRuleModel } from '../../../services/permission/worksheet-permission';
import { RangeProtectionRuleModel } from '../../../model/range-protection-rule.model';

const TEST_WORKBOOK_DATA_DEMO: IWorkbookData = {
id: 'test',
Expand Down Expand Up @@ -98,6 +99,7 @@ export function createCommandTestBed(workbookData?: IWorkbookData, dependencies?
override onStarting(injector: Injector): void {
injector.add([WorksheetPermissionService]);
injector.add([WorksheetProtectionPointModel]);
injector.add([RangeProtectionRuleModel]);
injector.add([WorkbookPermissionService]);
injector.add([WorksheetProtectionRuleModel]);
injector.add([SelectionManagerService]);
Expand Down
Loading