Cucumber: remove see layout test

activeadmin · Jul 13, 2023 · f72800b · f72800b
1 parent 93e21aa
commit f72800b
Show file tree

Hide file tree

Showing 5 changed files with 1 addition and 24 deletions.
diff --git a/features/dashboard.feature b/features/dashboard.feature
@@ -11,6 +11,5 @@ Feature: Dashboard
       """
     And I am logged in
     When I go to the dashboard
-    Then I should see the Active Admin layout
-    And I should not see the default welcome message
+    Then I should not see the default welcome message
     And I should see "Hello world from the dashboard page"
diff --git a/features/registering_pages.feature b/features/registering_pages.feature
@@ -17,7 +17,6 @@ Feature: Registering Pages
     When I go to the dashboard
     And I follow "Status"
     Then I should see the page title "Status"
-    And I should see the Active Admin layout
     And I should see the content "I love chocolate."
 
   Scenario: Registering a page with a complex name
@@ -32,7 +31,6 @@ Feature: Registering Pages
     When I go to the dashboard
     And I follow "Chocolate I lØve You!"
     Then I should see the page title "Chocolate I lØve You!"
-    And I should see the Active Admin layout
     And I should see the content "I love chocolate."
 
   Scenario: Registering an empty page
@@ -43,7 +41,6 @@ Feature: Registering Pages
     When I go to the dashboard
     And I follow "Status"
     Then I should see the page title "Status"
-    And I should see the Active Admin layout
 
   Scenario: Registering a page with a custom title as a string
     Given a configuration of:
@@ -130,10 +127,8 @@ Feature: Registering Pages
     And I follow "Status"
     And I follow "Post Check"
     Then I should see "Checked via POST"
-    And I should see the Active Admin layout
     When I follow "Get Check"
     Then I should see "Checked via GET"
-    And I should see the Active Admin layout
 
   Scenario: Adding a page action to a page
     Given a configuration of:
@@ -152,7 +147,6 @@ Feature: Registering Pages
     And I follow "Status"
     And I follow "Check"
     Then I should see the content "Chocolate I lØve You!"
-    And I should see the Active Admin layout
 
   @changes-filesystem
   Scenario: Adding a page action to a page with erb view
@@ -175,7 +169,6 @@ Feature: Registering Pages
     And I follow "Status"
     And I follow "Check"
     Then I should see the content "Chocolate lØves You Too!"
-    And I should see the Active Admin layout
 
   Scenario: Registering a page with paginated index table for a collection Array
     Given a user named "John Doe" exists
@@ -197,7 +190,6 @@ Feature: Registering Pages
     When I go to the dashboard
     And I follow "Special users"
     Then I should see the page title "Special users"
-    And I should see the Active Admin layout
     And I should see 1 user in the table
 
   Scenario: Displaying parent information from a belongs_to page

diff --git a/features/specifying_actions.feature b/features/specifying_actions.feature
@@ -65,7 +65,6 @@ Feature: Specifying Actions
     And I follow "Review"
     Then I should see "Review: Hello World"
     And I should see the page title "Review"
-    And I should see the Active Admin layout
 
   @changes-filesystem
   Scenario: Specify a custom member action with template using arb
@@ -92,7 +91,6 @@ Feature: Specifying Actions
     And I follow "Review"
     Then I should see "Review: Hello World"
     And I should see the page title "Review"
-    And I should see the Active Admin layout
 
   Scenario: Specify a custom member action with multiple http methods
     Given a configuration of:

diff --git a/features/step_definitions/layout_steps.rb b/features/step_definitions/layout_steps.rb
diff --git a/spec/unit/views/pages/layout_spec.rb b/spec/unit/views/pages/layout_spec.rb
@@ -47,13 +47,5 @@
     expect(layout.attributes[:lang]).to eq :en
   end
 
-  describe "the body" do
-    it "should have class 'active_admin'" do
-      expect(layout.build.class_list).to include "active_admin"
-    end
-
-    it "should have namespace class" do
-      expect(layout.build.class_list).to include "#{active_admin_namespace.name}_namespace"
-    end
   end
 end