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.

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

Autotests: Replace Add/Remove Hydrogens operations with valid helper functions #6052

Closed
AlexeyGirin opened this issue Dec 4, 2024 · 0 comments · Fixed by #6214
Closed

Autotests: Replace Add/Remove Hydrogens operations with valid helper functions #6052

AlexeyGirin opened this issue Dec 4, 2024 · 0 comments · Fixed by #6214
Assignees
Labels
Autotests refactor Code refactoring, without changing the functionality
Milestone

Comments

@AlexeyGirin
Copy link
Collaborator

AlexeyGirin commented Dec 4, 2024

Identify all Add/Remove Hydrogens operations made by using methods
selectTopPanelButton(TopPanelButton.toggleExplicitHydrogens, page);
and replace them with proper methods (create them if needed)

Do not forget to wrap it into

    await waitForSpinnerFinishedWork(
      page,
      async () =>
        await selectTopPanelButton(
          TopPanelButton.toggleExplicitHydrogens,
          page,
        ),
    );

Hint: Try to find TopPanelButton.toggleExplicitHydrogens

@AlexeyGirin AlexeyGirin added refactor Code refactoring, without changing the functionality Autotests labels Dec 4, 2024
@AlexeyGirin AlexeyGirin added this to the Autotests milestone Dec 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Autotests refactor Code refactoring, without changing the functionality
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants