Skip to content

Commit ec25060

Browse files
authored
Fluent: test fluentui bundles (#5142)
* Fluent: test fluentui bundles * Remove unnecessary serve instructions * Add precommit and fix issues * Rename package * Add build artifacts to docker context * Sort * Bring back botframework-webchat-fluent-theme.production.min.js.map * Reorder static entries * Fix extra space * Fix duplicate string * Apply customizations to the dark theme * Fix color
1 parent 2240f2a commit ec25060

21 files changed

+3916
-898
lines changed

.dockerignore

+2
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,9 @@
22
!/__tests__
33
/__tests__/__image_snapshots__
44
!/packages/bundle/dist
5+
!/packages/fluent-theme/dist
56
!/packages/playground/build
7+
!/packages/test/fluent-bundle/dist
68
!/packages/test/harness/dist
79
!/packages/test/page-object/dist
810
!/serve-test.json

.github/workflows/pull-request-validation.yml

+3-2
Original file line numberDiff line numberDiff line change
@@ -53,16 +53,17 @@ jobs:
5353
- name: Pack Docker artifact
5454
run: |
5555
zip docker.zip -r@ <<EOF
56-
./__tests__/
5756
./babel-jest-config.js
5857
./babel-passthru-transformer.js
5958
./babel.config.json
6059
./docker-compose-wsl2.yml
6160
./jest.config.js
62-
./package.json
6361
./package-lock.json
62+
./package.json
63+
./__tests__/
6464
./packages/bundle/dist/
6565
./packages/fluent-theme/dist/
66+
./packages/test/fluent-bundle/dist/
6667
./packages/test/harness/
6768
./packages/test/page-object/dist/
6869
./serve-test.json

__tests__/html/fluentTheme/fluentThemeFallback.dark.html

+33-448
Large diffs are not rendered by default.

__tests__/html/fluentTheme/fluentThemeFallback.html

+14-448
Large diffs are not rendered by default.

docker-compose-wsl2.yml

+1
Original file line numberDiff line numberDiff line change
@@ -46,6 +46,7 @@ services:
4646
- ./__tests__/html/:/var/web/__tests__/html/
4747
- ./packages/bundle/dist/:/var/web/packages/bundle/dist/
4848
- ./packages/fluent-theme/dist/:/var/web/packages/fluent-theme/dist/
49+
- ./packages/test/fluent-bundle/dist/:/var/web/packages/test/fluent-bundle/dist/
4950
- ./packages/test/harness/dist/:/var/web/packages/test/harness/dist/
5051
- ./packages/test/page-object/dist/:/var/web/packages/test/page-object/dist/
5152

packages/test/dev-server/src/index.js

+40
Original file line numberDiff line numberDiff line change
@@ -61,6 +61,46 @@ const resolveFromRepositoryRoot = resolveFromProjectRoot.bind(undefined, '../../
6161
)
6262
);
6363

64+
app.use(
65+
'/__dist__/fluent-bundle.development.js',
66+
express.static(
67+
resolve(
68+
fileURLToPath(import.meta.url),
69+
'../../../../test/fluent-bundle/dist/fluent-bundle.development.js'
70+
)
71+
)
72+
);
73+
74+
app.use(
75+
'/__dist__/fluent-bundle.development.js.map',
76+
express.static(
77+
resolve(
78+
fileURLToPath(import.meta.url),
79+
'../../../../test/fluent-bundle/dist/fluent-bundle.development.js.map'
80+
)
81+
)
82+
);
83+
84+
app.use(
85+
'/__dist__/fluent-bundle.production.min.js',
86+
express.static(
87+
resolve(
88+
fileURLToPath(import.meta.url),
89+
'../../../../test/fluent-bundle/dist/fluent-bundle.production.min.js'
90+
)
91+
)
92+
);
93+
94+
app.use(
95+
'/__dist__/fluent-bundle.production.min.js.map',
96+
express.static(
97+
resolve(
98+
fileURLToPath(import.meta.url),
99+
'../../../../test/fluent-bundle/dist/fluent-bundle.production.min.js.map'
100+
)
101+
)
102+
);
103+
64104
// /__dist__/ will be serve from ESBuild development server.
65105
app.use(
66106
'/__dist__/',
+12
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,12 @@
1+
extends:
2+
- ../../../.eslintrc.production.yml
3+
- ../../../.eslintrc.react.yml
4+
5+
# TODO: #3212 When we move to React Native, we should disable this.
6+
env:
7+
browser: true
8+
9+
rules:
10+
react/destructuring-assignment: off
11+
react/require-default-props: off
12+
dot-notation: off
+4
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,4 @@
1+
/*.tgz
2+
/dist
3+
/lib
4+
/node_modules
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,11 @@
1+
arrowParens: avoid
2+
bracketSameLine: false
3+
bracketSpacing: true
4+
endOfLine: auto
5+
printWidth: 120
6+
proseWrap: preserve
7+
quoteProps: as-needed
8+
semi: true
9+
singleQuote: true
10+
tabWidth: 2
11+
trailingComma: none

0 commit comments

Comments
 (0)