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 Text constants #136

Open
gbarkhatov opened this issue Sep 9, 2024 · 5 comments
Open

E2E Text constants #136

gbarkhatov opened this issue Sep 9, 2024 · 5 comments
Assignees

Comments

@gbarkhatov
Copy link
Contributor

gbarkhatov commented Sep 9, 2024

We use some inline text: I certify that I have read, I acknowledge that Keystone via QR code, Connect to BTC network, or even Staking Dashboard as the page name.
We need to extract these as constants and use both in:

  • our app
  • e2e tests

So these 2 consume the stuff from one place

graph TD
    A[Text Constants] --> B[App]
    A[Text Constants] --> C[E2E Tests]
Loading
@gbarkhatov
Copy link
Contributor Author

@vitsalis do we want our app to consume text constants or do we just want to extract needed text to e2e text constants?

i.e. this:

graph TD
    A[Text Constants] --> B[App]
    A[Text Constants] --> C[E2E Tests]
Loading

or this

graph TD
    A[App]
    B[Text Constants] --> C[E2E Tests]
Loading

2nd approach is related to E2E only
1st approach will involve extraction of the needed (by E2E tests) labels to a file, then both app and e2e will consume the stuff from one place. Other application labels/texts (not touched by e2e) will not be extracted

@vitsalis
Copy link
Member

I think the first option will be much easier to maintain, as if we update the text, we can be confident that both the app and e2e tests will receive the updates. Might make the code a bit more unreadable though as the text will not be directly visible.

@jrwbabylonlab
Copy link
Collaborator

@gbarkhatov is this ticket still ongoing?

@gbarkhatov
Copy link
Contributor Author

Partially implemented in #193, but since we have other priorities full implementation will be when I have time (and #193 is merged)

@jrwbabylonlab
Copy link
Collaborator

ok, i will put it outside of the next devnet iteration

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

No branches or pull requests

3 participants