Skip to content

Commit

Permalink
Merge pull request #214 from openstax/fixts
Browse files Browse the repository at this point in the history
adding element focuse and assertion
  • Loading branch information
omehes authored Sep 25, 2024
2 parents de8a57e + 7cd0881 commit 06c9628
Show file tree
Hide file tree
Showing 3 changed files with 2 additions and 6 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@ jobs:
sed -Ei 's/("version": ")(.+)(",)$/\1\2-'$VERSION_VALUE'\3/' package.json
npm run build:production
- name: Upload Extension
uses: actions/upload-artifact@v2
uses: actions/upload-artifact@v4
with:
name: editor-extension
path: editor-*.vsix
Expand Down
5 changes: 1 addition & 4 deletions tests/ui/pages/home.py
Original file line number Diff line number Diff line change
Expand Up @@ -79,10 +79,6 @@ def push_message_input_field_is_visible(self):
def push_message_input_field_not_visible(self):
return self.page.locator("div.monaco-inputbox.idle")

@property
def push_content_dialog_box(self):
return self.page.locator("div > div.notifications-toasts.visible")

@property
def generate_readme_button_is_visible(self):
return self.page.get_by_text("Generate README", exact=True)
Expand Down Expand Up @@ -279,6 +275,7 @@ def gitpod_menubar_locator(self):
return self.page.locator("div.menubar-menu-button")

def click_gitpod_menubar(self):
self.gitpod_menubar_locator.focus()
self.gitpod_menubar_locator.click()

@property
Expand Down
1 change: 0 additions & 1 deletion tests/ui/test_readme_validate_other_features.py
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,6 @@ def test_readme_validate_other_features(
home.click_push_content_button()

assert home.push_message_input_field_is_visible
assert "Push Content: Pushing..." in home.push_content_dialog_box.inner_text()

chrome_page.keyboard.press("Escape")
assert home.push_message_input_field_not_visible
Expand Down

0 comments on commit 06c9628

Please sign in to comment.