cable_ready 5.0.0.pre6 → 5.0.0.pre9

data/rollup.config.js

@@ -0,0 +1,75 @@
+import resolve from '@rollup/plugin-node-resolve'
+import commonjs from '@rollup/plugin-commonjs'
+import json from '@rollup/plugin-json'
+import { terser } from 'rollup-plugin-terser'
+
+const pretty = () => {
+  return terser({
+    mangle: false,
+    compress: false,
+    format: {
+      beautify: true,
+      indent_level: 2
+    }
+  })
+}
+
+const minify = () => {
+  return terser({
+    mangle: true,
+    compress: true
+  })
+}
+
+const esConfig = {
+  format: 'es',
+  inlineDynamicImports: true
+}
+
+const umdConfig = {
+  name: 'CableReady',
+  format: 'umd',
+  exports: 'named',
+  globals: { morphdom: 'morphdom' }
+}
+
+const distFolders = ['dist/', 'app/assets/javascripts/']
+
+const output = distFolders
+  .map(distFolder => [
+    {
+      ...esConfig,
+      file: `${distFolder}/cable_ready.js`,
+      plugins: [pretty()]
+    },
+    {
+      ...esConfig,
+      file: `${distFolder}/cable_ready.min.js`,
+      sourcemap: true,
+      plugins: [minify()]
+    },
+    {
+      ...umdConfig,
+      file: `${distFolder}/cable_ready.umd.js`,
+      plugins: [pretty()]
+    },
+    {
+      ...umdConfig,
+      file: `${distFolder}/cable_ready.umd.min.js`,
+      sourcemap: true,
+      plugins: [minify()]
+    }
+  ])
+  .flat()
+
+export default [
+  {
+    external: ['morphdom'],
+    input: 'javascript/index.js',
+    output,
+    plugins: [commonjs(), resolve(), json()],
+    watch: {
+      include: 'javascript/**'
+    }
+  }
+]

data/test/dummy/app/models/dugong.rb

@@ -0,0 +1,4 @@
+class Dugong < ApplicationRecord
+  include CableReady::Updatable
+  has_many_attached :images, enable_updates: true
+end

data/test/dummy/db/migrate/20220329222959_create_dugongs.rb

@@ -0,0 +1,8 @@
+class CreateDugongs < ActiveRecord::Migration[6.1]
+  def change
+    create_table :dugongs do |t|
+
+      t.timestamps
+    end
+  end
+end

data/test/dummy/db/migrate/20220329230221_create_active_storage_tables.active_storage.rb

@@ -0,0 +1,36 @@
+# This migration comes from active_storage (originally 20170806125915)
+class CreateActiveStorageTables < ActiveRecord::Migration[5.2]
+  def change
+    create_table :active_storage_blobs do |t|
+      t.string   :key,          null: false
+      t.string   :filename,     null: false
+      t.string   :content_type
+      t.text     :metadata
+      t.string   :service_name, null: false
+      t.bigint   :byte_size,    null: false
+      t.string   :checksum,     null: false
+      t.datetime :created_at,   null: false
+
+      t.index [ :key ], unique: true
+    end
+
+    create_table :active_storage_attachments do |t|
+      t.string     :name,     null: false
+      t.references :record,   null: false, polymorphic: true, index: false
+      t.references :blob,     null: false
+
+      t.datetime :created_at, null: false
+
+      t.index [ :record_type, :record_id, :name, :blob_id ], name: "index_active_storage_attachments_uniqueness", unique: true
+      t.foreign_key :active_storage_blobs, column: :blob_id
+    end
+
+    create_table :active_storage_variant_records do |t|
+      t.belongs_to :blob, null: false, index: false
+      t.string :variation_digest, null: false
+
+      t.index %i[ blob_id variation_digest ], name: "index_active_storage_variant_records_uniqueness", unique: true
+      t.foreign_key :active_storage_blobs, column: :blob_id
+    end
+  end
+end

data/test/dummy/db/schema.rb

@@ -10,7 +10,40 @@
 #
 # It's strongly recommended that you check this file into your version control system.
 
-ActiveRecord::Schema.define(version: 2021_09_13_191759) do
+ActiveRecord::Schema.define(version: 2022_03_29_230221) do
+
+  create_table "active_storage_attachments", force: :cascade do |t|
+    t.string "name", null: false
+    t.string "record_type", null: false
+    t.integer "record_id", null: false
+    t.integer "blob_id", null: false
+    t.datetime "created_at", null: false
+    t.index ["blob_id"], name: "index_active_storage_attachments_on_blob_id"
+    t.index ["record_type", "record_id", "name", "blob_id"], name: "index_active_storage_attachments_uniqueness", unique: true
+  end
+
+  create_table "active_storage_blobs", force: :cascade do |t|
+    t.string "key", null: false
+    t.string "filename", null: false
+    t.string "content_type"
+    t.text "metadata"
+    t.string "service_name", null: false
+    t.bigint "byte_size", null: false
+    t.string "checksum", null: false
+    t.datetime "created_at", null: false
+    t.index ["key"], name: "index_active_storage_blobs_on_key", unique: true
+  end
+
+  create_table "active_storage_variant_records", force: :cascade do |t|
+    t.integer "blob_id", null: false
+    t.string "variation_digest", null: false
+    t.index ["blob_id", "variation_digest"], name: "index_active_storage_variant_records_uniqueness", unique: true
+  end
+
+  create_table "dugongs", force: :cascade do |t|
+    t.datetime "created_at", precision: 6, null: false
+    t.datetime "updated_at", precision: 6, null: false
+  end
 
   create_table "posts", force: :cascade do |t|
     t.string "title"
