Merging pull request #94 from fga-gpp-mds/issue_91

Author: alaxalves

app/controllers/projects_controller.rb

@@ -80,6 +80,6 @@ def set_project
     end
 
     def project_params
-      params.require(:project).permit(:name, :description, :check_project, :user_id)
+      params.require(:project).permit(:name, :description, :is_project_from_github, :user_id)
     end
 end

app/models/project.rb

@@ -4,5 +4,5 @@ class Project < ApplicationRecord
 
   validates :name, presence: true, length: { maximum: 128, minimum: 2 }
   validates :description, length: { maximum: 256 }
-  validates :check_project, inclusion: { in: [true, false] }
+  validates :is_project_from_github, inclusion: { in: [true, false] }
 end

db/migrate/20171030030726_add_check_project_to_projects.rb

@@ -0,0 +1,5 @@
+class AddIsProjectFromGithubToProjects < ActiveRecord::Migration[5.1]
+  def change
+    add_column :projects, :is_project_from_github, :boolean
+  end
+end

db/migrate/20171107172605_add_is_project_from_github_to_projects.rb

@@ -10,7 +10,7 @@
 #
 # It's strongly recommended that you check this file into your version control system.
 
-ActiveRecord::Schema.define(version: 20171105003057) do
+ActiveRecord::Schema.define(version: 20171107172605) do
 
   # These are extensions that must be enabled in order to support this database
   enable_extension "plpgsql"
@@ -21,7 +21,7 @@
     t.datetime "created_at", null: false
     t.datetime "updated_at", null: false
     t.bigint "user_id"
-    t.boolean "check_project"
+    t.boolean "is_project_from_github"
     t.index ["user_id"], name: "index_projects_on_user_id"
   end
 

db/schema.rb

@@ -32,13 +32,13 @@
     name: "Owla",
     description: "This project helps improving classes",
     user_id: "1",
-    check_project: true
+    is_project_from_github: true
   ),
   Project.find_or_create_by(
     name: "Falko",
     description: "Agile Projects Manager",
     user_id: "2",
-    check_project: true
+    is_project_from_github: true
   )
 ]
 

db/seeds.rb

@@ -14,14 +14,14 @@ def setup
       "name": "Falko",
       "description": "Some project description 1.",
       "user_id": @user.id,
-      "check_project": true
+      "is_project_from_github": true
     )
 
     @project2 = Project.create(
       "name": "Falko",
       "description": "Some project description 2.",
       "user_id": @user.id,
-      "check_project": false
+      "is_project_from_github": false
     )
 
     @token = AuthenticateUser.call(@user.email, @user.password)
@@ -33,7 +33,7 @@ def setup
           "name": "Falko",
           "description": "Some project description.",
           "user_id": @user.id,
-          "check_project": true
+          "is_project_from_github": true
         }
       }, headers: { Authorization: @token.result }
 
@@ -47,7 +47,7 @@ def setup
         "project": {
           "name": "",
           "description": "A" * 260,
-          "check_project": true
+          "is_project_from_github": true
         }
       }, headers: { Authorization: @token.result }
 
@@ -69,7 +69,7 @@ def setup
         project: {
           "name": "Falko BackEnd",
           "description": "Falko BackEnd!",
-          "check_project": "true"
+          "is_project_from_github": "true"
         }
       }, headers: { Authorization: @token.result }
       @project.reload
@@ -87,7 +87,7 @@ def setup
         project: {
           "name": "a",
           "description": "a",
-          "check_project": "false"
+          "is_project_from_github": "false"
         }
       }, headers: { Authorization: @token.result }
       @project.reload
@@ -221,7 +221,7 @@ def mock.organization_repositories(login)
     end
   end
 
-  test "should not import a project from github if the check_project is invalid" do
+  test "should not import a project from github if the is_project_from_github is invalid" do
       post "/users/#{@user.id}/projects", params: {
         "project": {
           "name": "Falko",

test/controllers/projects_controller_test.rb

@@ -14,7 +14,7 @@ def setup
       name: "Falko",
       description: "Description.",
       user_id: @user.id,
-      check_project: true
+      is_project_from_github: true
     )
 
     @release = Release.create(
@@ -39,7 +39,7 @@ def setup
       name: "Futebol",
       description: "Description.",
       user_id: @user.id,
-      check_project: true
+      is_project_from_github: true
     )
 
     @another_release = Release.create(

test/controllers/releases_controller_test.rb

@@ -13,7 +13,7 @@ def setup
     @project = Project.create(
       name: "Falko",
       description: "A Project.",
-      check_project: true,
+      is_project_from_github: true,
       user_id: @user.id
     )
 
@@ -63,7 +63,7 @@ def setup
     @another_project = Project.create(
       name: "Falko",
       description: "Project Description.",
-      check_project: true,
+      is_project_from_github: true,
       user_id: @another_user.id
     )
 

test/controllers/retrospectives_controller_test.rb

@@ -14,7 +14,7 @@ def setup
       name: "Falko",
       description: "Some project description.",
       user_id: @user.id,
-      check_project: true
+      is_project_from_github: true
     )
 
     @release = Release.create(
@@ -48,7 +48,7 @@ def setup
       name: "Falko 2",
       description: "Some project description 2.",
       user_id: @another_user.id,
-      check_project: true
+      is_project_from_github: true
     )
 
     @another_release = Release.create(