From a1092849fca0092f752e031c3aa688f2b45e5534 Mon Sep 17 00:00:00 2001
From: Vincent Hardouin <vincent.hardouin@pix.fr>
Date: Mon, 16 Jan 2023 11:19:48 +0100
Subject: [PATCH] feat: reoder language-switcher choice

---
 config/localization/pix-site.js               | 12 ++--
 .../__snapshots__/LocaleSwitcher.test.js.snap | 56 +++++++++----------
 tests/config/localization/pix-site.test.js    | 12 ++--
 3 files changed, 40 insertions(+), 40 deletions(-)

diff --git a/config/localization/pix-site.js b/config/localization/pix-site.js
index 947ed806a..5a5910a06 100644
--- a/config/localization/pix-site.js
+++ b/config/localization/pix-site.js
@@ -52,18 +52,18 @@ export const localization = {
         },
       ],
     },
-    {
-      name: 'france',
-      localeCode: 'fr-fr',
-      icon: 'flag-fr.svg',
-      subtitle: null,
-    },
     {
       name: 'fwb-acronym',
       localeCode: 'fr-be',
       icon: 'flag-be.svg',
       subtitle: 'fwb',
     },
+    {
+      name: 'france',
+      localeCode: 'fr-fr',
+      icon: 'flag-fr.svg',
+      subtitle: null,
+    },
   ],
   locales: availableLocales,
   localesForCurrentSite: getLocalesForCurrentSite(),
diff --git a/tests/components/__snapshots__/LocaleSwitcher.test.js.snap b/tests/components/__snapshots__/LocaleSwitcher.test.js.snap
index f5924ef0f..ac7ee8a7b 100644
--- a/tests/components/__snapshots__/LocaleSwitcher.test.js.snap
+++ b/tests/components/__snapshots__/LocaleSwitcher.test.js.snap
@@ -85,18 +85,18 @@ exports[`Component: LocaleSwitcher when site is pix-pro should render the locale
       </div>
     </li>
     <li
-      class="locale-switcher__dropdown-menu--active"
+      class=""
     >
       <div
         class="locale-switcher__locale"
       >
         <a
-          href="http://example.fr/"
+          href="http://example.org/fr-be"
         >
           <img
             alt=""
             class="locale-switcher__img"
-            src="/images/flag-fr.svg"
+            src="/images/flag-be.svg"
           />
            
           <div
@@ -106,24 +106,30 @@ exports[`Component: LocaleSwitcher when site is pix-pro should render the locale
               
             </div>
              
-            <!---->
+            <div
+              class="locale-switcher__subtitle"
+            >
+              
+              
+            
+            </div>
           </div>
         </a>
       </div>
     </li>
     <li
-      class=""
+      class="locale-switcher__dropdown-menu--active"
     >
       <div
         class="locale-switcher__locale"
       >
         <a
-          href="http://example.org/fr-be"
+          href="http://example.fr/"
         >
           <img
             alt=""
             class="locale-switcher__img"
-            src="/images/flag-be.svg"
+            src="/images/flag-fr.svg"
           />
            
           <div
@@ -133,13 +139,7 @@ exports[`Component: LocaleSwitcher when site is pix-pro should render the locale
               
             </div>
              
-            <div
-              class="locale-switcher__subtitle"
-            >
-              
-              
-            
-            </div>
+            <!---->
           </div>
         </a>
       </div>
@@ -237,18 +237,18 @@ exports[`Component: LocaleSwitcher when site is pix-site should render the local
       </div>
     </li>
     <li
-      class="locale-switcher__dropdown-menu--active"
+      class=""
     >
       <div
         class="locale-switcher__locale"
       >
         <a
-          href="http://example.fr/"
+          href="http://example.org/fr-be"
         >
           <img
             alt=""
             class="locale-switcher__img"
-            src="/images/flag-fr.svg"
+            src="/images/flag-be.svg"
           />
            
           <div
@@ -258,24 +258,30 @@ exports[`Component: LocaleSwitcher when site is pix-site should render the local
               
             </div>
              
-            <!---->
+            <div
+              class="locale-switcher__subtitle"
+            >
+              
+              
+            
+            </div>
           </div>
         </a>
       </div>
     </li>
     <li
-      class=""
+      class="locale-switcher__dropdown-menu--active"
     >
       <div
         class="locale-switcher__locale"
       >
         <a
-          href="http://example.org/fr-be"
+          href="http://example.fr/"
         >
           <img
             alt=""
             class="locale-switcher__img"
-            src="/images/flag-be.svg"
+            src="/images/flag-fr.svg"
           />
            
           <div
@@ -285,13 +291,7 @@ exports[`Component: LocaleSwitcher when site is pix-site should render the local
               
             </div>
              
-            <div
-              class="locale-switcher__subtitle"
-            >
-              
-              
-            
-            </div>
+            <!---->
           </div>
         </a>
       </div>
diff --git a/tests/config/localization/pix-site.test.js b/tests/config/localization/pix-site.test.js
index 48f1c03e4..70d5b4ce3 100644
--- a/tests/config/localization/pix-site.test.js
+++ b/tests/config/localization/pix-site.test.js
@@ -74,18 +74,18 @@ describe('#localization', () => {
           },
         ],
       },
-      {
-        name: 'france',
-        localeCode: 'fr-fr',
-        icon: 'flag-fr.svg',
-        subtitle: null,
-      },
       {
         name: 'fwb-acronym',
         localeCode: 'fr-be',
         icon: 'flag-be.svg',
         subtitle: 'fwb',
       },
+      {
+        name: 'france',
+        localeCode: 'fr-fr',
+        icon: 'flag-fr.svg',
+        subtitle: null,
+      },
     ]
 
     // when & then