contentful_bootstrap 1.6.0 → 2.0.2

data/lib/contentful/bootstrap/templates/links.rb

@@ -1,2 +1,2 @@
-require "contentful/bootstrap/templates/links/entry"
-require "contentful/bootstrap/templates/links/asset"
+require 'contentful/bootstrap/templates/links/entry'
+require 'contentful/bootstrap/templates/links/asset'

data/lib/contentful/bootstrap/templates/links/asset.rb

@@ -1,5 +1,5 @@
-require "contentful/management"
-require "contentful/bootstrap/templates/links/base"
+require 'contentful/management'
+require 'contentful/bootstrap/templates/links/base'
 
 module Contentful
   module Bootstrap

data/lib/contentful/bootstrap/templates/links/base.rb

@@ -1,4 +1,4 @@
-require "contentful/management"
+require 'contentful/management'
 
 module Contentful
   module Bootstrap
@@ -11,7 +11,7 @@ module Contentful
           end
 
           def link_type
-            self.class.name.split("::").last
+            self.class.name.split('::').last
           end
 
           def type
@@ -25,7 +25,7 @@ module Contentful
           end
 
           def management_class
-            raise "must implement"
+            fail 'must implement'
           end
         end
       end

data/lib/contentful/bootstrap/templates/links/entry.rb

@@ -1,5 +1,5 @@
-require "contentful/management"
-require "contentful/bootstrap/templates/links/base"
+require 'contentful/management'
+require 'contentful/bootstrap/templates/links/base'
 
 module Contentful
   module Bootstrap

data/lib/contentful/bootstrap/token.rb

@@ -1,62 +1,63 @@
-require "inifile"
+require 'inifile'
 
 module Contentful
   module Bootstrap
     class Token
-      CONFIG_ENV = "CONTENTFUL_ENV".freeze
-      DEFAULT_SECTION = "global".freeze
-      DEFAULT_PATH = ".contentfulrc".freeze
-      MANAGEMENT_TOKEN = "CONTENTFUL_MANAGEMENT_ACCESS_TOKEN".freeze
-      DELIVERY_TOKEN = "CONTENTFUL_DELIVERY_ACCESS_TOKEN".freeze
-      SPACE_ID = "SPACE_ID".freeze
+      CONFIG_ENV = 'CONTENTFUL_ENV'.freeze
+      DEFAULT_SECTION = 'global'.freeze
+      DEFAULT_PATH = '.contentfulrc'.freeze
+      MANAGEMENT_TOKEN = 'CONTENTFUL_MANAGEMENT_ACCESS_TOKEN'.freeze
+      DELIVERY_TOKEN = 'CONTENTFUL_DELIVERY_ACCESS_TOKEN'.freeze
+      SPACE_ID = 'SPACE_ID'.freeze
 
-      def self.set_path!(config_path = "")
-        @@config_path = config_path
+      attr_reader :config_path
+
+      def initialize(config_path = '')
+        @config_path = config_path
+      end
+
+      def ==(other_token)
+        return false unless other_token.is_a?(Contentful::Bootstrap::Token)
+        other_token.config_path == @config_path
       end
 
-      def self.present?
-        return false unless File.exists? filename
-        config_file[config_section].has_key? MANAGEMENT_TOKEN
+      def present?
+        return false unless ::File.exist? filename
+        config_file[config_section].key? MANAGEMENT_TOKEN
       end
 
-      def self.read
-        begin
-          config_file[config_section].fetch(MANAGEMENT_TOKEN)
-        rescue KeyError
-          fail "Token not found"
-        end
+      def read
+        config_file[config_section].fetch(MANAGEMENT_TOKEN)
+      rescue KeyError
+        raise 'Token not found'
       end
 
-      def self.write(value, section = nil, key = MANAGEMENT_TOKEN)
+      def write(value, section = nil, key = MANAGEMENT_TOKEN)
         file = config_file
         file[section || config_section][key] = value
         file.save
       end
 
-      def self.write_access_token(space_name, token)
+      def write_access_token(space_name, token)
         write(token, space_name, DELIVERY_TOKEN)
       end
 
-      def self.write_space_id(space_name, space_id)
+      def write_space_id(space_name, space_id)
         write(space_id, space_name, SPACE_ID)
       end
 
-      def self.filename
-        return config_path if File.exist?(config_path)
-        File.join(ENV['HOME'], DEFAULT_PATH)
+      def filename
+        return config_path unless config_path.empty?
+        ::File.join(ENV['HOME'], DEFAULT_PATH)
       end
 
