Fixing tests

nanego · Jan 22, 2025 · 1d90bb5 · 1d90bb5
1 parent 35ec0f7
commit 1d90bb5
Show file tree

Hide file tree

Showing 4 changed files with 11 additions and 8 deletions.
diff --git a/app/controllers/power_distribution_units_controller.rb b/app/controllers/power_distribution_units_controller.rb
@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-class PowerDistributionUnitsController < ServersController
+class PowerDistributionUnitsController < ApplicationController
   before_action :set_pdu, only: %i[show edit update destroy destroy_connections]
 
   def index

diff --git a/app/views/power_distribution_units/_form.html.erb b/app/views/power_distribution_units/_form.html.erb
@@ -86,7 +86,7 @@
       <div class="col-12 mt-4" data-controller="nested-form">
         <template data-nested-form-target="template">
           <%= f.fields_for :cards, f.object.cards.klass.new, child_index: '__NEW_RECORD__' do |card_fields| %>
-            <%= render partial: "card_fields", locals: { f: card_fields, server: @pdu, enclosure: enclosure } %>
+            <%= render partial: "servers/card_fields", locals: { f: card_fields, server: @pdu, enclosure: enclosure } %>
           <% end %>
         </template>
 
@@ -99,7 +99,7 @@
             <% cards = @pdu.cards.select { |c| c.composant.enclosure == enclosure }
                                     .sort_by! { |c| c.composant.position } %>
             <%= f.fields_for :cards, cards do |card_fields| %>
-              <%= render partial: "card_fields", locals: { f: card_fields, server: @pdu, enclosure: enclosure } %>
+              <%= render partial: "servers/card_fields", locals: { f: card_fields, server: @pdu, enclosure: enclosure } %>
             <% end %>
 
             <div data-nested-form-target="target" class="d-none"></div>
@@ -126,7 +126,7 @@
   <fieldset class="col-12 mt-4" data-controller="nested-form">
     <template data-nested-form-target="template">
       <%= f.fields_for :documents, f.object.documents.klass.new, child_index: '__NEW_RECORD__' do |document_fields| %>
-        <%= render partial: "document_fields", locals: { f: document_fields } %>
+        <%= render partial: "servers/document_fields", locals: { f: document_fields } %>
       <% end %>
     </template>
 
@@ -136,7 +136,7 @@
       <% end %>
 
       <%= f.fields_for :documents do |document_fields| %>
-        <%= render partial: "document_fields", locals: { f:  document_fields } %>
+        <%= render partial: "servers/document_fields", locals: { f:  document_fields } %>
       <% end %>
 
       <div data-nested-form-target="target"></div>

diff --git a/test/fixtures/servers.yml b/test/fixtures/servers.yml
@@ -49,7 +49,7 @@ pdu:
   modele_id: 3
   side: right
   color: B
-  numero: PDU_1
+  numero: pdu_frame_1
 
 with_cluster:
   frame_id: 1

diff --git a/test/models/server_test.rb b/test/models/server_test.rb
@@ -8,13 +8,16 @@ class ServerTest < ActiveSupport::TestCase
     refute_includes Server.only_pdus, servers(:one)
   end
 
-  test 'scope no_puds' do
+  test 'scope no_pdus' do
     refute_includes Server.no_pdus, servers(:pdu)
     assert_includes Server.no_pdus, servers(:one)
   end
 
   test '#documentation_url with no documentation_url value on manufacturer' do
-    assert_nil servers(:pdu).documentation_url
+    server = servers(:pdu)
+    server.numero = nil
+
+    assert_nil server.documentation_url
   end
 
   test '#documentation_url with no numero value on manufacturer' do