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

Add initalSwitchConnectionFailEpic to rootEpic #1086

Merged
merged 1 commit into from
Apr 17, 2020

Conversation

HerrEmil
Copy link
Contributor

@HerrEmil HerrEmil commented Apr 16, 2020

In the autorun play commands PR I added the initialSwitchConnectionFailEpic epic, but forgot to combine it in the root epic so it was never active. This created the bug that the server switch fail frame was not displayed when starting browser from neo4j desktop without a started database. This PR adds the epic to rootEpic to fix this bug and adds an E2E test to verify that the server switch fail frame is displayed as intended.

changelog: Fix initial load behavior when opened in Neo4j Desktop without an active db.

@HerrEmil HerrEmil requested a review from oskarhane April 16, 2020 15:20
Copy link
Member

@oskarhane oskarhane left a comment

Choose a reason for hiding this comment

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

LGTM 🌮

@@ -61,4 +61,26 @@ describe('Neo4j Desktop environment', () => {
.first()
.should('contain', 'Connection updated')
})

it('displays disconnected banner and connection failed frame when initial state is INACTIVE', () => {
Copy link
Member

Choose a reason for hiding this comment

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

👍

@HerrEmil HerrEmil merged commit cc0b0e1 into neo4j:master Apr 17, 2020
@HerrEmil HerrEmil deleted the server-switch-fail branch April 17, 2020 06:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants