superset 0.2.6 → 0.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: ddf012f64fba5d3e1fe7584f41fc5ddcbdf9b20dea686efca47ae99e9e228c58
-  data.tar.gz: 9295923ac58ea7f24d92e978f94d8bbcbb6c06dd4cc821de4d67c3b57ebd08d0
+  metadata.gz: f9be8bc52c7852af239721ed6e260a12e068bb35042ae0d7a78507d39c498017
+  data.tar.gz: 3ddd9c26deca3cc72bd046d83435d22383f2986cee3bcfdebe9e4db0f0749ea3
 SHA512:
-  metadata.gz: 563f5e16113862396196f5b0bbfe8fc96b2edce69d14089e464592f3770a9da156d50ebacc2098e6b4fc1a7766c568571d25a33507db86c2641d92ac5f518ea7
-  data.tar.gz: 8a02610f9d42328bff42753363ddb651924af87da82abfcd9311a16dd8ab9293e1f035d4f6e60e097e7f82c1b28165d5a009dfae4de0b1c0b1d8b16a2f00f704
+  metadata.gz: b1e1e71dace1acffd37407100c583f9bb61e29fcffafabdd13bcad834c49366b190a068fbe90b529c809e3c8d786afb9a370656ac47d96adfd740985dbbe9917
+  data.tar.gz: bee9ae33375f128d4d9abc11259d9f300ac281c188ec90c26088af85c8620e10350ba44a1f1645403b9994fcadc8760e79d94098e373a2a07666f6991dee418a

data/.rubocop.yml

@@ -1,5 +1,5 @@
 AllCops:
-  TargetRubyVersion: 2.6
+  TargetRubyVersion: 3.0
 
 Style/StringLiterals:
   Enabled: true

data/.ruby-version

@@ -1 +1 @@
-2.7.8
+3.4.4

data/CHANGELOG.md

@@ -1,5 +1,21 @@
 ## Change Log
 
+## 0.3.0 - 2025-11-26
+* Rename reserved object_id to target_id in https://github.com/rdytech/superset-client/pull/58
+* Add dashboard cascade ownship in https://github.com/rdytech/superset-client/pull/59
+* allow dup dashboard dataset suffix naming override in https://github.com/rdytech/superset-client/pull/61
+* Bump ruby 3 bump gems by https://github.com/rdytech/superset-client/pull/60
+
+
+## 0.2.7 - 2025-10-16
+* add to_h method to list classes for Hash output
+* add ids method to list classes for array of ids output
+* add ability to over ride default page_size
+* add ruby console auto timeout set to 30 mins by default
+* add GetCatalogs endpoint caller
+* fix catalog bug related to dashboard export and import across env
+* add tag_equals param to dashboard list class
+
 ## 0.2.6 - 2025-08-05
 * Bump terminal-table from 1.8.0 to 4.0.0
 

data/Dockerfile

@@ -1,5 +1,5 @@
-# 1: Use ruby 2.7 as base:
-FROM ruby:2.7
+# 1: Use ruby 3.4.4 as base:
+FROM ruby:3.4.4
 
 RUN apt-get update && \
     apt-get install -y --no-install-recommends \

data/README.md

@@ -22,11 +22,14 @@ Build, bundle and open a ruby console
 docker-compose build
 docker-compose run --rm app bundle install
 docker-compose run --rm app bin/console
+
+# note .. windows users may need to call ruby the bin/console file
+docker-compose run --rm app ruby bin/console
 ```
 
-## Setup Locally ( no docker ) 
+## Setup Locally ( no docker )
 
-Requires Ruby >= 2.6.0 
+Requires Ruby >= 3
 
 Bundle install and open a ruby console.
 
@@ -37,7 +40,7 @@ bin/console
 
 ## Including in a Ruby app
 
-Add to your Gemfile `gem 'superset'`  
+Add to your Gemfile `gem 'superset'`
 And then execute: `bundle install`  
 Or install it yourself as `gem install superset`
 

data/doc/duplicate_dashboards.md

@@ -36,7 +36,7 @@ then you could go ahead and run something like this.
 ```ruby
 Superset::Services::DuplicateDashboard.new(
     source_dashboard_id: 90,
-    target_schema: 'public',
+    target_schema: 'client_1',
     target_database_id: 2
   ).perform
 
