feat: add branches

.gitignore

@@ -28,3 +28,5 @@
 
 # Ignore master key for decrypting credentials and more.
 /config/master.key
+
+/config/application.yml

.rubocop.yml

@@ -6,13 +6,10 @@ AllCops:
     - bin/*
   TargetRubyVersion: 2.4
 
-Gemspec/RequiredRubyVersion:
-  Enabled: false
-
 Naming/AccessorMethodName:
   Enabled: false
 
-Style/RescueStandardError:
+Lint/RescueWithoutErrorClass:
   Enabled: false
 
 Metrics/LineLength:

Gemfile

@@ -3,6 +3,8 @@ git_source(:github) { |repo| "https://github.com/#{repo}.git" }
 
 ruby '2.4.1'
 
+gem 'figaro'
+
 # Bundle edge Rails instead: gem 'rails', github: 'rails/rails'
 gem 'rails', '~> 5.2.0'
 # Use sqlite3 as the database for Active Record
@@ -38,16 +40,16 @@ gem 'bootsnap', '>= 1.1.0', require: false
 
 group :development, :test do
   # Call 'byebug' anywhere in the code to stop execution and get a debugger console
-  gem 'byebug', platforms: [:mri, :mingw, :x64_mingw]
+  gem 'byebug', platforms: %i[mri mingw x64_mingw]
 end
 
 group :development do
   # Access an interactive console on exception pages or by calling 'console' anywhere in the code.
-  gem 'web-console', '>= 3.3.0'
   gem 'listen', '>= 3.0.5', '< 3.2'
   # Spring speeds up development by keeping your application running in the background. Read more: https://github.com/rails/spring
   gem 'spring'
   gem 'spring-watcher-listen', '~> 2.0.0'
+  gem 'web-console', '>= 3.3.0'
 end
 
 group :test do
@@ -59,4 +61,4 @@ group :test do
 end
 
 # Windows does not include zoneinfo files, so bundle the tzinfo-data gem
-gem 'tzinfo-data', platforms: [:mingw, :mswin, :x64_mingw, :jruby]
+gem 'tzinfo-data', platforms: %i[mingw mswin x64_mingw jruby]

Gemfile.lock

@@ -76,6 +76,8 @@ GEM
     erubi (1.7.1)
     execjs (2.7.0)
     ffi (1.9.25)
+    figaro (1.1.1)
+      thor (~> 0.14)
     globalid (0.4.1)
       activesupport (>= 4.2.0)
     i18n (1.0.1)
@@ -197,6 +199,7 @@ DEPENDENCIES
   capybara (>= 2.15, < 4.0)
   chromedriver-helper
   coffee-rails (~> 4.2)
+  figaro
   jbuilder (~> 2.5)
   listen (>= 3.0.5, < 3.2)
   puma (~> 3.11)

app/assets/javascripts/repository_github.coffee

@@ -0,0 +1,3 @@
+# Place all the behaviors and hooks related to the matching controller here.
+# All this logic will automatically be available in application.js.
+# You can use CoffeeScript in this file: http://coffeescript.org/

app/assets/stylesheets/repository_github.scss

@@ -0,0 +1,3 @@
+// Place all the styles related to the RepositoryGithub controller here.
+// They will automatically be included in application.css.
+// You can use Sass (SCSS) here: http://sass-lang.com/

app/controllers/github_repository_controller.rb

@@ -0,0 +1,21 @@
+class GithubRepositoryController < ApplicationController
+  def create
+    @software = Software.find(params[:software_id])
+
+    if @software.id == params[:repository][:software_id].to_i
+      @repository = GithubRepository.new(params.require(:repository).permit(:software_id, :name))
+      @repository.save
+    end
+
+    redirect_to edit_software_path(@software)
+  end
+
+  def update
+    @software = Software.find(params[:software_id])
+    @repository = GithubRepository.find(params[:id])
+
+    @repository.update(params.require(:repository).permit(:name)) if @software.id == @repository.software_id
+
+    redirect_to edit_software_path(@software)
+  end
+end

app/controllers/home_controller.rb

@@ -1,4 +1,3 @@
 class HomeController < ApplicationController
-  def index
+  def index; end
-  end
 end

app/controllers/softwares_controller.rb

@@ -1,4 +1,6 @@
 class SoftwaresController < ApplicationController
+  before_action :find_software, only: %i[show edit update destroy]
+
   def index
     @softwares = Software.all
   end
@@ -7,19 +9,24 @@ class SoftwaresController < ApplicationController
     @software = Software.new
   end
 
-  def show
+  def show; end
-    @software = Software.find(params[:id])
-  end
 
   def edit
-    @software = Software.find(params[:id])
+    case @software.repository_type
+    when 'Github'
+      if GithubRepository.exists?(software_id: @software)
+        @repository = GithubRepository.where(software_id: @software.id).take
+        @repository_form_path = software_github_repository_path(software_id: @software.id, id: @repository.id)
+      else
+        @repository = GithubRepository.new
+        @repository_form_path = software_github_repository_index_path(@software.id)
+      end
+    end
   end
 
   def update
-    @software = Software.find(params[:id])
-
     if @software.update(software_params)
-      redirect_to @software
+      redirect_to edit_software_path(@software)
     else
       render 'edit'
     end
@@ -29,14 +36,13 @@ class SoftwaresController < ApplicationController
     @software = Software.new(software_params)
 
     if @software.save
-      redirect_to @software
+      redirect_to edit_software_path(@software)
     else
       render 'new'
     end
   end
 
   def destroy
-    @software = Software.find(params[:id])
     @software.destroy
 
     redirect_to softwares_path
@@ -45,6 +51,10 @@ class SoftwaresController < ApplicationController
   private
 
   def software_params
-    params.require(:software).permit(:name, :website)
+    params.require(:software).permit(:name, :website, :repository_type)
+  end
+
+  def find_software
+    @software = Software.find(params[:id])
   end
 end

app/helpers/repository_github_helper.rb

@@ -0,0 +1,2 @@
+module RepositoryGithubHelper
+end

app/models/branch.rb

@@ -0,0 +1,6 @@
+class Branch < ApplicationRecord
+  belongs_to :software
+  has_many :versions
+  validates :name, presence: true
+  validates :name, uniqueness: { scope: :software }
+end

app/models/github_repository.rb

@@ -0,0 +1,5 @@
+class GithubRepository < ApplicationRecord
+  belongs_to :software
+  validates :name, presence: true, uniqueness: true
+  validates :software_id, presence: true, uniqueness: true
+end

app/models/software.rb

@@ -1,4 +1,4 @@
 class Software < ApplicationRecord
-  has_many :versions
+  has_many :branches
   validates :name, presence: true, uniqueness: true
 end

app/models/version.rb

@@ -1,6 +1,6 @@
 class Version < ApplicationRecord
-  belongs_to :software
+  belongs_to :branch
   validates :number, presence: true
   validates :date, presence: true
-  validates :number, uniqueness: { scope: :software }
+  validates :number, uniqueness: { scope: :branch }
 end

app/views/softwares/edit.html.erb

@@ -18,7 +18,33 @@
     <%= form.label :website, class: 'uk-form-label' %><br>
     <%= form.text_field :website, class: 'uk-input' %>
   </p>
+  <p>
+    <%= form.label :repository_type, class: 'uk-form-label' %><br>
+    <%= form.select :repository_type, [nil, 'Github'], {}, class: 'uk-select' %>
+  </p>
   <p>
     <%= form.submit class: 'uk-button uk-button-default uk-margin' %>
   </p>
 <% end %>
+
+<% case @software.repository_type %>
+<% when 'Github' %>
+<h4>Github</h4>
+<%= form_with(
+      scope: :repository,
+      url: @repository_form_path,
+      class: 'uk-form-horizontal uk-form-width-large',
+      model: @repository,
+      local: true
+    ) do |form|
+%>
+  <p>
+    <%= form.label :repository, class: 'uk-form-label' %><br>
+    <%= form.text_field :name, value: @repository.name, class: 'uk-input' %>
+  </p>
+  <p>
+    <%= form.hidden_field :software_id, value: @software.id %>
+    <%= form.submit class: 'uk-button uk-button-default uk-margin' %>
+  </p>
+<% end %>
+<% end %>

app/views/softwares/new.html.erb

@@ -18,6 +18,10 @@
     <%= form.label :website, class: 'uk-form-label' %><br>
     <%= form.text_field :website, class: 'uk-input' %>
   </p>
+  <p>
+    <%= form.label :repository_type, class: 'uk-form-label' %><br>
+    <%= form.select :repository_type, [nil, 'Github'], {}, class: 'uk-select' %>
+  </p>
   <p>
     <%= form.submit class: 'uk-button uk-button-default uk-margin' %>
   </p>

app/views/softwares/show.html.erb

@@ -5,6 +5,8 @@
   <%= link_to 'Delete', software_path(@software), method: :delete, class: 'uk-icon-link', data: { confirm: 'Are you sure?' } %>
 </p>
 <h3><%= @software.name %></h3>
+<% @software.branches.order(name: :desc).each do |branch| %>
+<h4><%= branch.name %></h4>
 <table class="uk-table uk-table-striped">
   <thead>
     <tr>
@@ -12,10 +14,11 @@
       <th>Date</th>
     </tr>
   </thead>
-<% @software.versions.order(date: :desc).each do |version| %>
+<% branch.versions.order(date: :desc).each do |version| %>
   <tr>
     <td><%= version.number %></td>
     <td><%= version.date.strftime('%Y-%m-%d') %></td>
   </tr>
 <% end %>
 </table>
+<% end %>

config/routes.rb

@@ -1,4 +1,6 @@
 Rails.application.routes.draw do
   root 'home#index'
-  resources :softwares
+  resources :softwares do
+    resources :github_repository
+  end
 end

db/migrate/20180729143537_add_column_repository_type_in_software.rb

@@ -0,0 +1,7 @@
+class AddColumnRepositoryTypeInSoftware < ActiveRecord::Migration[5.2]
+  def change
+    change_table :softwares do |t|
+      t.string :repository_type
+    end
+  end
+end

db/migrate/20180729191829_create_github_repositories.rb

@@ -0,0 +1,10 @@
+class CreateGithubRepositories < ActiveRecord::Migration[5.2]
+  def change
+    create_table :github_repositories do |t|
+      t.string :name, uniqueness: true, null: false
+      t.references :software, uniqueness: true, foreign_key: true
+
+      t.timestamps
+    end
+  end
+end

db/migrate/20180804072602_create_branches.rb

@@ -0,0 +1,10 @@
+class CreateBranches < ActiveRecord::Migration[5.2]
+  def change
+    create_table :branches do |t|
+      t.string :name, null: false
+      t.references :software, foreign_key: true
+
+      t.timestamps
+    end
+  end
+end

db/migrate/20180805081706_change_reference_to_version.rb

@@ -0,0 +1,8 @@
+class ChangeReferenceToVersion < ActiveRecord::Migration[5.2]
+  def change
+    remove_reference :versions, :software, index: true, foreign_key: true
+    change_table :versions do |t|
+      t.references :branch, foreign_key: true
+    end
+  end
+end

db/schema.rb

@@ -10,22 +10,40 @@
 #
 # It's strongly recommended that you check this file into your version control system.
 
-ActiveRecord::Schema.define(version: 2018_07_22_183046) do
+ActiveRecord::Schema.define(version: 2018_08_05_081706) do
+
+  create_table "branches", force: :cascade do |t|
+    t.string "name", null: false
+    t.integer "software_id"
+    t.datetime "created_at", null: false
+    t.datetime "updated_at", null: false
+    t.index ["software_id"], name: "index_branches_on_software_id"
+  end
+
+  create_table "github_repositories", force: :cascade do |t|
+    t.string "name", null: false
+    t.integer "software_id"
+    t.datetime "created_at", null: false
+    t.datetime "updated_at", null: false
+    t.index ["software_id"], name: "index_github_repositories_on_software_id"
+  end
 
   create_table "softwares", force: :cascade do |t|
     t.string "name", null: false
     t.string "website"
     t.datetime "created_at", null: false
     t.datetime "updated_at", null: false
+    t.string "repository_type"
+    t.string "branch_pattern"
   end
 
   create_table "versions", force: :cascade do |t|
     t.string "number", null: false
     t.datetime "date", null: false
-    t.integer "software_id"
     t.datetime "created_at", null: false
     t.datetime "updated_at", null: false
-    t.index ["software_id"], name: "index_versions_on_software_id"
+    t.integer "branch_id"
+    t.index ["branch_id"], name: "index_versions_on_branch_id"
   end
 
 end

lib/tasks/repositories.rake

@@ -0,0 +1,72 @@
+require 'net/https'
+
+namespace :repositories do
+  def get_url_content(url, headers = {})
+    uri = URI.parse(url)
+    req = Net::HTTP::Get.new(uri)
+
+    headers.each do |header, value|
+      req[header] = value
+    end
+
+    Net::HTTP.start(uri.hostname, uri.port, use_ssl: true) do |http|
+      http.request(req)
+    end.body
+  end
+
+  def branch(software_id, branches, version)
+    name = version.match(/^v?(?<branch>\d+\.\d+)(\.\d+)?$/)[:branch]
+
+    return branches[name] if branches.include?(name)
+
+    branch = Branch.new(
+      name: name,
+      software_id: software_id
+    )
+    branch.save
+    branches[name] = branch.id
+    branch.id
+  end
+
+  desc 'Update version from github repositories'
+  task github: :environment do
+    headers = {
+      'Accept' => 'application/vnd.github.v3+json'
+    }
+    headers['Authorization'] = "token #{Figaro.env.github_token}" if Figaro.env.github_token
+
+    GithubRepository.all.each do |repo|
+      begin
+        software = Software.find(repo.software_id)
+        branches = software.branches.all.map { |b| [b.name, b.id] }.to_h
+        versions = software.branches.all.map { |b| b.versions.all.map(&:number) }
+        tags =
+          JSON.parse(
+            get_url_content(
+              "https://api.github.com/repos/#{repo.name}/tags",
+              headers
+            )
+          )
+
+        tags.each do |tag|
+          next if versions.include?(tag['name'])
+
+          branch_id = branch(software.id, branches, tag['name'])
+          date =
+            JSON.parse(
+              get_url_content(tag['commit']['url'], headers)
+            )['commit']['committer']['date']
+
+          puts Version.new(
+            branch_id: branch_id,
+            number: tag['name'],
+            date: date
+          ).save
+        end
+      rescue => e
+        puts e.strace
+        next
+      end
+    end
+  end
+end

test/application_system_test_case.rb

@@ -1,4 +1,4 @@
-require "test_helper"
+require 'test_helper'
 
 class ApplicationSystemTestCase < ActionDispatch::SystemTestCase
   driven_by :selenium, using: :chrome, screen_size: [1400, 1400]

test/controllers/github_repository_controller_test.rb

@@ -0,0 +1,63 @@
+require 'test_helper'
+
+class GithubRepositoryControllerTest < ActionDispatch::IntegrationTest
+  test 'should not save repository without data' do
+    repository = GithubRepository.new
+    assert_not repository.save
+  end
+
+  test 'should not save repository without software_id' do
+    data = { name: 'test/test' }
+    repository = GithubRepository.new(data)
+
+    assert_not repository.save
+  end
+
+  test 'should not save repository if a repository already exist' do
+    data = {
+      name: 'test/test',
+      software_id: Software.find_by_name('Ruby on Rails').id
+    }
+    repository = GithubRepository.new(data)
+
+    assert_not repository.save
+  end
+
+  test 'should not save repository if a repository has the same name' do
+    data = {
+      name: 'rails/rails',
+      software_id: Software.find_by_name('Kaiho').id
+    }
+    repository = GithubRepository.new(data)
+
+    assert_not repository.save
+  end
+
+  test 'should save repository' do
+    data = {
+      name: 'kaiho/kaiho',
+      software_id: Software.find_by_name('Kaiho').id
+    }
+    repository = GithubRepository.new(data)
+
+    assert repository.save
+  end
+
+  test 'should not update repository if a repository has the same name' do
+    data = {
+      name: 'rails/rails'
+    }
+    repository = GithubRepository.find_by_name('ruby/ruby')
+
+    assert_not repository.update(data)
+  end
+
+  test 'should update repository' do
+    data = {
+      name: 'rails/rails2'
+    }
+    repository = GithubRepository.find_by_name('rails/rails')
+
+    assert repository.update(data)
+  end
+end

test/fixtures/branches.yml

@@ -0,0 +1,9 @@
+# Read about fixtures at http://api.rubyonrails.org/classes/ActiveRecord/FixtureSet.html
+
+one:
+  name: 1.0
+  software: two 
+
+two:
+  name: 1.1
+  software: two

test/fixtures/github_repositories.yml

@@ -0,0 +1,9 @@
+# Read about fixtures at http://api.rubyonrails.org/classes/ActiveRecord/FixtureSet.html
+
+one:
+  name: rails/rails
+  software: two
+
+two:
+  name: ruby/ruby
+  software: three

test/fixtures/softwares.yml

@@ -7,3 +7,6 @@ one:
 two:
   name: Ruby on Rails
   website: https://rubyonrails.org
+
+three:
+  name: Ruby

test/fixtures/versions.yml

@@ -3,9 +3,9 @@
 one:
   number: 1.0.0
   date: 2018-07-22 20:30:46
-  software: two
+  branch: one
 
 two:
   number: 1.1.0
   date: 2018-12-22 20:30:46
-  software: two
+  branch: two

test/models/branch_test.rb

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

test/models/github_repository_test.rb

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

test/models/repository_github_test.rb

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