Integrate twitter #21

.gitignore

@@ -68,3 +68,8 @@ bower.json
 
 # yarn
 yarn-error.log
+
+config/settings.local.yml
+config/settings/*.local.yml
+config/environments/*.local.yml
+config/settings/development.yml

Gemfile

@@ -2,19 +2,17 @@ source 'https://rubygems.org'
 
 gem 'better_errors'
 gem 'binding_of_caller'
-gem 'byebug'
-
-# Bundle edge Rails instead: gem 'rails', github: 'rails/rails'
-gem 'rails', '~> 5.0.0', '>= 5.0.0.1'
-# Use Puma as the app server
-gem 'puma', '~> 3.0'
-
-gem 'npm-pipeline-rails'
-gem 'slim'
-gem 'sass-rails'
 gem 'bourbon'
+gem 'byebug'
+gem 'config'
 gem 'neat'
 gem 'normalize-rails'
+gem 'npm-pipeline-rails'
+gem 'puma', '~> 3.0'
+gem 'rails', '~> 5.0.0', '>= 5.0.0.1'
+gem 'sass-rails'
+gem 'slim'
+gem 'twitter'
 
 group :test do
   gem 'codecov', require: false
@@ -23,6 +21,7 @@ end
 group :development, :test do
   gem 'pry'
   gem 'pry-rails'
+  gem 'rspec-rails'
 end
 
 group :development do
@@ -33,4 +32,3 @@ group :development do
   gem 'spring-watcher-listen', '~> 2.0.0'
   gem 'web-console'
 end
-gem 'rspec-rails', group: [:development, :test]

Gemfile.lock

@@ -38,6 +38,8 @@ GEM
       i18n (~> 0.7)
       minitest (~> 5.1)
       tzinfo (~> 1.1)
+    addressable (2.5.0)
+      public_suffix (~> 2.0, >= 2.0.2)
     arel (7.1.4)
     ast (2.3.0)
     better_errors (2.1.1)
@@ -49,6 +51,7 @@ GEM
     bourbon (4.3.2)
       sass (~> 3.4)
       thor (~> 0.19)
+    buftok (0.2.0)
     builder (3.2.3)
     byebug (9.0.6)
     codecov (0.1.8)
@@ -57,13 +60,31 @@ GEM
       url
     coderay (1.1.1)
     concurrent-ruby (1.0.4)
+    config (1.4.0)
+      activesupport (>= 3.0)
+      deep_merge (~> 1.1.1)
     debug_inspector (0.0.2)
+    deep_merge (1.1.1)
     diff-lcs (1.2.5)
     docile (1.1.5)
+    domain_name (0.5.20170223)
+      unf (>= 0.0.5, < 1.0.0)
+    equalizer (0.0.11)
     erubis (2.7.0)
+    faraday (0.11.0)
+      multipart-post (>= 1.2, < 3)
     ffi (1.9.14)
     globalid (0.3.7)
       activesupport (>= 4.1.0)
+    http (2.2.1)
+      addressable (~> 2.3)
+      http-cookie (~> 1.0)
+      http-form_data (~> 1.0.1)
+      http_parser.rb (~> 0.6.0)
+    http-cookie (1.0.3)
+      domain_name (~> 0.5)
+    http-form_data (1.0.1)
+    http_parser.rb (0.6.0)
     i18n (0.8.0)
     json (2.0.2)
     listen (3.0.8)
@@ -73,12 +94,16 @@ GEM
       nokogiri (>= 1.5.9)
     mail (2.6.4)
       mime-types (>= 1.16, < 4)
+    memoizable (0.4.2)
+      thread_safe (~> 0.3, >= 0.3.1)
     method_source (0.8.2)
     mime-types (3.1)
       mime-types-data (~> 3.2015)
     mime-types-data (3.2016.0521)
     mini_portile2 (2.1.0)
     minitest (5.10.1)
+    multipart-post (2.0.0)
+    naught (1.1.0)
     neat (1.8.0)
       sass (>= 3.3)
       thor (~> 0.19)
@@ -98,6 +123,7 @@ GEM
       slop (~> 3.4)
     pry-rails (0.3.4)
       pry (>= 0.9.10)
+    public_suffix (2.0.5)
     puma (3.6.0)
     rack (2.0.1)
     rack-test (0.6.3)
@@ -162,6 +188,7 @@ GEM
       sprockets (>= 2.8, < 4.0)
       sprockets-rails (>= 2.0, < 4.0)
       tilt (>= 1.1, < 3)
+    simple_oauth (0.3.1)
     simplecov (0.12.0)
       docile (~> 1.1.0)
       json (>= 1.8, < 3)
@@ -187,8 +214,21 @@ GEM
     thor (0.19.4)
     thread_safe (0.3.5)
     tilt (2.0.5)
+    twitter (6.1.0)
+      addressable (~> 2.5)
+      buftok (~> 0.2.0)
+      equalizer (= 0.0.11)
+      faraday (~> 0.11.0)
+      http (~> 2.1)
+      http_parser.rb (~> 0.6.0)
+      memoizable (~> 0.4.2)
+      naught (~> 1.1)
+      simple_oauth (~> 0.3.1)
     tzinfo (1.2.2)
       thread_safe (~> 0.1)
+    unf (0.1.4)
+      unf_ext
+    unf_ext (0.0.7.2)
     unicode-display_width (1.1.1)
     url (0.3.2)
     web-console (3.3.1)
@@ -209,6 +249,7 @@ DEPENDENCIES
   bourbon
   byebug
   codecov
+  config
   listen (~> 3.0.5)
   neat
   normalize-rails
@@ -224,7 +265,8 @@ DEPENDENCIES
   slim
   spring
   spring-watcher-listen (~> 2.0.0)
+  twitter
   web-console
 
 BUNDLED WITH
-   1.13.7
+   1.14.3

app/controllers/api/oceans_controller.rb

@@ -0,0 +1,25 @@
+class Api::OceansController < ApplicationController
+  protect_from_forgery with: :null_session
+  before_filter :render_bad_request, unless: :limit_valid?
+
+  def index
+    oceans = Array.new(number_of_oceans) do
+      OceanService.new(Random.rand(1..5)).call
+    end
+    render json: oceans
+  end
+
+  private
+
+  def number_of_oceans
+    @number_of_oceans ||= params.fetch(:limit, 4).to_i
+  end
+
+  def render_bad_request
+    render nothing: true, status: :bad_request
+  end
+
+  def limit_valid?
+    number_of_oceans.between?(0, 20)
+  end
+end

app/services/twitter_service.rb

@@ -0,0 +1,13 @@
+class TwitterService
+  APP_CONFIG = YAML.load(File.read('config/settings/development.yml'))
+                   .with_indifferent_access
+
+  def initialize
+    @client = Twitter::REST::Client.new do |config|
+      config.consumer_key        = APP_CONFIG['consumer_key']
+      config.consumer_secret     = APP_CONFIG['consumer_secret']
+      config.access_token        = APP_CONFIG['access_token']
+      config.access_token_secret = APP_CONFIG['access_token_secret']
+    end
+  end
+end

config/initializers/config.rb

@@ -0,0 +1,36 @@
+Config.setup do |config|
+  # Name of the constant exposing loaded settings
+  config.const_name = 'Settings'
+
+  # Ability to remove elements of the array set in earlier loaded settings file. For example value: '--'.
+  #
+  # config.knockout_prefix = nil
+
+  # Overwrite arrays found in previously loaded settings file. When set to `false`, arrays will be merged.
+  #
+  # config.overwrite_arrays = true
+
+  # Load environment variables from the `ENV` object and override any settings defined in files.
+  #
+  # config.use_env = false
+
+  # Define ENV variable prefix deciding which variables to load into config.
+  #
+  # config.env_prefix = 'Settings'
+
+  # What string to use as level separator for settings loaded from ENV variables. Default value of '.' works well
+  # with Heroku, but you might want to change it for example for '__' to easy override settings from command line, where
+  # using dots in variable names might not be allowed (eg. Bash).
+  #
+  # config.env_separator = '.'
+
+  # Ability to process variables names:
+  #   * nil  - no change
+  #   * :downcase - convert to lower case
+  #
+  # config.env_converter = :downcase
+
+  # Parse numeric values as integers instead of strings.
+  #
+  # config.env_parse_values = true
+end

config/routes.rb

@@ -1,5 +1,8 @@
 Rails.application.routes.draw do
-  resources :islands, only: :index
   root 'islands#index'
+  resources :islands, only: :index
   resources :test_oceans, only: :index
+  namespace :api, defaults: { format: 'json' } do
+    resources :oceans, only: :index
+  end
 end

spec/controllers/api/oceans_controller_spec.rb

@@ -0,0 +1,31 @@
+require 'rails_helper'
+
+RSpec.describe Api::OceansController do
+  let(:json_response) { JSON.parse(response.body) }
+  let(:oceans) { json_response }
+  let(:ocean_service) { instance_double(OceanService, call: 'ocean') }
+
+  it 'renders 4 oceans, when there are no additional parameters' do
+    get :index
+    expect(response).to be_success
+    expect(oceans.size).to be 4
+  end
+
+  it 'renders 0 oceans' do
+    get :index, limit: '0'
+    expect(response).to be_success
+    expect(oceans.size).to be
+  end
+
+  it 'calls the OceanService with 4 as limit' do
+    expect(OceanService)
+      .to(receive(:new).exactly(4).times.and_return(ocean_service))
+    get :index, limit: '4'
+    expect(oceans).to eq %w(ocean ocean ocean ocean)
+  end
+
+  it 'calls the ocean with forbidden parameters (< 0 | > 20)' do
+    get :index, limit: '21'
+    expect(response).to be_bad_request
+  end
+end