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

E2E network management #5345

Merged
merged 23 commits into from
Oct 4, 2023
Merged

E2E network management #5345

merged 23 commits into from
Oct 4, 2023

Conversation

oskarleonard
Copy link
Contributor

@oskarleonard oskarleonard commented Sep 28, 2023

What was the problem?

This PR resolves #5083

How was it solved?

Added E2E tests for

  1. Adding network
  2. Switch network
  3. Edit network
  4. Delete network

Also removed unnecessary logic in before hook since we now can use a stable local network (customNode) when running the tests in jenkins.

How was it tested?

  1. git pull
  2. DEFAULT_NETWORK=customNode yarn dev
  3. PW_BASE_URL=http://localhost:8080/# yarn run cucumber:playwright:open
  4. Expected: Tests should pass

…e hook when we can initialize the application with customNode
@oskarleonard oskarleonard self-assigned this Sep 28, 2023
@oskarleonard oskarleonard marked this pull request as draft September 28, 2023 09:45
@oskarleonard oskarleonard marked this pull request as ready for review September 28, 2023 14:44
Jenkinsfile Outdated Show resolved Hide resolved
e2e/features/networkManagement/NetworkManagement.feature Outdated Show resolved Hide resolved
src/modules/network/utils/getNetwork.test.js Outdated Show resolved Hide resolved
@ManuGowda ManuGowda removed their request for review September 29, 2023 10:21
@ManuGowda ManuGowda merged commit dce6e8c into release/3.0.0 Oct 4, 2023
6 checks passed
@ManuGowda ManuGowda deleted the 5083-e2e-network-management branch October 4, 2023 15:35
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.

4 participants