Skip to content

Commit

Permalink
chore: modify detection of credentials to use for cypress tests (#4580)
Browse files Browse the repository at this point in the history
* chore: modify detection of credentials to use for cypress tests

* chore: add comment
  • Loading branch information
techfg authored Feb 7, 2025
1 parent b531dd5 commit 8e577ca
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 14 deletions.
10 changes: 2 additions & 8 deletions .github/workflows/ci.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -73,13 +73,13 @@ jobs:

- name: Build and test
env:
UESIO_DEV: "true"
UESIO_DEV: "true" # required to ensure packui gets built
run: npx nx affected -t build test --configuration=ci --parallel=5

- name: Prep for docker image
id: setDockerSHAs
env:
UESIO_DEV: "true"
UESIO_DEV: "true" # required to ensure packui gets built
run: |
# We lint/test/build affected but in order to build image, we need to ensure everything
# is built. Build anything that hasn't been built yet (takes advantage of nx cache
Expand Down Expand Up @@ -207,13 +207,9 @@ jobs:
run: npm ci

- name: Check formatting
env:
UESIO_DEV: "true"
run: npx nx format:check --verbose

- name: Lint
env:
UESIO_DEV: "true"
run: npx nx affected -t lint --configuration=ci --parallel=5

typecheck:
Expand Down Expand Up @@ -245,8 +241,6 @@ jobs:
run: npm ci

- name: Typecheck
env:
UESIO_DEV: "true"
run: npx nx affected -t typecheck --configuration=ci --parallel=5

update-dev-branch:
Expand Down
7 changes: 1 addition & 6 deletions apps/platform-e2e/cypress.config.ts
Original file line number Diff line number Diff line change
Expand Up @@ -5,13 +5,8 @@ const studioBaseUrl =
process.env.UESIO_APP_URL || `https://studio.uesio-dev.com:3000`
const automationUsername = process.env.UESIO_AUTOMATION_USERNAME || "uesio"
const automationPassword = process.env.UESIO_AUTOMATION_PASSWORD
const useMockLogin = process.env.UESIO_DEV
const useMockLogin = !automationPassword
const inCi = process.env.CI === "true"

if (!useMockLogin && !automationPassword) {
throw new Error("UESIO_AUTOMATION_PASSWORD was not provided")
}

export default defineConfig({
e2e: {
...nxE2EPreset(__filename, {
Expand Down

0 comments on commit 8e577ca

Please sign in to comment.