-      def self.config_section
+      def config_section
         return ENV[CONFIG_ENV] if config_file.has_section? ENV[CONFIG_ENV]
         DEFAULT_SECTION
       end
 
-      def self.config_file
-        File.exist?(filename) ? IniFile.load(filename) : IniFile.new(filename: filename)
-      end
-
-      def self.config_path
-        @@config_path ||= ""
+      def config_file
+        @file ||= ::File.exist?(filename) ? IniFile.load(filename) : IniFile.new(filename: filename)
       end
     end
   end

data/lib/contentful/bootstrap/version.rb

@@ -1,5 +1,5 @@
 module Contentful
   module Bootstrap
-    VERSION = "1.6.0"
+    VERSION = '2.0.2'
   end
 end

data/spec/contentful/bootstrap/command_runner_spec.rb

@@ -0,0 +1,111 @@
+require 'spec_helper'
+
+describe Contentful::Bootstrap::CommandRunner do
+  let(:path) { File.expand_path(File.join('spec', 'fixtures', 'ini_fixtures', 'contentfulrc.ini')) }
+  subject { Contentful::Bootstrap::CommandRunner.new(path) }
+
+  describe 'instance methods' do
+    before do
+    end
+    describe '#create_space' do
+      before do
+        allow_any_instance_of(Contentful::Bootstrap::Commands::CreateSpace).to receive(:run)
+      end
+
+      it 'default values' do
+        expect(Contentful::Bootstrap::Commands::CreateSpace).to receive(:new).with(
+          subject.token, 'foo', nil, nil, true
+        ).and_call_original
+
+        subject.create_space('foo')
+      end
+
+      it 'with options' do
+        expect(Contentful::Bootstrap::Commands::CreateSpace).to receive(:new).with(
+          subject.token, 'foo', 'bar', 'baz', false
+        ).and_call_original
+
+        subject.create_space('foo', template: 'bar', json_template: 'baz', trigger_oauth: false)
+      end
+
+      it 'runs command' do
+        expect_any_instance_of(Contentful::Bootstrap::Commands::CreateSpace).to receive(:run)
+
+        subject.create_space('foo', template: 'bar', json_template: 'baz', trigger_oauth: false)
+      end
+    end
+
+    describe '#generate_token' do
+      before do
+        allow_any_instance_of(Contentful::Bootstrap::Commands::GenerateToken).to receive(:run)
+        allow_any_instance_of(Contentful::Bootstrap::Commands::GenerateToken).to receive(:fetch_space)
+      end
+
+      it 'default values' do
+        expect(Contentful::Bootstrap::Commands::GenerateToken).to receive(:new).with(
+          subject.token, 'foo', 'Bootstrap Token', true
+        ).and_call_original
+
+        subject.generate_token('foo')
+      end
+
+      it 'with options' do
+        expect(Contentful::Bootstrap::Commands::GenerateToken).to receive(:new).with(
+          subject.token, 'foo', 'bar', false
+        ).and_call_original
+
+        subject.generate_token('foo', name: 'bar', trigger_oauth: false)
+      end
+
+      it 'runs command' do
+        expect_any_instance_of(Contentful::Bootstrap::Commands::GenerateToken).to receive(:run)
+
+        subject.generate_token('foo', name: 'bar', trigger_oauth: false)
+      end
+    end
+
+    describe '#generate_json' do
+      it 'requires access token to run' do
+        expect {
+          subject.generate_json('foo')
+        }.to raise_error('Access Token required')
+      end
+
+      it 'default values' do
+        allow_any_instance_of(Contentful::Bootstrap::Commands::GenerateJson).to receive(:run)
+
+        expect(Contentful::Bootstrap::Commands::GenerateJson).to receive(:new).with(
+          'foo', 'bar', nil
+        ).and_call_original
+
+        subject.generate_json('foo', access_token: 'bar')
+      end
+
+      it 'with options' do
+        allow_any_instance_of(Contentful::Bootstrap::Commands::GenerateJson).to receive(:run)
+
+        expect(Contentful::Bootstrap::Commands::GenerateJson).to receive(:new).with(
+          'foo', 'bar', 'baz'
+        ).and_call_original
+
+        subject.generate_json('foo', access_token: 'bar', filename: 'baz')
+      end
+
+      it 'runs command' do
+        expect_any_instance_of(Contentful::Bootstrap::Commands::GenerateJson).to receive(:run)
+
+        subject.generate_json('foo', access_token: 'bar', filename: 'baz')
+      end
+    end
+  end
+
+  describe 'attributes' do
+    it ':config_path' do
+      expect(subject.config_path).to eq path
+    end
+
+    it ':token' do
+      expect(subject.token == Contentful::Bootstrap::Token.new(path)).to be_truthy
+    end
+  end
+end

