Skip to content

Commit 61fbb83

Browse files
committed
fix: fixes from review
1 parent 763a637 commit 61fbb83

4 files changed

Lines changed: 2 additions & 10 deletions

File tree

src/course-outline/CourseOutline.test.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -461,7 +461,7 @@ describe('<CourseOutline />', () => {
461461
});
462462
await user.click(addSectionFromLibraryButton);
463463

464-
// Start the add existring section flow
464+
// Start the add existing section flow
465465
const courseBlockId = courseOutlineIndexMock.courseStructure.id;
466466
expect(startCurrentFlow).toHaveBeenCalledWith({
467467
flowType: ContainerType.Section,

src/course-outline/header-navigations/HeaderActions.test.tsx

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,6 @@ jest.mock('../outline-sidebar/OutlineSidebarContext', () => ({
3838
const renderComponent = (props?: Partial<HeaderActionsProps>) =>
3939
render(
4040
<HeaderActions
41-
actions={headerNavigationsActions}
4241
courseActions={courseActions}
4342
headerNavigationsActions={headerNavigationsActions}
4443
isReIndexShow

src/course-outline/header-navigations/HeaderActions.tsx

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -18,9 +18,6 @@ import { useOutlineSidebarContext } from '../outline-sidebar/OutlineSidebarConte
1818
import messages from './messages';
1919

2020
export interface HeaderActionsProps {
21-
actions: {
22-
lmsLink: string;
23-
};
2421
courseActions: XBlockActions;
2522
headerNavigationsActions: {
2623
handleNewSection: () => void;
@@ -34,18 +31,16 @@ export interface HeaderActionsProps {
3431
}
3532

3633
const HeaderActions = ({
37-
actions,
3834
courseActions,
3935
headerNavigationsActions,
4036
isReIndexShow,
4137
isDisabledReindexButton,
4238
errors,
4339
}: HeaderActionsProps) => {
4440
const intl = useIntl();
45-
const { lmsLink } = actions;
4641

4742
const { clearSelection, open, setCurrentPageKey } = useOutlineSidebarContext();
48-
const { handleReIndex } = headerNavigationsActions;
43+
const { handleReIndex, lmsLink } = headerNavigationsActions;
4944

5045
const handleCourseInfoClick = () => {
5146
clearSelection();

src/course-outline/subsection-card/SubsectionCard.test.tsx

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -329,8 +329,6 @@ describe('<SubsectionCard />', () => {
329329
expect(useUnitFromLibraryButton).toBeInTheDocument();
330330
await user.click(useUnitFromLibraryButton);
331331

332-
screen.logTestingPlaygroundURL();
333-
334332
expect(startCurrentFlow).toHaveBeenCalledWith({
335333
flowType: ContainerType.Unit,
336334
parentLocator: 'block-v1:UNIX+UX1+2025_T3+type@subsection+block@0',

0 commit comments

Comments
 (0)