@@ -45,5 +78,7 @@ ActiveRecord::Schema.define(version: 2021_09_13_191759) do
     t.index ["team_id"], name: "index_users_on_team_id"
   end
 
+  add_foreign_key "active_storage_attachments", "active_storage_blobs", column: "blob_id"
+  add_foreign_key "active_storage_variant_records", "active_storage_blobs", column: "blob_id"
   add_foreign_key "posts", "users"
 end

data/test/dummy/test/models/dugong_test.rb

@@ -0,0 +1,7 @@
+require "test_helper"
+
+class DugongTest < ActiveSupport::TestCase
+  # test "the truth" do
+  #   assert true
+  # end
+end

data/test/lib/cable_ready/updatable_test.rb

@@ -13,7 +13,7 @@ class CableReady::UpdatableTest < ActiveSupport::TestCase
   test "updates the collection when an item is added" do
     mock_server = mock("server")
     mock_server.expects(:broadcast).with(User, {}).once
-    mock_server.expects(:broadcast).with("gid://dummy/User/1:posts", {}).once
+    mock_server.expects(:broadcast).with("gid://dummy/User/1:posts", {changed: ["id", "title", "user_id", "created_at", "updated_at"]}).once
 
     ActionCable.stubs(:server).returns(mock_server)
     user = User.create(name: "John Doe")
@@ -26,7 +26,7 @@ class CableReady::UpdatableTest < ActiveSupport::TestCase
     post = user.posts.create(title: "Lorem")
 
     mock_server = mock("server")
-    mock_server.expects(:broadcast).with("gid://dummy/User/1:posts", {}).once
+    mock_server.expects(:broadcast).with("gid://dummy/User/1:posts", {changed: ["id", "title", "user_id", "created_at", "updated_at"]}).once
 
     ActionCable.stubs(:server).returns(mock_server)
 
@@ -38,7 +38,7 @@ class CableReady::UpdatableTest < ActiveSupport::TestCase
     post = user.posts.create(title: "Lorem")
 
     mock_server = mock("server")
-    mock_server.expects(:broadcast).with("gid://dummy/User/1:posts", {}).once
+    mock_server.expects(:broadcast).with("gid://dummy/User/1:posts", {changed: ["title", "updated_at"]}).once
 
     ActionCable.stubs(:server).returns(mock_server)
 
@@ -50,7 +50,7 @@ class CableReady::UpdatableTest < ActiveSupport::TestCase
 
     mock_server = mock("server")
     mock_server.expects(:broadcast).with(User, {}).once
-    mock_server.expects(:broadcast).with(user.to_global_id, {}).once
+    mock_server.expects(:broadcast).with(user.to_global_id, {changed: ["name", "updated_at"]}).once
 
     ActionCable.stubs(:server).returns(mock_server)
 
@@ -63,10 +63,10 @@ class CableReady::UpdatableTest < ActiveSupport::TestCase
 
     mock_server = mock("server")
     mock_server.expects(:broadcast).with(User, {}).once
-    mock_server.expects(:broadcast).with(user.to_global_id, {}).once
-    mock_server.expects(:broadcast).with("gid://dummy/Team/1:users", {}).once
+    mock_server.expects(:broadcast).with(user.to_global_id, {changed: ["name", "updated_at"]}).once
+    mock_server.expects(:broadcast).with("gid://dummy/Team/1:users", {changed: ["name", "updated_at"]}).once
     mock_server.expects(:broadcast).with(Team, {}).once
-    mock_server.expects(:broadcast).with(team.to_global_id, {}).once
+    mock_server.expects(:broadcast).with(team.to_global_id, {changed: ["id", "created_at", "updated_at"]}).once
 
     ActionCable.stubs(:server).returns(mock_server)
 
@@ -94,7 +94,7 @@ class CableReady::UpdatableTest < ActiveSupport::TestCase
     section.updates_enabled = true
 
     mock_server.expects(:broadcast).with(Section, {}).once
-    mock_server.expects(:broadcast).with(section.to_global_id, {}).once
+    mock_server.expects(:broadcast).with(section.to_global_id, {changed: ["title", "updated_at", "updates_enabled"]}).once
     section.update(title: "First Section")
   end
 
@@ -109,4 +109,27 @@ class CableReady::UpdatableTest < ActiveSupport::TestCase
 
     entity.fake_update
   end
+
+  test "updates the collection when a file is attached" do
+    mock_server = mock("server")
+    image = File.open(Rails.root.join("test", "fixtures", "files", "dugong.jpg"))
+    dugong = Dugong.create
+
+    mock_server.expects(:broadcast).with("gid://dummy/Dugong/1:images", {changed: ["id", "name", "record_type", "record_id", "blob_id", "created_at"]}).once
+    ActionCable.stubs(:server).returns(mock_server)
+
+    dugong.images.attach(io: image, filename: "dugong.jpg")
+  end
+
+  test "updates the collection when a file is destroyed" do
+    mock_server = mock("server")
+    image = File.open(Rails.root.join("test", "fixtures", "files", "dugong.jpg"))
+    dugong = Dugong.create
+    dugong.images.attach(io: image, filename: "dugong.jpg")
+
+    mock_server.expects(:broadcast).with("gid://dummy/Dugong/1:images", {changed: ["id", "name", "record_type", "record_id", "blob_id", "created_at"]}).once
+    ActionCable.stubs(:server).returns(mock_server)
+
+    dugong.images.first.destroy
+  end
 end