feat: add github repository
This commit is contained in:
parent
5154b4b632
commit
d2b621ddd6
16 changed files with 172 additions and 3 deletions
3
app/assets/javascripts/repository_github.coffee
Normal file
3
app/assets/javascripts/repository_github.coffee
Normal file
|
@ -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/
|
3
app/assets/stylesheets/repository_github.scss
Normal file
3
app/assets/stylesheets/repository_github.scss
Normal file
|
@ -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/
|
23
app/controllers/github_repository_controller.rb
Normal file
23
app/controllers/github_repository_controller.rb
Normal file
|
@ -0,0 +1,23 @@
|
||||||
|
class GithubRepositoryController < ApplicationController
|
||||||
|
def create
|
||||||
|
@software = Software.find(params[:software_id])
|
||||||
|
@repository = GithubRepository.new(repository_params)
|
||||||
|
@repository.save
|
||||||
|
|
||||||
|
redirect_to edit_software_path(@software)
|
||||||
|
end
|
||||||
|
|
||||||
|
def update
|
||||||
|
@software = Software.find(params[:software_id])
|
||||||
|
@repository = GithubRepository.find(params[:id])
|
||||||
|
@repository.update(repository_params)
|
||||||
|
|
||||||
|
redirect_to edit_software_path(@software)
|
||||||
|
end
|
||||||
|
|
||||||
|
private
|
||||||
|
|
||||||
|
def repository_params
|
||||||
|
params.require(:repository).permit(:name, :software_id)
|
||||||
|
end
|
||||||
|
end
|
|
@ -13,6 +13,17 @@ class SoftwaresController < ApplicationController
|
||||||
|
|
||||||
def edit
|
def edit
|
||||||
@software = Software.find(params[:id])
|
@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(@repository)
|
||||||
|
else
|
||||||
|
@repository = GithubRepository.new
|
||||||
|
@repository_form_path = software_github_repository_index_path(@software.id)
|
||||||
|
end
|
||||||
|
end
|
||||||
end
|
end
|
||||||
|
|
||||||
def update
|
def update
|
||||||
|
@ -45,6 +56,6 @@ class SoftwaresController < ApplicationController
|
||||||
private
|
private
|
||||||
|
|
||||||
def software_params
|
def software_params
|
||||||
params.require(:software).permit(:name, :website)
|
params.require(:software).permit(:name, :website, :repository_type)
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
2
app/helpers/repository_github_helper.rb
Normal file
2
app/helpers/repository_github_helper.rb
Normal file
|
@ -0,0 +1,2 @@
|
||||||
|
module RepositoryGithubHelper
|
||||||
|
end
|
5
app/models/github_repository.rb
Normal file
5
app/models/github_repository.rb
Normal file
|
@ -0,0 +1,5 @@
|
||||||
|
class GithubRepository < ApplicationRecord
|
||||||
|
belongs_to :software
|
||||||
|
validates :name, presence: true, uniqueness: true
|
||||||
|
validates :software_id, presence: true, uniqueness: true
|
||||||
|
end
|
|
@ -18,7 +18,33 @@
|
||||||
<%= form.label :website, class: 'uk-form-label' %><br>
|
<%= form.label :website, class: 'uk-form-label' %><br>
|
||||||
<%= form.text_field :website, class: 'uk-input' %>
|
<%= form.text_field :website, class: 'uk-input' %>
|
||||||
</p>
|
</p>
|
||||||
|
<p>
|
||||||
|
<%= form.label :repository_type, class: 'uk-form-label' %><br>
|
||||||
|
<%= form.select :repository_type, %w[Github], {}, class: 'uk-select' %>
|
||||||
|
</p>
|
||||||
<p>
|
<p>
|
||||||
<%= form.submit class: 'uk-button uk-button-default uk-margin' %>
|
<%= form.submit class: 'uk-button uk-button-default uk-margin' %>
|
||||||
</p>
|
</p>
|
||||||
<% end %>
|
<% 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 %>
|
||||||
|
|
|
@ -18,6 +18,10 @@
|
||||||
<%= form.label :website, class: 'uk-form-label' %><br>
|
<%= form.label :website, class: 'uk-form-label' %><br>
|
||||||
<%= form.text_field :website, class: 'uk-input' %>
|
<%= form.text_field :website, class: 'uk-input' %>
|
||||||
</p>
|
</p>
|
||||||
|
<p>
|
||||||
|
<%= form.label :repository_type, class: 'uk-form-label' %><br>
|
||||||
|
<%= form.select :repository_type, %w[Github], {}, class: 'uk-select' %>
|
||||||
|
</p>
|
||||||
<p>
|
<p>
|
||||||
<%= form.submit class: 'uk-button uk-button-default uk-margin' %>
|
<%= form.submit class: 'uk-button uk-button-default uk-margin' %>
|
||||||
</p>
|
</p>
|
||||||
|
|
|
@ -1,4 +1,6 @@
|
||||||
Rails.application.routes.draw do
|
Rails.application.routes.draw do
|
||||||
root 'home#index'
|
root 'home#index'
|
||||||
resources :softwares
|
resources :softwares do
|
||||||
|
resources :github_repository
|
||||||
|
end
|
||||||
end
|
end
|
||||||
|
|
|
@ -0,0 +1,7 @@
|
||||||
|
class AddColumnRepositoryTypeInSoftware < ActiveRecord::Migration[5.2]
|
||||||
|
def change
|
||||||
|
change_table :softwares do |t|
|
||||||
|
t.string :repository_type
|
||||||
|
end
|
||||||
|
end
|
||||||
|
end
|
10
db/migrate/20180729191829_create_github_repositories.rb
Normal file
10
db/migrate/20180729191829_create_github_repositories.rb
Normal file
|
@ -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
|
11
db/schema.rb
11
db/schema.rb
|
@ -10,13 +10,22 @@
|
||||||
#
|
#
|
||||||
# It's strongly recommended that you check this file into your version control system.
|
# 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_07_29_191829) do
|
||||||
|
|
||||||
|
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|
|
create_table "softwares", force: :cascade do |t|
|
||||||
t.string "name", null: false
|
t.string "name", null: false
|
||||||
t.string "website"
|
t.string "website"
|
||||||
t.datetime "created_at", null: false
|
t.datetime "created_at", null: false
|
||||||
t.datetime "updated_at", null: false
|
t.datetime "updated_at", null: false
|
||||||
|
t.string "repository_type"
|
||||||
end
|
end
|
||||||
|
|
||||||
create_table "versions", force: :cascade do |t|
|
create_table "versions", force: :cascade do |t|
|
||||||
|
|
45
test/controllers/github_repository_controller_test.rb
Normal file
45
test/controllers/github_repository_controller_test.rb
Normal file
|
@ -0,0 +1,45 @@
|
||||||
|
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 with the same name exist' 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
|
||||||
|
end
|
5
test/fixtures/github_repositories.yml
vendored
Normal file
5
test/fixtures/github_repositories.yml
vendored
Normal file
|
@ -0,0 +1,5 @@
|
||||||
|
# Read about fixtures at http://api.rubyonrails.org/classes/ActiveRecord/FixtureSet.html
|
||||||
|
|
||||||
|
one:
|
||||||
|
name: rails/rails
|
||||||
|
software: two
|
7
test/models/github_repository_test.rb
Normal file
7
test/models/github_repository_test.rb
Normal file
|
@ -0,0 +1,7 @@
|
||||||
|
require 'test_helper'
|
||||||
|
|
||||||
|
class GithubRepositoryTest < ActiveSupport::TestCase
|
||||||
|
# test "the truth" do
|
||||||
|
# assert true
|
||||||
|
# end
|
||||||
|
end
|
7
test/models/repository_github_test.rb
Normal file
7
test/models/repository_github_test.rb
Normal file
|
@ -0,0 +1,7 @@
|
||||||
|
require 'test_helper'
|
||||||
|
|
||||||
|
class RepositoryGithubTest < ActiveSupport::TestCase
|
||||||
|
# test "the truth" do
|
||||||
|
# assert true
|
||||||
|
# end
|
||||||
|
end
|
Loading…
Reference in a new issue