Put four logos where they belong

Really is an integrator, not a global partner -- it was in :global because
its link goes to Open Collective, which said more about how it pays us than
about what it does. AnyCam serves more than Russia, so it leaves the
territory-gated list for :integrators proper, next to GoodCam, and is now
shown to everyone rather than only to Russian-speaking visitors.

TUDSaT stays in :education, which was the reading rather than the
instruction until now.

Expo Electronica is a trade show and belongs in neither. It gets a group of
its own, :exhibitions, which no page composes: kept in the system with its
logo and link intact, rendered nowhere, ready for the exhibitions page that
may come. That is the same treatment /merchandise got, and like that one it
needs holding in place -- adding a group to a row is a word. A test asserts
it appears on none of the seven pages that carry a wall.

RU_INTEGRATORS is down to fifteen and the general integrator row is up to
four, so English finally has a row that is not the same three logos.

Author: openipc-ai

app/helpers/pages_helper.rb

@@ -19,8 +19,7 @@ module PagesHelper
     # Who hosts our money and our code.
     global: [
       { name: 'Open Source Collective', url: 'https://www.oscollective.org/', img: 'partners/osc_mini.png' },
-      { name: 'GitHub', url: 'https://github.com/', img: 'partners/github_mini.png' },
-      { name: 'Really', url: 'https://opencollective.com/really-541ee976', img: 'partners/really_mini.png' }
+      { name: 'GitHub', url: 'https://github.com/', img: 'partners/github_mini.png' }
     ],
     # Ships hardware with OpenIPC on it.
     manufacturers: [
@@ -33,7 +32,9 @@ module PagesHelper
     # group for Russian-speaking visitors only -- see partner_groups.
     integrators: [
       { name: 'GoodCam', url: 'https://www.goodcam.io/', img: 'partners/goodcam_mini.png' },
-      { name: 'Faceter', url: 'https://faceter.cam/',    img: 'partners/faceter_mini.png' }
+      { name: 'AnyCam',  url: 'https://anycam.io/',      img: 'partners/anycam_mini.png' },
+      { name: 'Faceter', url: 'https://faceter.cam/',    img: 'partners/faceter_mini.png' },
+      { name: 'Really',  url: 'https://opencollective.com/really-541ee976', img: 'partners/really_mini.png' }
     ],
     # The FPV projects we grew up alongside.
     fpv: [
@@ -46,6 +47,13 @@ module PagesHelper
       { name: 'TUDSaT',  url: 'https://www.tudsat.space/', img: 'partners/tudsat_mini.png' },
       { name: 'WuSpace', url: 'https://wuespace.de/',      img: 'partners/wuespace_mini.png' }
     ],
+    # Kept in the system, rendered nowhere. There is no page for trade shows and
+    # exhibitions yet; when there is, it asks for :exhibitions and both the logo
+    # and the link are already here. Deliberately absent from HOME_PARTNER_ROWS
+    # and from every partner_groups call -- a test holds it that way.
+    exhibitions: [
+      { name: 'Expo Electronica', url: 'https://expoelectronica.ru/en/', img: 'partners/expo-electronica_mini.png' }
+    ],
     # Reverse engineering and silicon research we build on.
     research: [
       { name: 'Linux Chenxing', url: 'https://linux-chenxing.org/', img: 'partners/linuxchenxing_mini.png' }
@@ -66,7 +74,6 @@ module PagesHelper
     { name: 'Sputnik',       url: 'https://sputnik.systems/',     img: 'partners/sputnik_mini.png' },
     { name: 'Techno-Shield', url: 'https://msvoko.ru/',           img: 'partners/techno-shield_mini.png' },
     { name: 'KeyTelecom',    url: 'https://keytele.com/',         img: 'partners/keytelecom_mini.png' },
-    { name: 'AnyCam',        url: 'https://anycam.io/',           img: 'partners/anycam_mini.png' },
     { name: 'WebGlazok',     url: 'https://webglazok.com/',       img: 'partners/webglazok_mini.png' },
     { name: 'Yucca',         url: 'https://yucca.app/en',         img: 'partners/yucca_mini.png' },
     { name: 'IPEYE',         url: 'https://ipeye.ru/',            img: 'partners/ipeye_mini.png' },
@@ -83,7 +90,6 @@ module PagesHelper
     # { name: 'Flagman',          url: 'https://flagman.org/',          img: 'partners/flagman_mini.png' },
     # { name: 'Meldana',          url: 'https://meldana.com/',          img: 'partners/meldana_mini.png' },
     # { name: 'Binary Machines',  url: 'https://bmachines.ru/',         img: 'partners/binary-machines_mini.png' },
-    # { name: 'Expo Electronica', url: 'https://expoelectronica.ru/en/', img: 'partners/expo-electronica_mini.png' },
     # { name: 'GAINS',            url: 'https://gains.company/',        img: 'partners/gain_mini.png' }
   ].freeze
 

test/controllers/relaunch_pages_test.rb

@@ -206,9 +206,33 @@ class RelaunchPagesTest < ActionDispatch::IntegrationTest
     labels.each do |label|
       assert_includes response.body, I18n.t("site.partners.#{label}"), "the #{label} row is missing"
     end
-    # Every group still reaches the page through one row or another.
-    PagesHelper::PARTNER_GROUPS.each_value do |logos|
-      logos.each { |l| assert_includes response.body, l[:img].sub('.png', '') }
+    # Every group the rows name still reaches the page through one of them.
+    PagesHelper::HOME_PARTNER_ROWS.each_value do |keys|
+      keys.each do |key|
+        PagesHelper::PARTNER_GROUPS.fetch(key).each do |logo|
+          assert_includes response.body, logo[:img].sub('.png', '')
+        end
+      end
+    end
+  end
+
+  # :exhibitions is held in the system on purpose and shown on no page. There
+  # is no trade-show page yet; when there is, it asks for the group and the
+  # logo and link are already there. Until then this is what keeps it off the
+  # site -- a group is easy to add to a row by accident.
+  test 'the exhibitions group is kept but rendered nowhere' do
+    held = PagesHelper::PARTNER_GROUPS.fetch(:exhibitions)
+
+    assert_not_empty held
+    assert_not_includes PagesHelper::HOME_PARTNER_ROWS.values.flatten, :exhibitions
+
+    %w[/ /business /ecosystem /community /donate /low-latency /get-started].each do |path|
+      get path
+
+      held.each do |logo|
+        assert_not_includes response.body, logo[:img].sub('.png', ''),
+                            "#{logo[:name]} is being rendered on #{path}"
+      end
     end
   end