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

Dlaliberte playwright 06 22 #3184

Merged
merged 104 commits into from
Sep 6, 2023
Merged
Show file tree
Hide file tree
Changes from 97 commits
Commits
Show all changes
104 commits
Select commit Hold shift + click to select a range
73fc4da
Initial install of playwright visual tests
dlaliberte Jun 28, 2023
9f5d80d
Merge remote-tracking branch 'origin/main' into dlaliberte-playwright…
dlaliberte Jun 28, 2023
589236d
Close to working.
dlaliberte Jun 29, 2023
be75002
Merge remote-tracking branch 'origin/main' into dlaliberte-playwright…
dlaliberte Jun 29, 2023
48e8359
who knows...
dlaliberte Jul 17, 2023
dcd1ac0
Merge remote-tracking branch 'origin/main' into dlaliberte-playwright…
dlaliberte Jul 17, 2023
c52a5ce
after updates
dlaliberte Jul 17, 2023
91810d1
Extend devcontainer docker compose file for playwright (#3192)
jcscottiii Jul 21, 2023
ce3a72a
Merge branch 'main' into dlaliberte-playwright-06-22
jcscottiii Jul 24, 2023
9dd5919
Fix docker compose environment
dlaliberte Jul 24, 2023
0b05ee4
ignore stuff and volume mounting
dlaliberte Jul 24, 2023
24a616a
Attempting to login reliably
dlaliberte Aug 3, 2023
68afc9c
Auto cleanups.
dlaliberte Aug 3, 2023
42d2e5c
Finally starting to work
dlaliberte Aug 8, 2023
2bdd12f
Thought it was getting better, but maybe not
dlaliberte Aug 9, 2023
713ed29
Seems reliable now
dlaliberte Aug 9, 2023
d4d1125
Merge remote-tracking branch 'origin/main' into dlaliberte-playwright…
dlaliberte Aug 9, 2023
c516049
Higher success rate
dlaliberte Aug 10, 2023
daa94f7
Add test of entering feature name. Remove new feature page content test.
dlaliberte Aug 10, 2023
824204c
Add playwright-report dir to Dockerfile.
dlaliberte Aug 10, 2023
004a942
Comment out mkdir -p /work/playwright-report line.
dlaliberte Aug 10, 2023
8eac2fa
Merge remote-tracking branch 'origin/main' into dlaliberte-playwright…
dlaliberte Aug 10, 2023
a06956b
Update screenshots.
dlaliberte Aug 10, 2023
78c76ce
Increasing timeout for entering feature name
dlaliberte Aug 10, 2023
a2f968c
try running as user (#3248)
jcscottiii Aug 11, 2023
7421ad0
Adjust login process to be more reliable.
dlaliberte Aug 11, 2023
1fa8a7a
Add 5 second delay before first test runs.
dlaliberte Aug 11, 2023
93580ab
Wait longer for first login.
dlaliberte Aug 11, 2023
1c0065f
Add auto-logout during login if already logged in
dlaliberte Aug 11, 2023
8641f65
Screenshot after login click
dlaliberte Aug 11, 2023
319ea5b
Add testName to screenshots. Upload artifacts. Fix form method="POST".
dlaliberte Aug 11, 2023
6f9b692
Add doc to README for Updating the test_html_rendering files.
dlaliberte Aug 11, 2023
1ae7a78
Maybe increase chance of login.
dlaliberte Aug 11, 2023
c1dbe24
Add a couple more delays.
dlaliberte Aug 11, 2023
cb66f85
Repeat auto-logout during login until no longer logged in, before log…
dlaliberte Aug 14, 2023
2b9b5a4
Add some more delays.
dlaliberte Aug 14, 2023
3fb1f76
Use loginButton.click(). Add delay before after-click screenshot.
dlaliberte Aug 14, 2023
b4b6793
Add console info an errors to login process.
dlaliberte Aug 14, 2023
806ffd7
Capture console messages, relay to terminal.
dlaliberte Aug 14, 2023
83f6a2a
Capture console messages properly.
dlaliberte Aug 14, 2023
4b60991
Add some more console messages during login.
dlaliberte Aug 14, 2023
75a333f
Ignore error on jsonValue.
dlaliberte Aug 14, 2023
56be7c2
Reduce most delays. Add more logs. Add top-level playwright call.
dlaliberte Aug 14, 2023
48eb1e3
More logging to figure out why login fails
dlaliberte Aug 15, 2023
e2d364d
More logging to detect if ever logged in.
dlaliberte Aug 15, 2023
f94e07a
Add more logging to each stage of testing
dlaliberte Aug 15, 2023
4146c13
Use history.replaceState before changing location.
dlaliberte Aug 15, 2023
5504826
Use location.reload() after a timeout.
dlaliberte Aug 15, 2023
2bc90a0
After login, just load home page.
dlaliberte Aug 16, 2023
a41e886
Add playwright-report to artifacts.
dlaliberte Aug 16, 2023
971f678
Turn off playwright-report during CI
dlaliberte Aug 16, 2023
f8c13be
Log response after login
dlaliberte Aug 16, 2023
a363be2
Only log first response after request that should log in.
dlaliberte Aug 16, 2023
b70e8e9
Look for 'currentuser/permissions' request/response.
dlaliberte Aug 16, 2023
8a94a63
Delay after mock_login and before reload of home page
dlaliberte Aug 16, 2023
ffb43b7
Comment out header and cookie logs.
dlaliberte Aug 17, 2023
6580391
Act like logged when in DEV_MODE and login attempted.
dlaliberte Aug 17, 2023
02b3976
Fix mock mock login.
dlaliberte Aug 17, 2023
70296fc
Undo fake mock_login.
dlaliberte Aug 18, 2023
6dc8051
rearrange docker (#3265)
jcscottiii Aug 18, 2023
3efdd5d
Updates after changes by James.
dlaliberte Aug 21, 2023
2c2b7dd
Fix previous updates.
dlaliberte Aug 21, 2023
7ab0a82
Remove pwtests-shutdown from "stop" command.
dlaliberte Aug 21, 2023
76b7f6b
Merge remote-tracking branch 'origin/main' into dlaliberte-playwright…
dlaliberte Aug 21, 2023
ca7361a
Remove pwtest from client-src/elements. (moved to packages/playwright…
dlaliberte Aug 21, 2023
ce28da8
update paths for run.sh
jcscottiii Aug 21, 2023
fca3b64
Add pwtests-shutdown to top-level stop command.
dlaliberte Aug 23, 2023
8e33385
Remove "playwright" from ./wait-for-app.sh
dlaliberte Aug 24, 2023
bfc3f5b
Add logging of user login status.
dlaliberte Aug 24, 2023
c41b165
add playwright_display for debugging
jcscottiii Aug 25, 2023
b25ff9f
add server debugging
jcscottiii Aug 25, 2023
c49401b
Remove extra "/__screenshots__" in path.
dlaliberte Aug 28, 2023
7c295e2
Update the path for test-results artifacts in the playwright.yml
dlaliberte Aug 28, 2023
f06cc57
add health checks
jcscottiii Aug 28, 2023
3b03827
fix action
jcscottiii Aug 28, 2023
3372ab4
fix run.sh
jcscottiii Aug 28, 2023
2754d84
Add beforeAll method, disabled.
dlaliberte Aug 28, 2023
c12db27
Merge remote-tracking branch 'origin/main' into dlaliberte-playwright…
dlaliberte Aug 28, 2023
5b5b6b4
Merge branch 'dlaliberte-playwright-06-22' of https://github.com/Goog…
dlaliberte Aug 28, 2023
75c8374
add dsadmin
jcscottiii Aug 29, 2023
aecef8c
Add top level pwtests command, not working yet.
dlaliberte Aug 29, 2023
6e31624
cleanup and undo changes
jcscottiii Aug 29, 2023
2385627
use transactions
jcscottiii Aug 29, 2023
28acf95
always reuse server
jcscottiii Aug 29, 2023
67156f7
temp: limit the workers
jcscottiii Aug 29, 2023
4a5fab6
Partial cleanup. Reduce timeouts and delays a bit.
dlaliberte Aug 30, 2023
d7c34a0
More cleanup. Reduce timeout/delay.
dlaliberte Aug 30, 2023
5c7013a
Merge remote-tracking branch 'origin/main' into dlaliberte-playwright…
dlaliberte Aug 30, 2023
e1e764b
Update the correct README file. Update html rendering tests.
dlaliberte Aug 30, 2023
2265b8e
Remove unnecessary changes.
dlaliberte Aug 31, 2023
5d45f53
set script-shell to fix source not found and call npm run setup (#3292)
jcscottiii Aug 31, 2023
6bd9d54
Merge remote-tracking branch 'origin/main' into dlaliberte-playwright…
dlaliberte Aug 31, 2023
cbb2b71
Reenable Google login
dlaliberte Sep 5, 2023
35c549b
Finish reenabling Google signin button. Add margin for Firefox.
dlaliberte Sep 5, 2023
4dbd2ff
Merge remote-tracking branch 'origin/main' into dlaliberte-playwright…
dlaliberte Sep 5, 2023
86ee7fd
Only show Google signin if playwright is not active
dlaliberte Sep 5, 2023
c8729b6
Preserve url on login. Use default num of workers. Retry tests in CI
dlaliberte Sep 5, 2023
557ce0f
Some updates after comments
dlaliberte Sep 6, 2023
baf0f73
optimize docker changes
jcscottiii Sep 6, 2023
4ca6546
Rename account-indicator. Shorten some delays.
dlaliberte Sep 6, 2023
42aecb1
Merge branch 'dlaliberte-playwright-06-22' of https://github.com/Goog…
dlaliberte Sep 6, 2023
f9d0955
revert the workers
jcscottiii Sep 6, 2023
4b3222c
Make playwright.yml push and pull_request triggers be like others.
dlaliberte Sep 6, 2023
be11d0c
Use getByTestId instead of locator with data-testid.
dlaliberte Sep 6, 2023
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 6 additions & 3 deletions .devcontainer/Dockerfile
Original file line number Diff line number Diff line change
@@ -1,11 +1,14 @@
# See here for image contents: https://github.com/microsoft/vscode-dev-containers/tree/v0.245.2/containers/python-3/.devcontainer/base.Dockerfile

# [Choice] Python version (use -bullseye variants on local arm64/Apple Silicon): 3, 3.10, 3.9, 3.8, 3.7, 3.6, 3-bullseye, 3.10-bullseye, 3.9-bullseye, 3.8-bullseye, 3.7-bullseye, 3.6-bullseye, 3-buster, 3.10-buster, 3.9-buster, 3.8-buster, 3.7-buster, 3.6-buster
ARG VARIANT=""
FROM mcr.microsoft.com/vscode/devcontainers/python:0-${VARIANT}
# Update 'VARIANT' to pick a version of Python: 3, 3.10, 3.9, 3.8, 3.7, 3.6
# Append -bullseye or -buster to pin to an OS version.
# Use -bullseye variants on local arm64/Apple Silicon.
ARG VARIANT="3.11-bullseye"
FROM mcr.microsoft.com/vscode/devcontainers/python:0-${VARIANT} as ide

# [Choice] Node.js version: none, lts/*, 16, 14, 12, 10
ARG NODE_VERSION="none"
ARG NODE_VERSION="18"
RUN if [ "${NODE_VERSION}" != "none" ]; then su vscode -c "umask 0002 && . /usr/local/share/nvm/nvm.sh && nvm install ${NODE_VERSION} 2>&1"; fi

RUN apt-get update && export DEBIAN_FRONTEND=noninteractive \
Expand Down
7 changes: 4 additions & 3 deletions .devcontainer/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,8 @@ You can click on the appropriate badge to get the environment setup:
| [Visual Studio Code Remote - Containers](https://code.visualstudio.com/docs/remote/create-dev-container) | [![Open in Remote - Containers](https://img.shields.io/static/v1?label=Remote%20-%20Containers&message=Open&color=blue&logo=visualstudiocode)](https://vscode.dev/redirect?url=vscode://ms-vscode-remote.remote-containers/cloneInVolume?url=https://github.com/GoogleChrome/chromium-dashboard) | Locally: Docker and Visual Studio Code | Runs locally. Have more resources | Need to have VS Code and Docker setup |
| [GitHub Codespaces](https://docs.github.com/en/enterprise-cloud@latest/codespaces) | <ul><li>East US[![Open in GitHub Codespaces](https://github.com/codespaces/badge.svg)](https://github.com/codespaces/new?hide_repo_select=true&ref=main&repo=8633551&machine=standardLinux32gb&location=EastUs&devcontainer_path=.devcontainer%2Fdevcontainer.json)</li><li>West US[![Open in GitHub Codespaces](https://github.com/codespaces/badge.svg)](https://github.com/codespaces/new?hide_repo_select=true&ref=main&repo=8633551&machine=standardLinux32gb&location=WestUs2&devcontainer_path=.devcontainer%2Fdevcontainer.json)</li></ul> | Access to Codespaces | Nothing to install locally | Limited resources: 4 cores, 8GB RAM, 32GB Storage |

Upon creating the devcontainer by clicking the appropriate badge, `npm run setup` is ran automatically.
Upon creating the devcontainer by clicking the appropriate badge, `npm run setup`
is run automatically.

*Note*: Sometimes when you open the terminal, it won't automatically activate
the python environment. Either 1) Open a new terminal (usually it works the
Expand Down Expand Up @@ -61,11 +62,11 @@ There are other ports needed for the IDE itself but mainly, you only need to wat
## Upgrading versions of Node and Python

In the docker-compose.yml file:
- For Python: Change the `VARIANT` argument to the appropirate [variant](https://github.com/microsoft/vscode-dev-containers/tree/main/containers/python-3)
- For Python: Change the `VARIANT` argument to the appropriate [variant](https://github.com/microsoft/vscode-dev-containers/tree/main/containers/python-3)
- For Node: Change the `NODE_VERSION` to the appropriate version used by [nvm](https://github.com/nvm-sh/nvm)

## Troubleshooting

When there are depedency changes and environment changes in requirements.txt or package.json, the Devcontainer needs to be rebuilt:
When there are dependency changes and environment changes in requirements.txt or package.json, the Devcontainer needs to be rebuilt:
- `rm -rf node_modules cs-env` in terminal
- View -> Command Palette -> Rebuild container without cache
17 changes: 17 additions & 0 deletions .devcontainer/db-docker-compose.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,17 @@
version: '3.8'

services:
db:
image: google/cloud-sdk:402.0.0-emulators
restart: unless-stopped
command: gcloud beta emulators datastore start --project=cr-status-staging --host-port 0.0.0.0:15606 --no-store-on-disk --use-firestore-in-datastore-mode

dsadmin:
image: ghcr.io/remko/dsadmin:v0.18.1
entrypoint: /ko-app/dsadmin -port 8888
depends_on:
- db
network_mode: service:db
environment:
DATASTORE_PROJECT_ID: cr-status-staging
DATASTORE_EMULATOR_HOST: "db:15606"
19 changes: 15 additions & 4 deletions .devcontainer/devcontainer.json
Original file line number Diff line number Diff line change
Expand Up @@ -2,8 +2,11 @@
// https://github.com/microsoft/vscode-dev-containers/tree/v0.245.2/containers/python-3
{
"name": "Chromium Dashboard",
"dockerComposeFile": "docker-compose.yml",
"service": "app",
"dockerComposeFile": [
"docker-compose.yml",
"db-docker-compose.yml"
],
"service": "ide",
"workspaceFolder": "/workspace",

// Configure tool-specific properties.
Expand Down Expand Up @@ -41,7 +44,8 @@
"extensions": [
"ms-python.python",
"bierner.lit-html",
"42Crunch.vscode-openapi"
"42Crunch.vscode-openapi",
"ms-playwright.playwright"
]
}
},
Expand All @@ -64,5 +68,12 @@
"postCreateCommand": "npm run clean-setup; . cs-env/bin/activate; npm run setup; python -m pip install pylint==2.13.4",

// Comment out to connect as root instead. More info: https://aka.ms/vscode-remote/containers/non-root.
"remoteUser": "vscode"
"remoteUser": "vscode",

"features": {
// https://github.com/devcontainers/features/tree/main/src/docker-in-docker
"ghcr.io/devcontainers/features/docker-in-docker:2": {
"dockerDashComposeVersion": "v2"
}
}
}
26 changes: 2 additions & 24 deletions .devcontainer/docker-compose.yml
Original file line number Diff line number Diff line change
@@ -1,18 +1,11 @@
version: '3.8'

services:
app:
ide:
build:
context: ..
dockerfile: .devcontainer/Dockerfile
args:
# Update 'VARIANT' to pick a version of Python: 3, 3.10, 3.9, 3.8, 3.7, 3.6
# Append -bullseye or -buster to pin to an OS version.
# Use -bullseye variants on local arm64/Apple Silicon.
VARIANT: 3.10-bullseye
# Optional Node.js version to install
NODE_VERSION: "18"

target: ide
volumes:
- ..:/workspace:cached

Expand All @@ -25,18 +18,3 @@ services:
environment:
DATASTORE_PROJECT_ID: cr-status-staging
DATASTORE_EMULATOR_HOST: localhost:15606

db:
image: google/cloud-sdk:402.0.0-emulators
restart: unless-stopped
command: gcloud beta emulators datastore start --project=cr-status-staging --host-port 0.0.0.0:15606 --no-store-on-disk --consistency=1.0

dsadmin:
image: ghcr.io/remko/dsadmin:v0.18.1
entrypoint: /ko-app/dsadmin -port 8888
depends_on:
- db
network_mode: service:db
environment:
DATASTORE_PROJECT_ID: cr-status-staging
DATASTORE_EMULATOR_HOST: "db:15606"
10 changes: 10 additions & 0 deletions .dockerignore
Original file line number Diff line number Diff line change
@@ -0,0 +1,10 @@
cs-env
node_modules
test-results
playwright-report/
playwright/.cache/

# Web test runner coverage report
coverage

__pycache__
6 changes: 3 additions & 3 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -45,14 +45,14 @@ jobs:
- name: installation
run: |
npm install -g gulp
npm ci
npm run deps
npm config set script-shell bash --global
npm run setup

- name: lint
run: npm run lint

- name: mypy
run: mypy --ignore-missing-imports --exclude cs-env/ --exclude gen/ --exclude appengine_config.py .
run: npm run mypy

- name: test
run: npm test
Expand Down
27 changes: 27 additions & 0 deletions .github/workflows/playwright.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,27 @@
name: Playwright Tests
on:
push:
dlaliberte marked this conversation as resolved.
Show resolved Hide resolved
branches-ignore:
- "dependabot/**"
- "daily-test-sync"

jobs:
playwright:
name: 'Playwright Tests'
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v3

# Steps for Playwright testing with container
- uses: actions/setup-node@v3
with:
node-version: 18
- name: Run your tests
timeout-minutes: 15
run: npm run pwtests-shutdown --workspace=playwright; npm run test --workspace=playwright
- uses: actions/upload-artifact@v3
if: failure()
with:
name: test-results
path: packages/playwright/test-results/
retention-days: 30
6 changes: 2 additions & 4 deletions .github/workflows/web_tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -55,11 +55,9 @@ jobs:
run: |
npm install -g gulp

- run: npm ci
- run: npm run deps
- run: npm config set script-shell bash --global
- run: npm run setup
- run: npm run build --if-present

- run: npx playwright install-deps

- name: Run web test runner
run: npm run webtest
10 changes: 8 additions & 2 deletions .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,8 @@
*.pyc
*.swp
.sass-cache
.vscode
.vscode/*
!.vscode/launch.json
*oauth2.data
.fcm_server_key
*.iml
Expand All @@ -18,7 +19,7 @@ env_vars.yaml
node_modules

# Generated files
# static/css/
static/css/
static/dist/
static/js/

Expand All @@ -35,5 +36,10 @@ coverage
# venv directory
cs-env

# playwright
test-results/
/playwright-report/
/playwright/.cache/

# API Key for local development
ot_api_key.txt
24 changes: 24 additions & 0 deletions .vscode/launch.json
Original file line number Diff line number Diff line change
@@ -0,0 +1,24 @@
{
// Use IntelliSense to learn about possible attributes.
// Hover to view descriptions of existing attributes.
// For more information, visit: https://go.microsoft.com/fwlink/?linkid=830387
"version": "0.2.0",
"configurations": [
{
"name": "Python: Remote Attach",
"type": "python",
"request": "attach",
"connect": {
"host": "localhost",
"port": 5678
},
"pathMappings": [
{
"localRoot": "${workspaceFolder}",
"remoteRoot": "/workspace"
}
],
"justMyCode": false
}
]
}
41 changes: 41 additions & 0 deletions Dockerfile
Original file line number Diff line number Diff line change
@@ -0,0 +1,41 @@
ARG PYTHON_VARIANT=3.11-bullseye
FROM python:${PYTHON_VARIANT} as app

RUN apt-get update && export DEBIAN_FRONTEND=noninteractive \
# openapi-generator-cli dependencies
&& apt-get install -y openjdk-17-jre

RUN groupadd -g 1001 appuser && \
useradd appuser -u 1001 -g 1001 -m -d /home/appuser -s /bin/bash
USER appuser

# Install node
RUN mkdir /home/appuser/.nvm
ENV NVM_DIR /home/appuser/.nvm
# Need the full major.minor.patch version for NODE_VERISON
# https://nodejs.org/en/download/releases
ENV NODE_VERSION 18.17.1

RUN curl https://raw.githubusercontent.com/nvm-sh/nvm/v0.39.4/install.sh | bash \
&& . $NVM_DIR/nvm.sh \
&& nvm install $NODE_VERSION \
&& nvm alias default $NODE_VERSION \
&& nvm use default \
&& npm config set script-shell bash


ENV NODE_PATH $NVM_DIR/v$NODE_VERSION/lib/node_modules
dlaliberte marked this conversation as resolved.
Show resolved Hide resolved
ENV PATH="${PATH}:$NVM_DIR/versions/node/v$NODE_VERSION/bin"

# Reload shell
SHELL ["/bin/bash", "-c"]


WORKDIR /workspace
COPY --chown=appuser:appuser openapi /workspace/openapi/
COPY --chown=appuser:appuser gen /workspace/gen/
# Cache the infrequent but time consuming changes early
COPY --chown=appuser:appuser requirements.txt requirements.dev.txt package.json package-lock.json /workspace/
RUN npm install -g gulp-cli && npm run setup && cs-env/bin/python -m pip install debugpy==1.6.7.post1
# Copy the rest
COPY --chown=appuser:appuser . /workspace
26 changes: 25 additions & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -62,7 +62,31 @@ npm test

This will start a local datastore emulator, run unit tests, and then shut down the emulator.

There are some developing information in developer-documentation.md.
To update test_html_rendering.html, modify the `test_html_rendering` method in
the corresponding test file, uncomment the line that looks like:

```python
# TESTDATA.make_golden(template_text, 'test_html_rendering.html')
```

Then run the test again (and maybe one more time), and then you can revert
your change of the test files.

To run the Playwright visual tests (aka end-to-end tests), the command to use is:
```bash
npm run pwtests --packages/playwright
```

If there are errors, or if you need to update any of the screeenshot images,
you can update all images for all tests with:
```bash
npm run pwtests-update --packages/playwright
```

Then check the `packages/playwright/test-results` directory for any changes.


There is some additional information for developers in developer-documentation.md.

### Origin Trials
To test the functionality of this application locally that interacts with data from the Origin Trials API, an API key will need to be acquired. To do this, run the following command:
Expand Down
4 changes: 4 additions & 0 deletions client-src/elements/chromedash-app.js
Original file line number Diff line number Diff line change
Expand Up @@ -100,6 +100,7 @@ class ChromedashApp extends LitElement {
loading: {type: Boolean},
appTitle: {type: String},
googleSignInClientId: {type: String},
devMode: {type: String},
currentPage: {type: String},
bannerMessage: {type: String},
bannerTime: {type: Number},
Expand All @@ -117,6 +118,7 @@ class ChromedashApp extends LitElement {
this.loading = true;
this.appTitle = '';
this.googleSignInClientId = '',
this.devMode = '';
this.currentPage = '';
this.bannerMessage = '';
this.bannerTime = null;
Expand Down Expand Up @@ -543,6 +545,7 @@ class ChromedashApp extends LitElement {
<chromedash-header
.user=${this.user}
.appTitle=${this.appTitle}
.devMode=${this.devMode}
.googleSignInClientId=${this.googleSignInClientId}
.currentPage=${this.currentPage}
@drawer-clicked=${this.handleShowDrawer}>
Expand All @@ -557,6 +560,7 @@ class ChromedashApp extends LitElement {
.user=${this.user}
.currentPage=${this.currentPage}
?defaultOpen=${true}
.devMode=${this.devMode}
.googleSignInClientId=${this.googleSignInClientId}>
</chromedash-drawer>
</div>
Expand Down
Loading
Loading