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

Wrong behavior when create a factory with an existing name on the dashboard #10121

Closed
artaleks9 opened this issue Jun 20, 2018 · 1 comment
Closed

Wrong behavior when create a factory with an existing name on the dashboard #10121

artaleks9 opened this issue Jun 20, 2018 · 1 comment
Assignees
@akurinnoy
Labels
e2e-test/failure Issues that is related to a test failures reported by our CI platform and our QE. kind/bug Outline of a bug - must adhere to the bug report template. severity/P2 Has a minor but important impact to the usage or development of the system.

Comments

@artaleks9
Copy link
Contributor

artaleks9 commented Jun 20, 2018 •
edited by SkorikSergey
Loading

Description

  • Open the dashboard and create a factory
  • Try to create a factory with a same name

Expected behavior:

  • the 'Create' button should be disabled in this case

Observed behavior:

  • there is the error message below the name field: 'This factory name is already used.'
  • but the 'Create' button' is enabled and the factory will be created with a random generate name

Che version: 6.8.0
OS and version: Fedora 28
Che install: Docker/OCP
Docker version: 17.09
API docker version: 1.32
Failed selenium test: CreateFactoryTest

Additional information:

  • Related selenium tests: CreateFactoryTest [checkHandlingOfFactoryNames()]
  • Problem can be reliably reproduced, doesn't happen randomly: [Yes]
  • See the attachment:
    factorry-exist-name
@artaleks9 artaleks9 added kind/bug Outline of a bug - must adhere to the bug report template. severity/P2 Has a minor but important impact to the usage or development of the system. team/ide2 labels Jun 20, 2018
@skabashnyuk
Copy link
Contributor

@ashumilova wdyt is it a bug or new feature?

@artaleks9 artaleks9 added the e2e-test/failure Issues that is related to a test failures reported by our CI platform and our QE. label Jul 26, 2018
@akurinnoy akurinnoy self-assigned this Aug 15, 2018
@akurinnoy akurinnoy mentioned this issue Aug 15, 2018
Merged
This was referenced Aug 20, 2018
Closed
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@akurinnoy akurinnoy

Labels
e2e-test/failure Issues that is related to a test failures reported by our CI platform and our QE. kind/bug Outline of a bug - must adhere to the bug report template. severity/P2 Has a minor but important impact to the usage or development of the system.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@skabashnyuk @artaleks9 @akurinnoy