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

Fix tests #5

Merged
merged 3 commits into from
Dec 17, 2024
Merged

Fix tests #5

merged 3 commits into from
Dec 17, 2024

Conversation

feedmypixel
Copy link
Contributor

No description provided.

- New teams work
- Fix issues around create flow
- Send baseUrl and Debug to wdio as envs
- Align local conf with base conf
- Delete docker conf
- Fix debug env now being passed correctly
@feedmypixel feedmypixel merged commit 5d2ebe9 into main Dec 17, 2024
1 check passed
@feedmypixel feedmypixel deleted the fix-tests branch December 17, 2024 14:15
"test:docker": "npm run clean && wdio run wdio.docker.conf.js",
"test:local": "npm run clean && BASE_URL=\"http://localhost:3000\" wdio run wdio.local.conf.js",
"test:local:debug": "npm run clean && DEBUG=true BASE_URL=\"http://localhost:3000\" wdio run wdio.local.conf.js",
"test:docker": "npm run clean && BASE_URL=\"http://cdp.127.0.0.1.sslip.io:3333\" wdio run wdio.local.conf.js",
Copy link
Contributor

Choose a reason for hiding this comment

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

I was hoping we could remove the :docker one and just use the :local one listening to an env var if set otherwise default to localhost. That way we could also push this to the template and let tenants override it in an env var if they want to

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.

2 participants