Merge pull request #13776 from opf/fix/use_meeting_duration_from_seed…

…_data

Use meeting duration from seed data

modules/meeting/app/seeders/meetings/demo_data/meeting_seeder.rb

@@ -43,9 +43,14 @@ def model_attributes(meeting_data)
         {
           title: meeting_data['title'],
           author: seed_data.find_reference(meeting_data['author']),
+          duration: minutes_to_hours(meeting_data['duration']),
           project:
         }
       end
+
+      def minutes_to_hours(duration)
+        duration && (duration / 60.0)
+      end
     end
   end
 end

modules/meeting/spec/seeders/demo_data/project_seeder_spec.rb

@@ -58,6 +58,9 @@
       meetings:
         - title: Weekly
           reference: :weekly_meeting
+          duration: 30
+          author: :openproject_user
+        - title: Implicit 1h duration
           author: :openproject_user
       meeting_agenda_items:
         - title: First topic
@@ -74,10 +77,14 @@
     SEEDING_DATA_YAML
   end
 
-  it 'creates an associated meeting' do
+  before do
     project_seeder.seed!
+  end
+
+  it 'creates an associated meeting' do
     meeting = Meeting.find_by(title: 'Weekly')
     expect(meeting.author).to eq user
+    expect(meeting.duration).to eq 0.5
 
     expect(meeting.agenda_items.count).to eq 2
 
@@ -92,4 +99,9 @@
     expect(second.notes).to eq 'Some **markdown**'
     expect(second.work_package).to eq work_package
   end
+
+  it 'uses default duration of 1h if not specified' do
+    meeting = Meeting.find_by(title: 'Implicit 1h duration')
+    expect(meeting.duration).to eq 1
+  end
 end

spec/seeders/root_seeder_standard_edition_spec.rb

@@ -120,6 +120,7 @@
     include_examples 'it creates records', model: Status, expected_count: 14
     include_examples 'it creates records', model: TimeEntryActivity, expected_count: 6
     include_examples 'it creates records', model: Workflow, expected_count: 1172
+    include_examples 'it creates records', model: Meeting, expected_count: 1
   end
 
   describe 'demo data' do