@@ -47,12 +47,12 @@ Superset::Services::DuplicateDashboard.new(
 # cat log/superset-client.log
 # INFO -- : >>>>>>>>>>>>>>>>> Starting DuplicateDashboard Service <<<<<<<<<<<<<<<<<<<<<<
 # INFO -- : Source Dashboard URL: https://your-superset-host/superset/dashboard/90/
-# INFO -- : Duplicating dashboard 90 into Target Schema: public in database 2
+# INFO -- : Duplicating dashboard 90 into Target Schema: client_1 in database 2
 # INFO -- :   Copy Dashboard/Charts Completed - New Dashboard ID: 401
 # INFO -- : Duplicating Source Dataset examples.video_game_sales with id 11
-# INFO -- :     Finished. Duplicate Dataset Name video_game_sales-example_two with id 542
-# INFO -- :     Validating Dataset ID: 542 schema update to public on Database: 2
-# INFO -- :     Successfully updated dataset schema to public on Database: 2
+# INFO -- :     Finished. Duplicate Dataset Name video_game_sales-example_two-client_1 with id 542
+# INFO -- :     Validating Dataset ID: 542 schema update to client_1 on Database: 2
+# INFO -- :     Successfully updated dataset schema to client_1 on Database: 2
 # INFO -- : Updating Charts to point to New Datasets and updating Dashboard json_metadata ...
 # INFO -- :   Update Chart 55752 to new dataset_id 542
 # INFO -- :   Updated new Dashboard json_metadata charts with new dataset ids
@@ -71,14 +71,53 @@ If your using the embedded dashboards you can also provied attributes for
 ```ruby
 Superset::Services::DuplicateDashboard.new(
     source_dashboard_id: 37,
-    target_schema: 'public',
+    target_schema: 'client_1',
     target_database_id: 2,
     allowed_domains: ["https://wylee-coyote-domain/"],
     tags: ["product:acme_fu", "client:wylee_coyote", "embedded"],
     publish: true
   ).perform
+
+
+
+```
+
+## Duplication when DB schemas are not used
+
+If your multitenant setup is at the database level, and database structures are replicated into the same base schema, ie public,
+then the target resulted datasets name must be overridden in order to get around the superset unique datasets name validation restriction.  This can be done through the parameter 'target_dataset_suffix_override'.
+
+```ruby
+Superset::Services::DuplicateDashboard.new(
+    source_dashboard_id: 90,
+    target_schema: 'public',
+    target_database_id: 2,
+    target_dataset_suffix_override: 'client_1' 
+  ).perform
+
 ```
 
+## Duplication now supports catalogs
+
+If you Database infrastructure uses catalogs, you can provide the target catalog as a paramater.
+If there is only 1 catalog in the target database, then it will be used by default.
+
+If multiple catalogs exist, and no catalog option is provided, an error will be raised.
+
+```ruby
+Superset::Services::DuplicateDashboard.new(
+    source_dashboard_id: 90,
+    target_schema: 'public',
+    target_database_id: 2,
+    target_dataset_suffix_override: 'client_1',
+    target_catalog_name: 'staging_client_1',
+  ).perform
+
+```
+
+
+
+
 ### What is my Database ID ?
 
 ``` ruby

data/lib/superset/base_put_request.rb

@@ -1,9 +1,9 @@
 module Superset
   class BasePutRequest < Superset::Request
-    attr_reader :object_id, :params
+    attr_reader :target_id, :params
 
-    def initialize(object_id: ,params: )
-      @object_id = object_id
+    def initialize(target_id: ,params: )
+      @target_id = target_id
       @params = params
     end
 
@@ -19,7 +19,7 @@ module Superset
     private
 
     def validate
-      raise "Error: object_id integer is required" unless object_id.present? && object_id.is_a?(Integer)
+      raise "Error: target_id integer is required" unless target_id.present? && target_id.is_a?(Integer)
       raise "Error: params hash is required" unless params.present? && params.is_a?(Hash)
     end
 

data/lib/superset/chart/bulk_delete.rb

@@ -18,7 +18,7 @@ module Superset
         raise InvalidParameterError, "chart_ids array of integers expected" unless chart_ids.is_a?(Array)
         raise InvalidParameterError, "chart_ids array must contain Integer only values" unless chart_ids.all? { |item| item.is_a?(Integer) }
 
-        logger.info("Attempting to delete charts with id: #{chart_ids.join(', ')}")
+        logger.info("Deleting charts with id: #{chart_ids.join(', ')}")
         response
       end
 

data/lib/superset/chart/delete.rb

@@ -12,7 +12,7 @@ module Superset
       def perform
         raise InvalidParameterError, "chart_id integer is required" unless chart_id.present? && chart_id.is_a?(Integer)
 
-        logger.info("Attempting to delete chart with id: #{chart_id}")
+        logger.info("Deleting chart with id: #{chart_id}")
         response
       end
 

data/lib/superset/chart/get.rb

@@ -17,28 +17,24 @@ module Superset
         self
       end
 
-      def result
-        [ super ]
-      end
-
       def datasource_id
-        if result.first['query_context'].present? && JSON.parse(result.first['query_context'])['datasource'].present?
-          JSON.parse(result.first['query_context'])['datasource']['id']
-        elsif result.first['params'].present? && JSON.parse(result.first['params'])['datasource'].present?
-          JSON.parse(result.first['params'])['datasource'].match(/^\d+/)[0].to_i
+        if result['query_context'].present? && JSON.parse(result['query_context'])['datasource'].present?
+          JSON.parse(result['query_context'])['datasource']['id']
+        elsif result['params'].present? && JSON.parse(result['params'])['datasource'].present?
+          JSON.parse(result['params'])['datasource'].match(/^\d+/)[0].to_i
         end
       end
 
       def owner_ids
-        result.first['owners'].map{|o| o['id']}
+        result['owners'].map{|o| o['id']}
       end
 
       def params
-        JSON.parse(result.first['params']) if result.first['params'].present?
+        JSON.parse(result['params']) if result['params'].present?
       end
 
       def query_context
-        JSON.parse(result.first['query_context']) if result.first['query_context'].present?
+        JSON.parse(result['query_context']) if result['query_context'].present?
       end
 
       private

data/lib/superset/chart/list.rb

@@ -2,13 +2,13 @@ module Superset
   module Chart
     class List < Superset::Request
 
-      attr_reader :name_contains, :dashboard_id_eq, :dataset_id_eq
+      attr_reader :title_contains, :dashboard_id_eq, :dataset_id_eq
 
-      def initialize(page_num: 0, name_contains: '', dashboard_id_eq: '', dataset_id_eq: '')
-        @name_contains = name_contains
+      def initialize(title_contains: '', dashboard_id_eq: '', dataset_id_eq: '', **kwargs)
+        @title_contains = title_contains
         @dashboard_id_eq = dashboard_id_eq
         @dataset_id_eq = dataset_id_eq
-        super(page_num: page_num)
+        super(**kwargs)
       end
 
       def self.call
@@ -38,7 +38,7 @@ module Superset
       def filters
         # TODO filtering across all classes needs a refactor to support multiple options in a more flexible way
         filter_set = []
-        filter_set << "(col:slice_name,opr:ct,value:'#{name_contains}')" if name_contains.present?
+        filter_set << "(col:slice_name,opr:ct,value:'#{title_contains}')" if title_contains.present?
         filter_set << "(col:dashboards,opr:rel_m_m,value:#{dashboard_id_eq})" if dashboard_id_eq.present? # rel many to many
         filter_set << "(col:datasource_id,opr:eq,value:#{dataset_id_eq})" if dataset_id_eq.present?
 

data/lib/superset/chart/put.rb

@@ -2,7 +2,7 @@
 #
 # Usage:
 # params = { owners: [ 58, 3 ] }
-# Superset::Chart::Put.new(object_id: 202, params: params ).perform
+# Superset::Chart::Put.new(target_id: 202, params: params ).perform
 
 module Superset
   module Chart
@@ -11,7 +11,7 @@ module Superset
       private
 
       def route
-        "chart/#{object_id}"
+        "chart/#{target_id}"
       end
     end
   end

data/lib/superset/dashboard/bulk_delete.rb

@@ -20,7 +20,7 @@ module Superset
         raise InvalidParameterError, "dashboard_ids array of integers expected" unless dashboard_ids.is_a?(Array)
         raise InvalidParameterError, "dashboard_ids array must contain Integer only values" unless dashboard_ids.all? { |item| item.is_a?(Integer) }
 
-        logger.info("Attempting to delete dashboards with id: #{dashboard_ids.join(', ')}")
+        logger.info("Deleting dashboards with id: #{dashboard_ids.join(', ')}")
         response
       end
 

data/lib/superset/dashboard/cascade_ownership/add_new_owner.rb

@@ -0,0 +1,54 @@
+module Superset
+  module Dashboard
+    module CascadeOwnership
+      class AddNewOwner < Superset::Request
+        attr_reader :dashboard_id, :user_id
+
+        def initialize(dashboard_id:, user_id:)
+          @dashboard_id = dashboard_id
+          @user_id = user_id
+        end
+
+        def perform
+          raise "Error: dashboard_id integer is required" unless dashboard_id.present? && dashboard_id.is_a?(Integer)
+          raise "Error: user_id integer is required" unless user_id.present? && user_id.is_a?(Integer)
+
+          add_user_to_dashboard_ownership
+          add_user_to_charts_ownership
+          add_user_to_datasets_ownership
+        end
+
+        private
+
+        def add_user_to_dashboard_ownership
+          return if current_dashboard_owner_ids.include?(user_id)
+          Superset::Dashboard::Put.new(target_id: dashboard_id, params: { "owners":  current_dashboard_owner_ids << user_id }).perform
+        end
+
+        def add_user_to_charts_ownership
+          chart_ids = Superset::Dashboard::Charts::List.new(dashboard_id).ids
+          chart_ids.each do |chart_id|
+            current_chart_owner_ids = Superset::Chart::Get.new(chart_id).result['owners'].map{|c| c['id']}
+            next if current_chart_owner_ids.include?(user_id)
+
+            Superset::Chart::Put.new(target_id: chart_id, params: { "owners":  current_chart_owner_ids << user_id }).perform
+          end
+        end
+
+        def add_user_to_datasets_ownership
+          dataset_ids = Superset::Dashboard::Datasets::List.new(dashboard_id: dashboard_id).ids
+          dataset_ids.each do |dataset_id|
+            current_dataset_owner_ids = Superset::Dataset::Get.new(dataset_id).result['owners'].map{|c| c['id']}
+            next if current_dataset_owner_ids.include?(user_id)
+
+            Superset::Dataset::Put.new(target_id: dataset_id, params: { "owners":  current_dataset_owner_ids << user_id }).perform
+          end
+        end
+
+        def current_dashboard_owner_ids
+          @current_dashboard_owner_ids ||= Superset::Dashboard::Get.new(dashboard_id).result['owners'].map{|i| i['id'] }
+        end
+      end
+    end
+  end
+end

data/lib/superset/dashboard/datasets/list.rb

@@ -30,7 +30,7 @@ module Superset
             # For the current superset setup we will assume a dashboard datasets will point to EXACTLY one schema, their own.
             # if not .. we need to know about it. Potentially we could override this check if others do not consider it a problem.
             if all_dashboard_schemas.count > 1
-              Rollbar.error("SUPERSET DASHBOARD ERROR: Dashboard id #{id} has multiple dataset schema linked: #{all_dashboard_schemas.to_s}")
+              Rollbar.error("SUPERSET DASHBOARD ERROR: Dashboard id #{id} has multiple dataset schema linked: #{all_dashboard_schemas.to_s}") if defined?(Rollbar)
             end
             all_dashboard_schemas
           end
@@ -48,6 +48,20 @@ module Superset
           chart_datasets + filter_datasets(filter_dataset_ids_not_used_in_charts)
         end
 
+        def rows
+          datasets_details.map do |d|
+            [
+              d[:id],
+              d[:datasource_name],
+              d[:database][:id],
+              d[:database][:name],
+              d[:database][:backend],
+              d[:schema],
+              d[:filter_only]
+            ]
+          end
+        end
+
         private
 
         def filter_dataset_ids
@@ -74,20 +88,6 @@ module Superset
           ['id', 'datasource_name', 'database_id', 'database_name', 'database_backend', 'schema', 'filter_only'].map(&:to_sym)
         end
 
-        def rows
-          datasets_details.map do |d|
-            [
-              d[:id],
-              d[:datasource_name],
-              d[:database][:id],
-              d[:database][:name],
-              d[:database][:backend],
-              d[:schema],
-              d[:filter_only]
-            ]
-          end
-        end
-
         # when displaying a list of datasets, show dashboard title as well
         def title
           @title ||= [id, dashboard.title].join(' ')

data/lib/superset/dashboard/delete.rb

@@ -16,7 +16,7 @@ module Superset
 
         confirm_zero_charts_on_dashboard if confirm_zero_charts
 
-        logger.info("Attempting to delete dashboard with id: #{dashboard_id}")
+        logger.info("Deleting dashboard with id: #{dashboard_id}")
         response
       end
 

data/lib/superset/dashboard/list.rb

@@ -5,15 +5,18 @@
 module Superset
   module Dashboard
     class List < Superset::Request
-      attr_reader :title_contains, :title_equals, :tags_equal, :ids_not_in, :include_filter_dataset_schemas
+      attr_reader :title_contains, :title_equals,
+                  :tags_contain, :tags_equal,
+                  :ids_not_in, :include_filter_dataset_schemas
 
-      def initialize(page_num: 0, title_contains: '', title_equals: '', tags_equal: [], ids_not_in: [], include_filter_dataset_schemas: false)
+      def initialize(title_contains: '', title_equals: '', tags_contain: [], tags_equal: [], ids_not_in: [], include_filter_dataset_schemas: false, **kwargs)
         @title_contains = title_contains
         @title_equals = title_equals
+        @tags_contain = tags_contain
         @tags_equal = tags_equal
         @ids_not_in = ids_not_in
         @include_filter_dataset_schemas = include_filter_dataset_schemas
-        super(page_num: page_num)
+        super(**kwargs)
       end
 
       def self.call
@@ -57,8 +60,16 @@ module Superset
         end
       end
 
+      def to_h
+        result.map do |d|
+          list_attributes.to_h do |la|
+            la == :url ? [la, "#{superset_host}#{d[la]}"] : [la, d[la]]
+          end
+        end
+      end
+
       def ids
-        result.map { |d| d[:id] }
+        result.is_a?(Hash) ? [result[:id]] : result.map { |d| d[:id] }
       end
 
       private
@@ -72,15 +83,30 @@ module Superset
         filter_set = []
         filter_set << "(col:dashboard_title,opr:ct,value:'#{title_contains}')" if title_contains.present?
         filter_set << "(col:dashboard_title,opr:eq,value:'#{title_equals}')" if title_equals.present?
-        filter_set << tag_filters if tags_equal.present?
+        filter_set << tags_contain_filters if tags_contain.present?
+        filter_set << tags_equal_filters if tags_equal.present?
         filter_set << ids_not_in_filters if ids_not_in.present?
+
         unless filter_set.empty?
           "filters:!(" + filter_set.join(',') + "),"
         end
       end
 
-      def tag_filters
-        tags_equal.map {|tag| "(col:tags,opr:dashboard_tags,value:'#{tag}')"}.join(',')
+      def tags_equal_filters
+        tags_equal_ids.map { |id| "(col:tags,opr:dashboard_tag_id,value:#{id})" }.join(',')
+      end
+
+      def tags_equal_ids
+        tags_equal.map do |tag_name|
+          ids = Superset::Tag::List.new(name_equals: tag_name).rows.map(&:first)
+          raise "No ID found for tag: #{tag_name}" if ids.empty?
+          raise "Multiple IDs found for tag: #{tag_name}" if ids.size > 1
+          ids.first
+        end
+      end
+
+      def tags_contain_filters
+        tags_contain.map {|tag| "(col:tags,opr:dashboard_tags,value:'#{tag}')"}.join(',')
       end
 
       def ids_not_in_filters
@@ -94,8 +120,8 @@ module Superset
       def validate_constructor_args
         raise InvalidParameterError, "title_contains must be a String type" unless title_contains.is_a?(String)
         raise InvalidParameterError, "title_equals must be a String type" unless title_equals.is_a?(String)
-        raise InvalidParameterError, "tags_equal must be an Array type" unless tags_equal.is_a?(Array)
-        raise InvalidParameterError, "tags_equal array must contain string only values" unless tags_equal.all? { |item| item.is_a?(String) }
+        raise InvalidParameterError, "tags_contain must be an Array type of String values" unless tags_contain.is_a?(Array) && tags_contain.all? { |item| item.is_a?(String) }
+        raise InvalidParameterError, "tags_equal must be an Array type of String values" unless tags_equal.is_a?(Array) && tags_equal.all? { |item| item.is_a?(String) }
         raise InvalidParameterError, "ids_not_in must be an Array type" unless ids_not_in.is_a?(Array)
       end
     end

data/lib/superset/dashboard/put.rb

@@ -1,36 +1,19 @@
 
 # frozen_string_literal: true
 
+# Example Usage: Update Dashboard Ownership to a new set of users
+# this will override the existing owners and set the new owners
+# params = { owners: [ 58, 3 ] }
+# Superset::Dashboard::Put.new(target_id: 101, params: params ).perform
+
 module Superset
   module Dashboard
-    class Put < Superset::Request
-
-      attr_reader :target_dashboard_id, :params
-
-      def initialize(target_dashboard_id:, params:)
-        @target_dashboard_id = target_dashboard_id
-        @params = params
-      end
-
-      def perform
-        raise "Error: target_dashboard_id integer is required" unless target_dashboard_id.present? && target_dashboard_id.is_a?(Integer)
-        raise "Error: params hash is required" unless params.present? && params.is_a?(Hash)
-
-        response
-      end
-
-      def response
-        @response ||= client.put(route, params)
-      end
-
-      def id
-        response["result"]["id"]
-      end
+    class Put < Superset::BasePutRequest
 
       private
 
       def route
-        "dashboard/#{target_dashboard_id}"
+        "dashboard/#{target_id}"
       end
     end
   end

data/lib/superset/dashboard/warm_up_cache.rb

@@ -19,7 +19,7 @@ module Superset
           begin
             warm_up_dataset(dataset["datasource_name"], dataset["name"])
           rescue => e
-            Rollbar.error("Warm up cache failed for the dashboard #{dashboard_id.to_s} and for the dataset #{dataset["datasource_name"]} - #{e}")
+            Rollbar.error("Warm up cache failed for the dashboard #{dashboard_id.to_s} and for the dataset #{dataset["datasource_name"]} - #{e}") if defined?(Rollbar)
           end 
         end
       end

data/lib/superset/database/get_catalogs.rb

@@ -0,0 +1,38 @@
+module Superset
+  module Database
+    class GetCatalogs < Superset::Request
+
+      attr_reader :id, :include_system_catalogs
+
+      def initialize(id, include_system_catalogs: false)
+        @id = id
+        @include_system_catalogs = include_system_catalogs
+      end
+
+      def self.call(id)
+        self.new(id).catalogs
+      end
+ 
+      def catalogs
+        return result if include_system_catalogs
+
+        remove_system_catalogs
+      end
+
+      private
+
+      def route
+        "database/#{id}/catalogs/"
+      end
+
+      # exclude system catalog values for certain databases that support them
+      def remove_system_catalogs
+        result - postgres_system_catalogs
+      end
+
+      def postgres_system_catalogs
+        %w(postgres rdsadmin template1 template0)
+      end
+    end
+  end
+end

data/lib/superset/database/list.rb

@@ -6,10 +6,10 @@ module Superset
     class List < Superset::Request
       attr_reader :title_contains, :uuid_equals
 
-      def initialize(page_num: 0, title_contains: '', uuid_equals: '')
+      def initialize(title_contains: '', uuid_equals: '', **kwargs)
         @title_contains = title_contains
         @uuid_equals = uuid_equals
-        super(page_num: page_num)
+        super(**kwargs)
       end
 
       def self.call

data/lib/superset/dataset/bulk_delete.rb

@@ -19,7 +19,7 @@ module Superset
         raise InvalidParameterError, "dataset_ids array of integers expected" unless dataset_ids.is_a?(Array)
         raise InvalidParameterError, "dataset_ids array must contain Integer only values" unless dataset_ids.all? { |item| item.is_a?(Integer) }
 
-        logger.info("Attempting to delete datasets with id: #{dataset_ids.join(', ')}")
+        logger.info("Deleting datasets with id: #{dataset_ids.join(', ')}")
         response
       end
 

data/lib/superset/dataset/delete.rb

@@ -12,7 +12,7 @@ module Superset
       def perform
         raise InvalidParameterError, "dataset_id integer is required" unless dataset_id.present? && dataset_id.is_a?(Integer)
 
-        logger.info("Attempting to delete dataset with id: #{dataset_id}")
+        logger.info("Deleting dataset with id: #{dataset_id}")
         response
       end
 

data/lib/superset/dataset/get.rb

@@ -18,7 +18,11 @@ module Superset
       end
 
       def rows
-        [ [title, schema, database_name, database_id] ]
+        [to_h.values]
+      end
+
+      def to_h
+        list_attributes.to_h { |a| [a, send(a)] }
       end
 
       def schema
@@ -26,7 +30,7 @@ module Superset
       end
 
       def title
-        result['name']
+        result['table_name']
       end
 
       def database_name
@@ -47,8 +51,8 @@ module Superset
         "dataset/#{id}"
       end
 
-      def display_headers
-        %w[title schema database_name, database_id]
+      def list_attributes
+        [:title, :schema, :database_name, :database_id]
       end
     end
   end