-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
Test reorg: KV #26289
Test reorg: KV #26289
Conversation
CI Results: failed ❌ |
@@ -13,7 +13,7 @@ import consoleClass from 'vault/tests/pages/components/console/ui-panel'; | |||
import authForm from 'vault/tests/pages/components/auth-form'; | |||
import controlGroup from 'vault/tests/pages/components/control-group'; | |||
import controlGroupSuccess from 'vault/tests/pages/components/control-group-success'; | |||
import { writeSecret } from 'vault/tests/helpers/kv/kv-run-commands'; | |||
import { writeSecret } from 'vault/tests/helpers/kv/kv-selectors'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I thought I remembered us talking about naming files <resource>-helpers
even if it's redundant since they contain helper functions and selectors?
|
||
import { create } from 'ember-cli-page-object'; | ||
import { deleteEngineCmd, runCmd } from 'vault/tests/helpers/commands'; | ||
import { KV_LIST } from 'vault/tests/helpers/components/kv/page/list-selectors'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Even though page
components are technically components, they map to route templates and I would have expected these selectors to be a separate block in the kv/kv-selectors
file.
I had imagined that the helpers/components/
would contain components that are reused all over the app.
b793b7f
to
b4edfe5
Compare
We can do the KV test reorg as a separate follow-on |
WIP -- looking for feedback on whether this is the right approach