bigbluebutton_rails 1.3.0 → 1.4.0.beta1

data/spec/models/bigbluebutton_recording_spec.rb

@@ -11,6 +11,8 @@ describe BigbluebuttonRecording do
 
   it { should belong_to(:room) }
 
+  it { should belong_to(:meeting) }
+
   it { should validate_presence_of(:recordid) }
   it { should validate_uniqueness_of(:recordid) }
 
@@ -44,7 +46,7 @@ describe BigbluebuttonRecording do
     }
   end
 
-  describe "#sync" do
+  describe ".sync" do
     let(:data) {
       [
        {
@@ -192,7 +194,7 @@ describe BigbluebuttonRecording do
     end
   end
 
-  describe "#update_recording" do
+  describe ".update_recording" do
     let(:old_attrs) { FactoryGirl.attributes_for(:bigbluebutton_recording) }
     let(:attrs) { FactoryGirl.attributes_for(:bigbluebutton_recording) }
     let(:recording) { FactoryGirl.create(:bigbluebutton_recording, old_attrs) }
@@ -226,11 +228,13 @@ describe BigbluebuttonRecording do
     it { recording.room.should == @room }
   end
 
-  describe "#create_recording" do
+  describe ".create_recording" do
+    let(:meeting_start_time) { DateTime.now }
+    let(:recordid) { "#{SecureRandom.uuid}-#{meeting_start_time.to_i}" }
     let(:attrs) { FactoryGirl.attributes_for(:bigbluebutton_recording) }
     let(:data) {
       {
-        :recordid => attrs[:recordid],
+        :recordid => recordid,
         :meetingid => attrs[:meetingid],
         :name => attrs[:name],
         :published => attrs[:published],
@@ -244,22 +248,29 @@ describe BigbluebuttonRecording do
 
     before {
       @room = FactoryGirl.create(:bigbluebutton_room, :meetingid => attrs[:meetingid])
+      @meeting = FactoryGirl.create(:bigbluebutton_meeting, :room => @room, :start_time => meeting_start_time.utc)
+
       BigbluebuttonRecording.should_receive(:sync_additional_data)
         .with(anything, data)
       BigbluebuttonRecording.send(:create_recording, new_server, data)
       @recording = BigbluebuttonRecording.last
     }
-    it { @recording.recordid.should == attrs[:recordid] }
-    it { @recording.meetingid.should == attrs[:meetingid] }
-    it { @recording.name.should == attrs[:name] }
-    it { @recording.published.should == attrs[:published] }
-    it { @recording.end_time.utc.to_i.should == attrs[:end_time].utc.to_i }
-    it { @recording.start_time.utc.to_i.should == attrs[:start_time].utc.to_i }
-    it { @recording.server.should == new_server }
-    it { @recording.room.should == @room }
+    it("sets recordid") { @recording.recordid.should == recordid }
+    it("sets meetingid") { @recording.meetingid.should == attrs[:meetingid] }
+    it("sets name") { @recording.name.should == attrs[:name] }
+    it("sets published") { @recording.published.should == attrs[:published] }
+    it("sets end_time") { @recording.end_time.utc.to_i.should == attrs[:end_time].utc.to_i }
+    it("sets start_time") { @recording.start_time.utc.to_i.should == attrs[:start_time].utc.to_i }
+    it("sets server") { @recording.server.should == new_server }
+    it("sets room") { @recording.room.should == @room }
+    it("sets meeting") { @recording.meeting.should == @meeting }
+    it("sets description") {
+      time = data[:start_time].utc.to_formatted_s(:long)
+      @recording.description.should == I18n.t('bigbluebutton_rails.recordings.default.description', :time => time)
+    }
   end
 
-  describe "#adapt_recording_hash" do
+  describe ".adapt_recording_hash" do
     let(:before) {
       { :recordID => "anything",
         :meetingID => "anything",
@@ -282,7 +293,7 @@ describe BigbluebuttonRecording do
     it { should eq(after) }
   end
 
-  describe "#sync_additional_data" do
+  describe ".sync_additional_data" do
     let(:attrs) { FactoryGirl.attributes_for(:bigbluebutton_recording) }
     let(:recording) { FactoryGirl.create(:bigbluebutton_recording) }
     let(:data) {
@@ -327,7 +338,7 @@ describe BigbluebuttonRecording do
     end
   end
 
-  describe "#sync_metadata" do
+  describe ".sync_metadata" do
     let(:recording) { FactoryGirl.create(:bigbluebutton_recording) }
 
     context "updates metadata that are already in the db" do
@@ -386,7 +397,7 @@ describe BigbluebuttonRecording do
 
   end
 
-  describe "#sync_playback_formats" do
+  describe ".sync_playback_formats" do
     let(:recording) { FactoryGirl.create(:bigbluebutton_recording) }
 
     context "with several formats" do
@@ -447,4 +458,46 @@ describe BigbluebuttonRecording do
 
   end
 
+  describe ".find_matching_meeting" do
+
+    context "if no recording is informed" do
+      let(:recording) { FactoryGirl.create(:bigbluebutton_recording, :room => nil) }
+      subject { BigbluebuttonRecording.send(:find_matching_meeting, nil) }
+      it { subject.should be_nil }
+    end
+
+    context "if the recording has no room associated to it" do
+      let(:recording) { FactoryGirl.create(:bigbluebutton_recording, :room => nil) }
+      subject { BigbluebuttonRecording.send(:find_matching_meeting, recording) }
+      it { subject.should be_nil }
+    end
+
+    context "if can't find the start time in recordid" do
+      let(:recording) { FactoryGirl.create(:bigbluebutton_recording, :recordid => "without-timestamp") }
+      subject { BigbluebuttonRecording.send(:find_matching_meeting, recording) }
+      it { subject.should be_nil }
+    end
+
+    context "if found a start time in recordid" do
+      let(:meeting_start_time) { DateTime.now }
+      let(:recording) {
+        FactoryGirl.create(:bigbluebutton_recording, :recordid => "#{SecureRandom.uuid}-#{meeting_start_time.to_i}")
+      }
+
+      context "when there's no associated meeting" do
+        subject { BigbluebuttonRecording.send(:find_matching_meeting, recording) }
+        it { subject.should be_nil }
+      end
+
+      context "when there's one associated meeting" do
+        before {
+          @meeting = FactoryGirl.create(:bigbluebutton_meeting, :room => recording.room, :start_time => meeting_start_time)
+        }
+        subject { BigbluebuttonRecording.send(:find_matching_meeting, recording) }
+        it { subject.should eq(@meeting) }
+      end
+    end
+
+  end
+
 end

data/spec/models/bigbluebutton_room_db_spec.rb

@@ -21,6 +21,8 @@ describe BigbluebuttonRoom do
     it { should have_db_column(:param).of_type(:string) }
     it { should have_db_column(:record).of_type(:boolean) }
     it { should have_db_column(:duration).of_type(:integer) }
+    it { should have_db_column(:created_at).of_type(:datetime) }
+    it { should have_db_column(:updated_at).of_type(:datetime) }
     it { should have_db_index(:server_id) }
     it { should have_db_index(:meetingid).unique(true) }
     it { should have_db_index(:voice_bridge).unique(true) }

data/spec/models/bigbluebutton_room_options_db_spec.rb

@@ -0,0 +1,16 @@
+require 'spec_helper'
+
+describe BigbluebuttonRoomOptions do
+
+  # to ensure that the migration is correct
+  context "db" do
+    it { should have_db_column(:room_id).of_type(:integer) }
+    it { should have_db_column(:default_layout).of_type(:string) }
+    it { should have_db_column(:created_at).of_type(:datetime) }
+    it { should have_db_column(:updated_at).of_type(:datetime) }
+    it { should have_db_column(:presenter_share_only).of_type(:boolean)}
+    it { should have_db_column(:auto_start_video).of_type(:boolean) }
+    it { should have_db_column(:auto_start_audio).of_type(:boolean) }
+  end
+
+end

data/spec/models/bigbluebutton_room_options_spec.rb

@@ -0,0 +1,206 @@
+# -*- coding: utf-8 -*-
+require 'spec_helper'
+
+describe BigbluebuttonRoomOptions do
+  it "loaded correctly" do
+    BigbluebuttonRoomOptions.new.should be_a_kind_of(ActiveRecord::Base)
+  end
+
+  let(:room_options) { FactoryGirl.create(:bigbluebutton_room).room_options }
+
+  it { should belong_to(:room) }
+  it {
+    room = FactoryGirl.create(:bigbluebutton_room)
+    BigbluebuttonRoomOptions.new(:room => room)
+      .room.should be_a_kind_of(BigbluebuttonRoom) }
+  it { should validate_presence_of(:room_id) }
+
+  describe "#get_available_layouts" do
+    it "returns the layouts available" do
+      expected = ["Default", "Video Chat", "Meeting", "Webinar", "Lecture assistant", "Lecture"]
+      room_options.get_available_layouts.should eql(expected)
+    end
+  end
+
+  describe "#set_on_config_xml" do
+    let(:config_xml) { '<config></config>' }
+
+    context "if the xml changed" do
+      before {
+        # set a few values to true and a few to false to test both cases
+        room_options.update_attributes(:default_layout => "AnyLayout",
+                                       :presenter_share_only => true,
+                                       :auto_start_video => false,
+                                       :auto_start_audio => false)
+        BigBlueButton::BigBlueButtonConfigXml.any_instance
+          .should_receive(:set_attribute)
+          .with('layout', 'defaultLayout', "AnyLayout", false)
+        BigBlueButton::BigBlueButtonConfigXml.any_instance
+          .should_receive(:set_attribute)
+          .with('VideoconfModule', 'presenterShareOnly', true, true)
+        BigBlueButton::BigBlueButtonConfigXml.any_instance
+          .should_receive(:set_attribute)
+          .with('PhoneModule', 'presenterShareOnly', true, true)
+        BigBlueButton::BigBlueButtonConfigXml.any_instance
+          .should_receive(:set_attribute)
+          .with('VideoconfModule', 'autoStart', false, true)
+        BigBlueButton::BigBlueButtonConfigXml.any_instance
+          .should_receive(:set_attribute)
+          .with('PhoneModule', 'autoJoin', false, true)
+        BigBlueButton::BigBlueButtonConfigXml.any_instance
+          .should_receive(:is_modified?).and_return(true)
+        BigBlueButton::BigBlueButtonConfigXml.any_instance
+          .should_receive(:as_string).and_return('new xml as string')
+      }
+      subject { room_options.set_on_config_xml(config_xml) }
+      it("returns the new xml") { should eql('new xml as string') }
+    end
+
+    context "if the xml did not change" do
+      before {
+        room_options.update_attributes(:default_layout => "AnyLayout",
+                                       :presenter_share_only => false,
+                                       :auto_start_video => false,
+                                       :auto_start_audio => false)
+        BigBlueButton::BigBlueButtonConfigXml.any_instance
+          .should_receive(:set_attribute)
+          .with('layout', 'defaultLayout', "AnyLayout", false)
+        BigBlueButton::BigBlueButtonConfigXml.any_instance
+          .should_receive(:set_attribute)
+          .with('VideoconfModule', 'presenterShareOnly', false, true)
+        BigBlueButton::BigBlueButtonConfigXml.any_instance
+          .should_receive(:set_attribute)
+          .with('PhoneModule', 'presenterShareOnly', false, true)
+        BigBlueButton::BigBlueButtonConfigXml.any_instance
+          .should_receive(:set_attribute)
+          .with('VideoconfModule', 'autoStart', false, true)
+        BigBlueButton::BigBlueButtonConfigXml.any_instance
+          .should_receive(:set_attribute)
+          .with('PhoneModule', 'autoJoin', false, true)
+        BigBlueButton::BigBlueButtonConfigXml.any_instance
+          .should_receive(:is_modified?).and_return(false)
+      }
+      subject { room_options.set_on_config_xml(config_xml) }
+      it("returns false") { should be_false }
+    end
+
+    context "if #default_layout is" do
+      context "nil" do
+        before {
+          room_options.update_attributes(:default_layout => nil)
+          BigBlueButton::BigBlueButtonConfigXml.any_instance
+            .should_not_receive(:set_attribute)
+            .with('layout', 'defaultLayout', anything, anything)
+        }
+        it("doesn't set the property in the xml") { room_options.set_on_config_xml(config_xml) }
+      end
+
+      context "empty string" do
+        before {
+          room_options.update_attributes(:default_layout => "")
+          BigBlueButton::BigBlueButtonConfigXml.any_instance
+            .should_not_receive(:set_attribute)
+            .with('layout', 'defaultLayout', anything, anything)
+        }
+        it("doesn't set the property in the xml") { room_options.set_on_config_xml(config_xml) }
+      end
+    end
+
+    context "if #presenter_share_only is" do
+      context "nil" do
+        before {
+          room_options.update_attributes(:presenter_share_only => nil)
+          BigBlueButton::BigBlueButtonConfigXml.any_instance
+            .should_not_receive(:set_attribute)
+            .with('VideoconfModule', 'presenterShareOnly', anything, anything)
+          BigBlueButton::BigBlueButtonConfigXml.any_instance
+            .should_not_receive(:set_attribute)
+            .with('PhoneModule', 'presenterShareOnly', anything, anything)
+        }
+        it("doesn't set the property in the xml") { room_options.set_on_config_xml(config_xml) }
+      end
+    end
+
+    context "if #auto_start_video is" do
+      context "nil" do
+        before {
+          room_options.update_attributes(:auto_start_video => nil)
+          BigBlueButton::BigBlueButtonConfigXml.any_instance
+            .should_not_receive(:set_attribute)
+            .with('VideoconfModule', 'autoStart', anything, anything)
+        }
+        it("doesn't set the property in the xml") { room_options.set_on_config_xml(config_xml) }
+      end
+    end
+
+    context "if #auto_start_audio is" do
+      context "nil" do
+        before {
+          room_options.update_attributes(:auto_start_audio => nil)
+          BigBlueButton::BigBlueButtonConfigXml.any_instance
+            .should_not_receive(:set_attribute)
+            .with('PhoneModule', 'autoJoin', anything, anything)
+        }
+        it("doesn't set the property in the xml") {room_options.set_on_config_xml(config_xml) }
+      end
+    end
+
+  end
+
+  describe "#is_modified?" do
+    context "if default_layout is set" do
+      before { room_options.update_attributes(:default_layout => 'Any') }
+      subject { room_options.is_modified? }
+      it("returns true") { should be_true }
+    end
+
+    context "if default_layout is not set" do
+      before { room_options.update_attributes(:default_layout => nil) }
+      subject { room_options.is_modified? }
+      it("returns false") { should be_false }
+    end
+
+    context "if default_layout is empty" do
+      before { room_options.update_attributes(:default_layout => "") }
+      subject { room_options.is_modified? }
+      it("returns true") { should be_true }
+    end
+
+    context "if presenter_share_only is set" do
+      before { room_options.update_attributes(:presenter_share_only => true) }
+      subject { room_options.is_modified? }
+      it("returns true") { should be_true}
+    end
+
+    context "if presenter_share_only is not set" do
+      before { room_options.update_attributes(:presenter_share_only => nil) }
+      subject { room_options.is_modified? }
+      it("returns false") { should be_false }
+    end
+
+    context "if auto_start_video is set" do
+      before { room_options.update_attributes(:auto_start_video => true) }
+      subject { room_options.is_modified? }
+      it("returns true") { should be_true}
+    end
+
+    context "if auto_start_video is not set" do
+      before { room_options.update_attributes(:auto_start_video => nil) }
+      subject { room_options.is_modified? }
+      it("returns false" ) { should be_false }
+    end
+
+    context "if auto_start_audio is set" do
+      before { room_options.update_attributes(:auto_start_audio => true) }
+      subject { room_options.is_modified? }
+      it("returns true") { should be_true}
+    end
+
+    context "if auto_start_audio is not set" do
+      before { room_options.update_attributes(:auto_start_audio => nil) }
+      subject { room_options.is_modified? }
+      it("returns false" ) { should be_false }
+    end
+  end
+
+end