Skip to content

feat: Add additional keyboard navigation tests #496

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

Merged
RoboErikG merged 10 commits into from
May 7, 2025
Merged

feat: Add additional keyboard navigation tests #496

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
9e95807
chore: Implement field navigation tests.
johnnesky May 2, 2025
0ebb1bc
Merge branch 'main' into nesky_block_nav_tests
johnnesky May 2, 2025
ed79dd6
Merge branch 'main' into nesky_block_nav_tests
RoboErikG May 5, 2025
5db10b7
Update expected block with the new one it ends on
RoboErikG May 6, 2025
e074129
Refactor tests to use FocusManager/focus() and comment out failing tests
RoboErikG May 7, 2025
b1d425f
Merge branch 'main' into field-nav-tests
RoboErikG May 7, 2025
98fc0fc
Use .skip instead of commenting out tests
RoboErikG May 7, 2025
d3c0672
Fix or skip tests as needed
RoboErikG May 7, 2025
a7ce3c5
Add TODO for failing test
RoboErikG May 7, 2025
913a96e
Auto-format fixes
RoboErikG May 7, 2025
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
3 changes: 3 additions & 0 deletions test/index.html
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -164,6 +164,9 @@
<option value="sun">sun</option>
<option value="blank">blank canvas</option>
<option value="moreBlocks">more blocks</option>
<option value="navigationTestBlocks">
navigation test blocks
</option>
</select>
</div>
<div>
Expand Down
159 changes: 158 additions & 1 deletion test/loadTestBlocks.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -220,7 +220,7 @@ const simpleCircle = {
},
{
'type': 'p5_draw',
'id': 'draw_root',
'id': 'p5_draw_1',
'x': 0,
'y': 332,
'deletable': false,
Expand Down Expand Up @@ -414,6 +414,162 @@ const moreBlocks = {
},
};

const navigationTestBlocks = {
'blocks': {
'languageVersion': 0,
'blocks': [
{
'type': 'p5_setup',
'id': 'p5_setup_1',
'x': 0,
'y': 75,
'deletable': false,
'inputs': {
'STATEMENTS': {
'block': {
'type': 'p5_canvas',
'id': 'p5_canvas_1',
'deletable': false,
'movable': false,
'fields': {
'WIDTH': 400,
'HEIGHT': 400,
},
},
},
},
},
{
'type': 'p5_draw',
'id': 'p5_draw_1',
'x': 0,
'y': 332,
'deletable': false,
'inputs': {
'STATEMENTS': {
'block': {
'type': 'controls_if',
'id': 'controls_if_1',
'next': {
'block': {
'type': 'controls_if',
'id': 'controls_if_2',
'inputs': {
'IF0': {
'block': {
'type': 'logic_boolean',
'id': 'logic_boolean_1',
'fields': {
'BOOL': 'TRUE',
},
},
},
'DO0': {
'block': {
'type': 'text_print',
'id': 'text_print_1',
'inputs': {
'TEXT': {
'shadow': {
'type': 'text',
'id': 'text_1',
'fields': {
'TEXT': 'abc',
},
},
},
},
},
},
},
'next': {
'block': {
'type': 'controls_repeat',
'id': 'controls_repeat_1',
'fields': {
'TIMES': 10,
},
'inputs': {
'DO': {
'block': {
'type': 'draw_emoji',
'id': 'draw_emoji_1',
'fields': {
'emoji': '❤️',
},
'next': {
'block': {
'type': 'simple_circle',
'id': 'simple_circle_1',
'inputs': {
'COLOR': {
'shadow': {
'type': 'colour_picker',
'id': 'colour_picker_1',
'fields': {
'COLOUR': '#ff0000',
},
},
},
},
},
},
},
},
},
'next': {
'block': {
'type': 'controls_repeat_ext',
'id': 'controls_repeat_ext_1',
'inputs': {
'TIMES': {
'shadow': {
'type': 'math_number',
'id': 'math_number_1',
'fields': {
'NUM': 10,
},
},
'block': {
'type': 'math_modulo',
'id': 'math_modulo_1',
'inputs': {
'DIVIDEND': {
'shadow': {
'type': 'math_number',
'id': 'math_number_2',
'fields': {
'NUM': 64,
},
},
},
'DIVISOR': {
'shadow': {
'type': 'math_number',
'id': 'math_number_3',
'fields': {
'NUM': 10,
},
},
},
},
},
},
},
},
},
},
},
},
},
},
},
},
},
],
},
};

/**
* Loads saved state from local storage into the given workspace.
* @param {Blockly.Workspace} workspace Blockly workspace to load into.
Expand All @@ -425,6 +581,7 @@ export const load = function (workspace, scenarioString) {
'sun': sunnyDay,
'simpleCircle': simpleCircle,
'moreBlocks': moreBlocks,
'navigationTestBlocks': navigationTestBlocks,
};

const data = JSON.stringify(scenarioMap[scenarioString]);
Expand Down
Loading
Loading