Skip to content

Conversation

JackDaTsai
Copy link

  1. New Parameter Setting:

    • Tests the process of adding a new parameter through the UI
    • Verifies that the newly added parameter appears in the selection list
  2. Power Input Validation:

    • Implements tests for various power input scenarios
    • Covers edge cases such as values below 10, decimal inputs, and extreme values
    • Ensures proper handling of non-numeric inputs
  3. Color Layer Toggle:

    • Adds a test for switching between single and full color layers
    • Verifies the presence and functionality of the toggle option in the context menu

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remaining comments which cannot be posted as a review comment to avoid GitHub Rate Limit

eslint

🚫 [eslint] <@typescript-eslint/indent> reported by reviewdog 🐶
Expected indentation of 6 spaces but found 8.


🚫 [eslint] <@typescript-eslint/indent> reported by reviewdog 🐶
Expected indentation of 2 spaces but found 4.


🚫 [eslint] <no-trailing-spaces> reported by reviewdog 🐶
Trailing spaces not allowed.


🚫 [eslint] <@typescript-eslint/indent> reported by reviewdog 🐶
Expected indentation of 2 spaces but found 4.

it('should handle extreme low values', () => {


🚫 [eslint] <@typescript-eslint/indent> reported by reviewdog 🐶
Expected indentation of 4 spaces but found 6.

cy.get('#power-input').clear().type('0');


🚫 [eslint] <@typescript-eslint/indent> reported by reviewdog 🐶
Expected indentation of 4 spaces but found 6.

cy.get('#power-input').type('{enter}');


🚫 [eslint] <@typescript-eslint/indent> reported by reviewdog 🐶
Expected indentation of 4 spaces but found 6.

// should have number 1 as the lowest value


🚫 [eslint] <@typescript-eslint/indent> reported by reviewdog 🐶
Expected indentation of 4 spaces but found 6.

cy.get('#power-input').should('have.value', '1');


🚫 [eslint] <@typescript-eslint/indent> reported by reviewdog 🐶
Expected indentation of 4 spaces but found 6.

cy.get('.src-web-app-views-beambox-Right-Panels-ConfigPanel-Block-module__warning--aboql')


🚫 [eslint] <@typescript-eslint/indent> reported by reviewdog 🐶
Expected indentation of 6 spaces but found 8.


🚫 [eslint] <no-trailing-spaces> reported by reviewdog 🐶
Trailing spaces not allowed.


🚫 [eslint] <@typescript-eslint/indent> reported by reviewdog 🐶
Expected indentation of 4 spaces but found 6.

cy.get('#power-input').clear().type('-1');


🚫 [eslint] <@typescript-eslint/indent> reported by reviewdog 🐶
Expected indentation of 4 spaces but found 6.

cy.get('#power-input').type('{enter}');


🚫 [eslint] <@typescript-eslint/indent> reported by reviewdog 🐶
Expected indentation of 4 spaces but found 6.

cy.get('.src-web-app-views-beambox-Right-Panels-ConfigPanel-Block-module__warning--aboql')


🚫 [eslint] <@typescript-eslint/indent> reported by reviewdog 🐶
Expected indentation of 6 spaces but found 8.


🚫 [eslint] <@typescript-eslint/indent> reported by reviewdog 🐶
Expected indentation of 2 spaces but found 4.


🚫 [eslint] <no-trailing-spaces> reported by reviewdog 🐶
Trailing spaces not allowed.


🚫 [eslint] <@typescript-eslint/indent> reported by reviewdog 🐶
Expected indentation of 2 spaces but found 4.

it('should handle extreme high values', () => {


🚫 [eslint] <@typescript-eslint/indent> reported by reviewdog 🐶
Expected indentation of 4 spaces but found 6.

cy.get('#power-input').clear().type('100');


🚫 [eslint] <@typescript-eslint/indent> reported by reviewdog 🐶
Expected indentation of 4 spaces but found 6.

cy.get('#power-input').type('{enter}');


🚫 [eslint] <@typescript-eslint/indent> reported by reviewdog 🐶
Expected indentation of 4 spaces but found 6.

cy.get('.src-web-app-views-beambox-Right-Panels-ConfigPanel-Block-module__warning--aboql')


🚫 [eslint] <@typescript-eslint/indent> reported by reviewdog 🐶
Expected indentation of 6 spaces but found 8.


🚫 [eslint] <no-trailing-spaces> reported by reviewdog 🐶
Trailing spaces not allowed.


🚫 [eslint] <@typescript-eslint/indent> reported by reviewdog 🐶
Expected indentation of 4 spaces but found 6.

cy.get('#power-input').clear().type('1000');


🚫 [eslint] <@typescript-eslint/indent> reported by reviewdog 🐶
Expected indentation of 4 spaces but found 6.

cy.get('#power-input').type('{enter}');


🚫 [eslint] <@typescript-eslint/indent> reported by reviewdog 🐶
Expected indentation of 4 spaces but found 6.

cy.get('.src-web-app-views-beambox-Right-Panels-ConfigPanel-Block-module__warning--aboql')


🚫 [eslint] <@typescript-eslint/indent> reported by reviewdog 🐶
Expected indentation of 6 spaces but found 8.


🚫 [eslint] <@typescript-eslint/indent> reported by reviewdog 🐶
Expected indentation of 2 spaces but found 4.


🚫 [eslint] <no-trailing-spaces> reported by reviewdog 🐶
Trailing spaces not allowed.


🚫 [eslint] <@typescript-eslint/indent> reported by reviewdog 🐶
Expected indentation of 2 spaces but found 4.

it('should handle non-numeric input', () => {


🚫 [eslint] <@typescript-eslint/indent> reported by reviewdog 🐶
Expected indentation of 4 spaces but found 6.

cy.get('#power-input').clear().type('abc');


🚫 [eslint] <@typescript-eslint/indent> reported by reviewdog 🐶
Expected indentation of 4 spaces but found 6.

cy.get('#power-input').type('{enter}');


🚫 [eslint] <@typescript-eslint/indent> reported by reviewdog 🐶
Expected indentation of 4 spaces but found 6.

cy.get('#power-input').should('have.value', '15.0');


🚫 [eslint] <@typescript-eslint/indent> reported by reviewdog 🐶
Expected indentation of 2 spaces but found 4.


🚫 [eslint] <@typescript-eslint/indent> reported by reviewdog 🐶
Expected indentation of 0 spaces but found 2.


🚫 [eslint] <eol-last> reported by reviewdog 🐶
Newline required at end of file but not found.


⚠️ [eslint] <cypress/no-force> reported by reviewdog 🐶
Do not use force on click and type calls

cy.get('span.ant-select-selection-item').click({ multiple: true, force: true });


⚠️ [eslint] <cypress/no-force> reported by reviewdog 🐶
Do not use force on click and type calls


⚠️ [eslint] <cypress/no-force> reported by reviewdog 🐶
Do not use force on click and type calls

cy.contains('button.ant-btn', 'Confirm')


🚫 [eslint] <no-trailing-spaces> reported by reviewdog 🐶
Trailing spaces not allowed.


🚫 [eslint] <no-trailing-spaces> reported by reviewdog 🐶
Trailing spaces not allowed.

// check exist or not : Switch to single color layer


🚫 [eslint] <cypress/no-unnecessary-waiting> reported by reviewdog 🐶
Do not wait for arbitrary time periods


🚫 [eslint] <eol-last> reported by reviewdog 🐶
Newline required at end of file but not found.


cy.contains('li[role="menuitem"]', 'Document Settings').click();

cy.get('span.ant-select-selection-item').click({ multiple: true, force: true });

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ [eslint] <cypress/no-force> reported by reviewdog 🐶
Do not use force on click and type calls

cy.get('span.ant-select-selection-item').click({ multiple: true, force: true });
cy.contains('.ant-select-item-option-content', 'Ador').click();

cy.contains('span', 'Save')

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ [eslint] <cypress/no-force> reported by reviewdog 🐶
Do not use force on click and type calls

.should('be.visible')
.click();

cy.contains('button.ant-btn', 'Confirm')

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ [eslint] <cypress/no-force> reported by reviewdog 🐶
Do not use force on click and type calls

.should('exist', 'Switch to single color layer')
.click();

cy.wait(1500);

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚫 [eslint] <cypress/no-unnecessary-waiting> reported by reviewdog 🐶
Do not wait for arbitrary time periods

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant