Merge pull request #458 from concord-consortium/fix-wrapping-plugins

fix wrapping plugins
concord-consortium · Aug 7, 2019 · 203394f · 203394f
2 parents c714248 + 13f4194
commit 203394f
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 1 deletion.
diff --git a/app/models/embeddable/embeddable_plugin.rb b/app/models/embeddable/embeddable_plugin.rb
@@ -10,7 +10,7 @@ def self.table_name
     attr_accessible :plugin, :approved_script_id, :description, :author_data,
     :is_full_width, :is_hidden, :component_label
 
-    has_one :plugin, as: :plugin_scope
+    has_one :plugin, as: :plugin_scope, autosave: true
 
     has_many :page_items, :as => :embeddable, :dependent => :destroy
     has_many :interactive_pages, :through => :page_items

diff --git a/spec/models/embeddable/embeddable_plugin_spec.rb b/spec/models/embeddable/embeddable_plugin_spec.rb
@@ -87,4 +87,14 @@
     end
 
   end
+
+  describe 'delegated to plugin methods' do
+    it 'should save value after a simple create, set, save, reload' do
+      embeddable = Embeddable::EmbeddablePlugin.create!
+      embeddable.approved_script_id = "123"
+      embeddable.save!
+      embeddable.reload
+      expect(embeddable.approved_script_id).to eq("123")
+    end
+  end
 end