data/spec/contentful/bootstrap/commands/base_spec.rb

@@ -0,0 +1,102 @@
+require 'spec_helper'
+require 'net/http'
+require 'contentful/bootstrap/commands/base'
+
+describe Contentful::Bootstrap::Commands::Base do
+  let(:path) { File.expand_path(File.join('spec', 'fixtures', 'ini_fixtures', 'contentfulrc.ini')) }
+  let(:no_token_path) { File.expand_path(File.join('spec', 'fixtures', 'ini_fixtures', 'no_token.ini')) }
+  let(:token) { Contentful::Bootstrap::Token.new path }
+  let(:non_management_token) { Contentful::Bootstrap::Token.new no_token_path }
+  subject { Contentful::Bootstrap::Commands::Base.new token, 'foo', false }
+
+  describe 'abstract methods' do
+    it '#run' do
+      expect { subject.run }.to raise_error 'must implement'
+    end
+  end
+
+  describe 'initialize' do
+    describe 'configuration' do
+      it 'runs configuration when trigger_oauth is true' do
+        expect_any_instance_of(subject.class).to receive(:configuration)
+        subject.class.new(token, 'foo')
+      end
+
+      it 'doesnt run configuration when trigger_oauth is false' do
+        expect_any_instance_of(subject.class).not_to receive(:configuration)
+        subject.class.new(token, 'foo', false)
+      end
+    end
+
+    describe 'management_client_init' do
+      it 'runs management_client_init when trigger_oauth is true' do
+        expect_any_instance_of(subject.class).to receive(:management_client_init)
+        subject.class.new(token, 'foo')
+      end
+
+      it 'doesnt run management_client_init when trigger_oauth is false' do
+        expect_any_instance_of(subject.class).not_to receive(:management_client_init)
+        subject.class.new(token, 'foo', false)
+      end
+    end
+  end
+
+  describe 'instance methods' do
+    it '#management_client_init' do
+      allow_any_instance_of(subject.class).to receive(:configuration)
+      expect(Contentful::Management::Client).to receive(:new).with(token.read, raise_errors: true)
+
+      subject.class.new(token, 'foo')
+    end
+
+    describe '#configuration' do
+      before do
+        allow_any_instance_of(subject.class).to receive(:management_client_init)
+      end
+
+      it 'passes if token is found' do
+        expect { subject.class.new(token, 'foo') }.to output("OAuth token found, moving on!\n").to_stdout
+      end
+
+      describe 'token not found' do
+        it 'exits if answer is no' do
+          expect_any_instance_of(subject.class).to receive(:gets) { "n\n" }
+          expect { subject.class.new(non_management_token, 'foo') }.to raise_error SystemExit
+        end
+
+        it 'runs token_server if other answer' do
+          expect_any_instance_of(subject.class).to receive(:gets) { "y\n" }
+          expect_any_instance_of(subject.class).to receive(:token_server)
+
+          subject.class.new(non_management_token, 'foo')
+        end
+      end
+    end
+
+    it '#token_server' do
+      allow_any_instance_of(subject.class).to receive(:management_client_init)
+
+      expect_any_instance_of(subject.class).to receive(:gets) { "y\n" }
+      expect_any_instance_of(Contentful::Bootstrap::Server).to receive(:start) {}
+      expect_any_instance_of(Contentful::Bootstrap::Server).to receive(:running?) { true }
+      expect(Net::HTTP).to receive(:get).with(URI('http://localhost:5123')) {}
+      expect(non_management_token).to receive(:present?).and_return(false, true)
+      expect_any_instance_of(Contentful::Bootstrap::Server).to receive(:stop) {}
+
+      subject.class.new(non_management_token, 'foo')
+    end
+  end
+
+  describe 'attributes' do
+    it ':space' do
+      expect(subject.space).to eq 'foo'
+    end
+
+    it ':token' do
+      expect(subject.token == Contentful::Bootstrap::Token.new(path)).to be_truthy
+    end
+  end
+
+  describe '#initialize' do
+  end
+end

data/spec/contentful/bootstrap/commands/create_space_spec.rb

@@ -0,0 +1,72 @@
+require 'spec_helper'
+
+describe Contentful::Bootstrap::Commands::CreateSpace do
+  let(:path) { File.expand_path(File.join('spec', 'fixtures', 'ini_fixtures', 'contentfulrc.ini')) }
+  let(:token) { Contentful::Bootstrap::Token.new path }
+  subject { Contentful::Bootstrap::Commands::CreateSpace.new token, 'foo', 'bar', 'baz', false }
+  let(:space_double) { SpaceDouble.new }
+
+  describe 'instance methods' do
+    describe '#run' do
+      it 'with all non nil attributes' do
+        expect(subject).to receive(:fetch_space) { space_double }
+        expect(subject).to receive(:create_template).with(space_double)
+        expect(subject).to receive(:create_json_template).with(space_double)
+        expect(subject).to receive(:generate_token).with(space_double)
+
+        subject.run
+      end
+
+      it 'does not create template when template_name is nil' do
+        create_space_command = subject.class.new(token, 'foo', nil, 'baz', false)
+
+        expect(create_space_command.template_name).to eq nil
+
+        expect(create_space_command).to receive(:fetch_space) { space_double }
+        expect(create_space_command).not_to receive(:create_template).with(space_double)
+        expect(create_space_command).to receive(:create_json_template).with(space_double)
+        expect(create_space_command).to receive(:generate_token).with(space_double)
+
+        create_space_command.run
+      end
+
+      it 'does not create json template when json_template is nil' do
+        create_space_command = subject.class.new(token, 'foo', 'bar', nil, false)
+
+        expect(create_space_command.json_template).to eq nil
+
+        expect(create_space_command).to receive(:fetch_space) { space_double }
+        expect(create_space_command).to receive(:create_template).with(space_double)
+        expect(create_space_command).not_to receive(:create_json_template).with(space_double)
+        expect(create_space_command).to receive(:generate_token).with(space_double)
+
+        create_space_command.run
+      end
+    end
+  end
+
+  describe 'attributes' do
+    it ':template_name' do
+      expect(subject.template_name).to eq 'bar'
+    end
+
+    it ':json_template' do
+      expect(subject.json_template).to eq 'baz'
+    end
+  end
+
+  describe 'issues' do
+    it 'Importing asset array values does not work #22' do
+      json_path = File.expand_path(File.join('spec', 'fixtures', 'json_fixtures', 'issue_22.json'))
+
+      allow_any_instance_of(subject.class).to receive(:gets).and_return('y')
+      allow_any_instance_of(Contentful::Bootstrap::Commands::GenerateToken).to receive(:gets).and_return('n')
+
+      command = subject.class.new(token, 'issue_22', nil, json_path)
+
+      vcr('issue_22') {
+        command.run
+      }
+    end
+  end
+end

data/spec/contentful/bootstrap/commands/generate_json_spec.rb

@@ -0,0 +1,64 @@
+require 'spec_helper'
+
+describe Contentful::Bootstrap::Commands::GenerateJson do
+  subject { Contentful::Bootstrap::Commands::GenerateJson.new('foo', 'bar') }
+
+  describe 'instance methods' do
+    describe '#run' do
+      it 'exits if access_token is nil' do
+        subject.instance_variable_set(:@access_token, nil)
+
+        expect { subject.run }.to raise_error SystemExit
+      end
+
+      it 'calls generator' do
+        allow(subject).to receive(:write).with('json')
+
+        expect_any_instance_of(Contentful::Bootstrap::Generator).to receive(:generate_json) { 'json' }
+
+        subject.run
+      end
+
+      it 'writes json' do
+        vcr('generate_json') {
+          subject.instance_variable_set(:@space_id, 'wl1z0pal05vy')
+          subject.instance_variable_set(:@access_token, '48d7db7d4cd9d09df573c251d456f4acc72141b92f36e57f8684b36cf5cfff6e')
+
+          json_fixture('wl1z0pal05vy') { |json|
+            expect(subject).to receive(:write).with(JSON.pretty_generate(json))
+          }
+
+          subject.run
+        }
+      end
+    end
+
+    describe '#write' do
+      it 'outputs json to stoud if no filename provided' do
+        expect { subject.write('json') }.to output("json\n").to_stdout
+      end
+
+      it 'writes to file if filename provided' do
+        subject.instance_variable_set(:@filename, 'foo')
+
+        expect(::File).to receive(:write).with('foo', 'json')
+
+        expect { subject.write('json') }.to output("Saving JSON template to 'foo'\n").to_stdout
+      end
+    end
+  end
+
+  describe 'attributes' do
+    it ':space_id' do
+      expect(subject.space_id).to eq 'foo'
+    end
+
+    it ':access_token' do
+      expect(subject.access_token).to eq 'bar'
+    end
+
+    it ':filename' do
+      expect(subject.filename).to eq nil
+    end
+  end
+end