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

[Coverage]: Add Reset tests for known non-resetable configurations #44

Closed
2 tasks done
OndrejKotek opened this issue Jun 13, 2023 · 1 comment
Closed
2 tasks done

[Coverage]: Add Reset tests for known non-resetable configurations #44

OndrejKotek opened this issue Jun 13, 2023 · 1 comment
Assignees
@OndrejKotek
Labels
coverage distributable-web interface

Comments

@OndrejKotek
Copy link
Contributor

OndrejKotek commented Jun 13, 2023 •
edited
Loading

Description

#32 and #35 removes failing Reset tests. The removed tests should be replaced by a check that the None of the attributes could be reset. warning is displayed.

  • Configuration => Subsystem => Distributable Web => Configuration
  • Configuration => Interface

For inspiration: https://github.com/hal/berg/blob/main/packages/testsuite/cypress/support/verification-utils.ts#L3

Affects

configuration

Subsystem

distributable-web
This was referenced Jun 13, 2023
Merged
Merged
@OndrejKotek OndrejKotek changed the title [Coverage]: add Reset tests for known non-resetable configurations [Coverage]: Add Reset tests for known non-resetable configurations Jun 13, 2023
@OndrejKotek OndrejKotek self-assigned this Jun 13, 2023
OndrejKotek added a commit to OndrejKotek/berg that referenced this issue Jun 14, 2023
@OndrejKotek
[hal#44] Add Reset tests for known non-resetable configurations
e135b56 
Improved resetForm command to take non-resetable configurations into account.

Re-added the Reset test to
- Configuration => Subsystem => Distributable Web => Configuration
- Configuration => Interface
@OndrejKotek OndrejKotek mentioned this issue Jun 14, 2023
Merged
spriadka added a commit that referenced this issue Jun 14, 2023
@spriadka
Merge pull request #60 from OndrejKotek/boot-experience
ac919cc 
[#44] Add Reset tests for known non-resetable configurations
jedla97 pushed a commit to jedla97/berg that referenced this issue Jun 14, 2023
@OndrejKotek @jedla97
[hal#44] Add Reset tests for known non-resetable configurations
78e683f 
Improved resetForm command to take non-resetable configurations into account.

Re-added the Reset test to
- Configuration => Subsystem => Distributable Web => Configuration
- Configuration => Interface
@OndrejKotek
Copy link
Contributor Author

Fixed with #60

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@OndrejKotek OndrejKotek

Labels
coverage distributable-web interface
Projects
Berg
Status: Done
Milestone
No milestone
Development

No branches or pull requests
1 participant
@OndrejKotek