Skip to content

Commit 7e30969

Browse files
authored
Bug Fix: Start using expect_angular_legacy_material_checkbox (#6547)
## Motivation for features / changes #6493 Broke our internal sync Googlers see [cl/557499468](http://cl/557499468) The issue is related to our internal Angular component migration. New components are not allowed to rely on the legacy components. Due to this the new component added in #6493 uses the non legacy checkbox. However, our dependency remapping always remaps dependencies on material_checkbox to legacy_material_checkbox. To fix this I am changing the name of the remapping rule in [cl/557515881](http://cl/557515881) and adjusting all the OSS usage EXCEPT for the usage in filter_dialog. [cl/557590954](http://cl/557590954) is also required to map the testing dependency needed to use `MatCheckboxHarness`
1 parent 6627a7f commit 7e30969

File tree

6 files changed

+26
-7
lines changed
  • tensorboard/webapp

6 files changed

+26
-7
lines changed

tensorboard/webapp/angular/BUILD

Lines changed: 18 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -87,6 +87,24 @@ tf_ts_library(
8787
],
8888
)
8989

90+
# This is a dummy rule used as a @angular/material/legeacy_checkbox dependency.
91+
tf_ts_library(
92+
name = "expect_angular_legacy_material_checkbox",
93+
srcs = [],
94+
deps = [
95+
"@npm//@angular/material",
96+
],
97+
)
98+
99+
# This is a dummy rule used as a @angular/material/checkbox/testing dependency.
100+
tf_ts_library(
101+
name = "expect_angular_material_checkbox_testing",
102+
srcs = [],
103+
deps = [
104+
"@npm//@angular/material",
105+
],
106+
)
107+
90108
# This is a dummy rule used as a @angular/material/button dependency.
91109
tf_ts_library(
92110
name = "expect_angular_material_button",

tensorboard/webapp/metrics/views/right_pane/BUILD

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -27,9 +27,9 @@ tf_ng_module(
2727
deps = [
2828
"//tensorboard/webapp:app_state",
2929
"//tensorboard/webapp:selectors",
30+
"//tensorboard/webapp/angular:expect_angular_legacy_material_checkbox",
3031
"//tensorboard/webapp/angular:expect_angular_material_button",
3132
"//tensorboard/webapp/angular:expect_angular_material_button_toggle",
32-
"//tensorboard/webapp/angular:expect_angular_material_checkbox",
3333
"//tensorboard/webapp/angular:expect_angular_material_icon",
3434
"//tensorboard/webapp/angular:expect_angular_material_select",
3535
"//tensorboard/webapp/angular:expect_angular_material_slider",
@@ -58,8 +58,8 @@ tf_ts_library(
5858
"//tensorboard/webapp:app_state",
5959
"//tensorboard/webapp:selectors",
6060
"//tensorboard/webapp/angular:expect_angular_core_testing",
61+
"//tensorboard/webapp/angular:expect_angular_legacy_material_checkbox",
6162
"//tensorboard/webapp/angular:expect_angular_material_button_toggle",
62-
"//tensorboard/webapp/angular:expect_angular_material_checkbox",
6363
"//tensorboard/webapp/angular:expect_angular_material_select",
6464
"//tensorboard/webapp/angular:expect_angular_material_slider",
6565
"//tensorboard/webapp/angular:expect_angular_platform_browser_animations",

tensorboard/webapp/metrics/views/right_pane/scalar_column_editor/BUILD

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ tf_ng_module(
2626
deps = [
2727
"//tensorboard/webapp:app_state",
2828
"//tensorboard/webapp:selectors",
29-
"//tensorboard/webapp/angular:expect_angular_material_checkbox",
29+
"//tensorboard/webapp/angular:expect_angular_legacy_material_checkbox",
3030
"//tensorboard/webapp/angular:expect_angular_material_tabs",
3131
"//tensorboard/webapp/metrics:types",
3232
"//tensorboard/webapp/metrics/actions",

tensorboard/webapp/runs/views/runs_table/BUILD

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -81,8 +81,8 @@ tf_ng_module(
8181
"//tensorboard/webapp:selectors",
8282
"//tensorboard/webapp/alert",
8383
"//tensorboard/webapp/alert/actions",
84+
"//tensorboard/webapp/angular:expect_angular_legacy_material_checkbox",
8485
"//tensorboard/webapp/angular:expect_angular_material_button",
85-
"//tensorboard/webapp/angular:expect_angular_material_checkbox",
8686
"//tensorboard/webapp/angular:expect_angular_material_dialog",
8787
"//tensorboard/webapp/angular:expect_angular_material_form_field",
8888
"//tensorboard/webapp/angular:expect_angular_material_icon",
@@ -140,7 +140,7 @@ tf_ts_library(
140140
"//tensorboard/webapp/alert/actions",
141141
"//tensorboard/webapp/angular:expect_angular_cdk_overlay",
142142
"//tensorboard/webapp/angular:expect_angular_core_testing",
143-
"//tensorboard/webapp/angular:expect_angular_material_checkbox",
143+
"//tensorboard/webapp/angular:expect_angular_legacy_material_checkbox",
144144
"//tensorboard/webapp/angular:expect_angular_material_dialog",
145145
"//tensorboard/webapp/angular:expect_angular_material_input",
146146
"//tensorboard/webapp/angular:expect_angular_material_menu",

tensorboard/webapp/settings/_views/BUILD

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -18,8 +18,8 @@ tf_ng_module(
1818
],
1919
deps = [
2020
"//tensorboard/webapp:tb_polymer_interop_types",
21+
"//tensorboard/webapp/angular:expect_angular_legacy_material_checkbox",
2122
"//tensorboard/webapp/angular:expect_angular_material_button",
22-
"//tensorboard/webapp/angular:expect_angular_material_checkbox",
2323
"//tensorboard/webapp/angular:expect_angular_material_dialog",
2424
"//tensorboard/webapp/angular:expect_angular_material_icon",
2525
"//tensorboard/webapp/angular:expect_angular_material_input",
@@ -44,8 +44,8 @@ tf_ts_library(
4444
":_views",
4545
"//tensorboard/webapp/angular:expect_angular_cdk_overlay",
4646
"//tensorboard/webapp/angular:expect_angular_core_testing",
47+
"//tensorboard/webapp/angular:expect_angular_legacy_material_checkbox",
4748
"//tensorboard/webapp/angular:expect_angular_material_button",
48-
"//tensorboard/webapp/angular:expect_angular_material_checkbox",
4949
"//tensorboard/webapp/angular:expect_angular_material_dialog",
5050
"//tensorboard/webapp/angular:expect_angular_material_icon",
5151
"//tensorboard/webapp/angular:expect_angular_material_input",

tensorboard/webapp/widgets/data_table/BUILD

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -182,6 +182,7 @@ tf_ts_library(
182182
"//tensorboard/webapp/angular:expect_angular_cdk_testing_testbed",
183183
"//tensorboard/webapp/angular:expect_angular_core_testing",
184184
"//tensorboard/webapp/angular:expect_angular_material_checkbox",
185+
"//tensorboard/webapp/angular:expect_angular_material_checkbox_testing",
185186
"//tensorboard/webapp/angular:expect_angular_platform_browser_animations",
186187
"//tensorboard/webapp/testing:mat_icon",
187188
"//tensorboard/webapp/widgets/custom_modal",

0 commit comments

Comments
 (0)