Fix inflection of OpenProject

Author: oliverguenther

demo/config/initializers/lookbook_inflector_patch.rb

@@ -34,14 +34,12 @@ module Patches
     # Allow directory labels in lookbook to be inflected
     module LookbookTreeNodeInflector
       def label
-        return name if name == "OpenProject"
+        return "OpenProject" if %w[OpenProject open_project].include?(name.to_s)
 
         super
       end
     end
   end
 end
 
-# if Object.const_defined?(:Lookbook)
-#   Lookbook::TreeNode.prepend OpenProject::Patches::LookbookTreeNodeInflector
-# end
+Lookbook::TreeNode.prepend OpenProject::Patches::LookbookTreeNodeInflector if Object.const_defined?(:Lookbook)

previews/primer/open_project/page_header_preview.rb

@@ -2,7 +2,6 @@
 
 module Primer
   module OpenProject
-    # @logical_path primer/OpenProject
     # @component Primer::OpenProject::PageHeader
     # @label Page Header
     class PageHeaderPreview < ViewComponent::Preview

test/accessibility_test.rb

@@ -13,7 +13,7 @@ class AccessibilityTest < System::TestCase
 
       scenarios.each do |scenario|
         define_method(:"test_#{scenario.lookup_path.parameterize(separator: "_")}") do
-          visit "/rails/view_components/#{scenario.lookup_path.underscore}"
+          visit "/rails/view_components/#{scenario.lookup_path}"
 
           excludes = Primer::Accessibility.axe_rules_to_skip(
             component: preview.components.first&.component_class,

test/playwright/snapshots.test.ts

@@ -29,11 +29,10 @@ test.describe('generate snapshots', () => {
   for (const preview of previewsJson) {
     for (const example of preview.examples) {
       if (example.snapshot !== 'false') {
-        const previewPath = example.preview_path.replace('OpenProject', 'open_project')
         if (example.snapshot === 'interactive') {
           for (const theme of themes) {
             test(`${example.preview_path}-${theme}`, async ({page}) => {
-              await page.goto(`/rails/view_components/${previewPath}?theme=${theme}`)
+              await page.goto(`/rails/view_components/${example.preview_path}?theme=${theme}`)
 
               // Focus state
               await page.keyboard.press('Tab')
@@ -45,16 +44,16 @@ test.describe('generate snapshots', () => {
               await new Promise(resolve => setTimeout(resolve, 100))
 
               const focusedScreenshot = await page.screenshot({animations: 'disabled'})
-              expect(focusedScreenshot).toMatchSnapshot([previewPath, `${theme}.png`])
+              expect(focusedScreenshot).toMatchSnapshot([example.preview_path, `${theme}.png`])
             })
           }
         }
 
         test(example.preview_path, async ({page}) => {
-          await page.goto(`/rails/view_components/${previewPath}?theme=all`)
+          await page.goto(`/rails/view_components/${example.preview_path}?theme=all`)
 
           const defaultScreenshot = await page.locator('#component-preview').screenshot({animations: 'disabled'})
-          expect(defaultScreenshot).toMatchSnapshot([previewPath, 'default.png'])
+          expect(defaultScreenshot).toMatchSnapshot([example.preview_path, 'default.png'])
 
           // Focus state
           await page.keyboard.press('Tab')
@@ -63,7 +62,7 @@ test.describe('generate snapshots', () => {
           await new Promise(resolve => setTimeout(resolve, 100))
 
           const focusedScreenshot = await page.locator('#component-preview').screenshot({animations: 'disabled'})
-          expect(focusedScreenshot).toMatchSnapshot([previewPath, 'focused.png'])
+          expect(focusedScreenshot).toMatchSnapshot([example.preview_path, 'focused.png'])
         })
       }
     }