From d54c17456d0357709218d9932321756e2902c93b Mon Sep 17 00:00:00 2001
From: Andy O'Neill <andy@scratch.mit.edu>
Date: Mon, 5 Dec 2022 13:04:26 -0500
Subject: [PATCH] test: add integration test that tests monitor labels after
 switching languages

---
 test/fixtures/monitor-variable.sb3    | Bin 0 -> 1004 bytes
 test/integration/localization.test.js |  36 ++++++++++++++++++++++++--
 2 files changed, 34 insertions(+), 2 deletions(-)
 create mode 100644 test/fixtures/monitor-variable.sb3

diff --git a/test/fixtures/monitor-variable.sb3 b/test/fixtures/monitor-variable.sb3
new file mode 100644
index 0000000000000000000000000000000000000000..c1f0bc592cecf6c191a5f27df145b6186e171134
GIT binary patch
literal 1004
zcmWIWW@h1HU|`^2$nI+mUF0vm(wm8ap^$}vfd?pBP?VpQnp~onRh*x<G$b(pvVp)q
z?TM8ymrStC-4yziaoMJWTc+BUNI72)iTOQihE-XLb*%Tx`sZ`f8U$9UeH6Mocfa-h
z^6!?>k7nu~ur{i?A@p+DJW&U?du9T{XSKGRv=f^wx_Q!BQH3(^eRbCH`%c%LXb%@Z
zy?yV)UoYP1RmSEnN*B1MT))e(_RdikPw{oP7Flj@<2+aQ`_zZj^y1*q1@rwYl69++
zAJ+&=En>WI>-W0dX3u-{TTM&5)^a_&*}75mbH#%&*0veN5A<1o#eBJJbC%^S`%5Ps
zvCnn;{f!qKC>A)}qqsxJa?M}v4JY^QFr8PK`9<P4OSQk--=2T6f1TW3PIxDm{K#jq
z>kHYpQv-EZPDyG?lHOn*){q-nwIa*eO+91X5{BaJnPsMj6F=}uK0Tj!r=OqMU{=eV
zt@}+cOEpZt?j!EI+yAbHXx_S>Y4>jA-A<I?+cJ6A3{$=w>k|w@3+FVjW=}cZu=1zf
z7k-0nhganVuG@C$@bWU*hsP8oueP|b*m*uYJuQrLlSXa!x`}Q+W^c<?=jOC-7M~w{
z;;i42yvDi5uiag;q*`Bg`vdDgdhdSxFi>t~p1E+<dlj3;zou=@SNNXq?RmOWFW|vb
z#!ZK(<(U15-?qc+PVtSli*qMvebBM-VZW;NqMb)*)`br_+9j8k%zqMD!N%Gp$SnLh
zR_kcr=N7lUoR5R5AD=S6sOlB!D$$X*>|*VdOOdnA`}Hr{yesU3<g@Tg^3R(urxXZ%
zU1xXCZ{1^$ld3NA4f<yTP4w2Td|<KkaDnx;d7D`OG3^rBbo(z$07|0kI48h38JO5k
z0kH}YC#M)0ni`s<7?>Ixrlq8%B^enR8JMIRq$ZoDrdXIH>J^uz=lXi{H7khRsqVVI
zo&Cr<lS7L<{Vxb_@oEV<dSLa1+t;I}`=%Mr%$L5-7$&^v^Jb@$s!m0=k*%Ulrg<B-
zT)sN9<YT1b<P8g(d{1AtaBrQKk=f#*`LX1g9nT`I(?L0h-IttPJbV8CrcLXzH|}1+
z-SF**M9k+$%roWW@^_Rz^qb4d=5GFv#m@54vU<)0AMv-}z3UkQycwB97;xt$pwEy1
ia?V26i5?&bZKc4xNlfqrc(byBbTI*850Lg^2JrxZ{G%%X

literal 0
HcmV?d00001

diff --git a/test/integration/localization.test.js b/test/integration/localization.test.js
index 1f66c160c..12554507f 100644
--- a/test/integration/localization.test.js
+++ b/test/integration/localization.test.js
@@ -4,12 +4,13 @@ import SeleniumHelper from '../helpers/selenium-helper';
 const {
     clickText,
     clickXpath,
+    findByText,
+    findByXpath,
     getDriver,
     getLogs,
     loadUri,
-    scope,
     rightClickText,
-    findByText
+    scope
 } = new SeleniumHelper();
 
 const uri = path.resolve(__dirname, '../../build/index.html');
@@ -74,4 +75,35 @@ describe('Localization', () => {
         const logs = await getLogs();
         await expect(logs).toEqual([]);
     });
+
+    // Regression test for ENA-142, monitor can lag behind language selection
+    test('Monitor labels update on locale change', async () => {
+        await loadUri(uri);
+        await clickText('File');
+        await clickText('Load from your computer');
+        const input = await findByXpath('//input[@accept=".sb,.sb2,.sb3"]');
+        await input.sendKeys(path.resolve(__dirname, '../fixtures/monitor-variable.sb3'));
+
+        // Monitors are present
+        await findByText('username', scope.monitors);
+        await findByText('language', scope.monitors);
+
+        // Change locale to ja
+        await clickXpath('//*[@aria-label="language selector"]');
+        await clickText('日本語');
+
+        // Monitor labels updated
+        await findByText('ユーザー名', scope.monitors);
+        await findByText('言語', scope.monitors);
+
+        // Change locale to ja-hira
+        await clickText('にほんご');
+
+        // Monitor labels updated
+        await findByText('ユーザーめい', scope.monitors);
+        await findByText('げんご', scope.monitors);
+
+        const logs = await getLogs();
+        await expect(logs).toEqual([]);
+    });
 });