Skip to content

chore: add selenium script to simulate user#30

Merged
beeme1mr merged 1 commit intoopen-feature:mainfrom
open-feature-forking:selenium
Apr 29, 2025
Merged

chore: add selenium script to simulate user#30
beeme1mr merged 1 commit intoopen-feature:mainfrom
open-feature-forking:selenium

Conversation

@alexandraoberaigner
Copy link
Copy Markdown
Contributor

This PR

adds a selenium ide project file which simulates user purchases in the toggle shop ui

Notes

chrome doesn't support selenium ide extension any more, but it works with edge

How to test

install the selenium browser extension & import the new file as project, run the suite

Signed-off-by: Alexandra Oberaigner <alexandra.oberaigner@dynatrace.com>
@toddbaert toddbaert requested review from beeme1mr and toddbaert April 23, 2025 19:57
Copy link
Copy Markdown
Member

@toddbaert toddbaert left a comment

Choose a reason for hiding this comment

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

I like this addition. It even makes for a good demo.

Copy link
Copy Markdown
Member

@beeme1mr beeme1mr left a comment

Choose a reason for hiding this comment

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

I'm iffy about this honestly. I see the value but I'm not sure I want to maintain a +1k selenium file.

@toddbaert
Copy link
Copy Markdown
Member

I'm iffy about this honestly. I see the value but I'm not sure I want to maintain a +1k selenium file.

It's generated by recording extensions.

I think at least until we have a k6 demo that includes tracking, it's worth having. We needed it for our presentations.

@beeme1mr beeme1mr changed the title feat: add selenium IDE project file for user simulation & add docs chore: add selenium script to simulate user Apr 29, 2025
Copy link
Copy Markdown
Member

@beeme1mr beeme1mr left a comment

Choose a reason for hiding this comment

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

I'd like to replace this with a k6 browser script eventually but this is good enough for now. Thanks!

@beeme1mr beeme1mr added this pull request to the merge queue Apr 29, 2025
Merged via the queue into open-feature:main with commit 1df3646 Apr 29, 2025
4 checks passed
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.

3 participants