Merging pull request #86 from fga-gpp-mds/US22_Manter_Issue

Adding Issue support

Author: MatheusRich

app/controllers/issues_controller.rb

@@ -0,0 +1,70 @@
+class IssuesController < ApplicationController
+  before_action :set_authorization, :set_path
+
+  def index
+    @issues = @client.list_issues(@path)
+
+    convert_form_params(@issues)
+
+    render json: @form_params
+  end
+
+  def create
+    @issue = @client.create_issue(@path, issue_params[:name], issue_params[:body], options = { assignee: issue_params[:assignee], labels: issue_params[:labels] })
+
+    convert_form_params(@issue)
+
+    render json: @form_params, status: :created
+  end
+
+  def update
+    @issue = @client.update_issue(@path, issue_params[:number], issue_params[:name], issue_params[:body], options = { assignee: issue_params[:assignee], labels: [issue_params[:labels]] })
+
+    convert_form_params(@issue)
+
+    render json: @form_params
+  end
+
+  def close
+    @issue = @client.close_issue(@path, issue_params[:number])
+
+    render status: 200
+  end
+
+  private
+
+    def set_authorization
+      @current_user = AuthorizeApiRequest.call(request.headers).result
+      @client = Octokit::Client.new(access_token: @current_user.access_token)
+    end
+
+    def set_path
+      @project = Project.find(params[:id])
+
+      if @project.name.include? "/"
+        @path = @project.name
+      else
+        @name = @client.user.login
+        @repo = @project.name
+        @path = @name + "/" + @repo
+      end
+
+      @path
+    end
+
+    def convert_form_params(issue)
+      @form_params = { issues_infos: [] }
+      if issue.kind_of?(Array)
+        @issues.each do |issue|
+          @form_params[:issues_infos].push(name: issue.title, number: issue.number, body: issue.body)
+        end
+      else
+        @form_params[:issues_infos].push(name: issue.title, number: issue.number, body: issue.body)
+      end
+      @form_params
+    end
+
+    def issue_params
+      params.require(:issue).permit(:name, :body, :assignee, :labels, :number)
+    end
+end

app/models/issue.rb

@@ -0,0 +1,2 @@
+class Issue < ApplicationRecord
+end

config/routes.rb

@@ -5,6 +5,12 @@
 
   get "repos", to: "projects#github_projects_list"
 
+  get "projects/:id/issues", to: "issues#index"
+  post "projects/:id/issues", to: "issues#create"
+  put "projects/:id/issues", to: "issues#update"
+  patch "projects/:id/issues", to: "issues#update"
+  delete "projects/:id/issues", to: "issues#close"
+
   resources :users, shallow: true do
     resources :projects do
       resources :releases do

db/migrate/20171028121912_create_issues.rb

@@ -0,0 +1,14 @@
+class CreateIssues < ActiveRecord::Migration[5.1]
+  def change
+    create_table :issues do |t|
+      t.string :title
+      t.text :body
+      t.string :assignee
+      t.integer :milestone
+      t.string :labels
+      t.string :assignees
+
+      t.timestamps
+    end
+  end
+end

test/controllers/issues_controller_test.rb

@@ -0,0 +1,206 @@
+require "test_helper"
+
+class IssuesControllerTest < ActionDispatch::IntegrationTest
+  def setup
+    @user = User.create(name: "Ronaldo", email: "Ronaldofenomeno@gmail.com", password: "123456789", password_confirmation: "123456789", github: "ronaldobola")
+    @token = AuthenticateUser.call(@user.email, @user.password)
+    @project = Project.create(name: "Falko", description: "Descrição do projeto.", user_id: @user.id, is_project_from_github: true)
+    @project2 = Project.create(name: "Falko", description: "Descrição do projeto.", user_id: @user.id, is_project_from_github: false)
+  end
+
+  test "should see issues if user is loged in" do
+
+    mock = Minitest::Mock.new
+
+    def mock.user()
+      Sawyer::Resource.new(Sawyer::Agent.new("/issues_test"), login: "username_test")
+    end
+
+    def mock.list_issues(name)
+      [ Sawyer::Resource.new(Sawyer::Agent.new("/issues_test"), title: "issue", number: "3", body: "This is a template body") ]
+    end
+
+
+    Octokit::Client.stub :new, mock do
+      get "/projects/#{@project.id}/issues", headers: { Authorization: @token.result }
+
+      assert response.parsed_body["issues_infos"][0]["name"] == "issue"
+      assert response.parsed_body["issues_infos"][0]["number"] == "3"
+      assert response.parsed_body["issues_infos"][0]["body"] == "This is a template body"
+      assert_response :success
+    end
+  end
+
+  test "should not see issues if user is not loged in" do
+
+    mock = Minitest::Mock.new
+
+    def mock.user()
+      Sawyer::Resource.new(Sawyer::Agent.new("/issues_test"), login: "username_test")
+    end
+
+    def mock.list_issues(name)
+      [ Sawyer::Resource.new(Sawyer::Agent.new("/issues_test"), title: "issue", number: "3", body: "This is a template body") ]
+    end
+
+
+    Octokit::Client.stub :new, mock do
+      get "/projects/#{@project.id}/issues"
+
+      assert_response :unauthorized
+    end
+  end
+
+  test "should create issues if user is loged in" do
+
+    mock = Minitest::Mock.new
+
+    def mock.user()
+      Sawyer::Resource.new(Sawyer::Agent.new("/issues_test"), login: "username_test")
+    end
+
+    def mock.create_issue(path, name, body, options)
+      Sawyer::Resource.new(Sawyer::Agent.new("/issues_test"), title: name, body: body, number: "3")
+    end
+
+
+    Octokit::Client.stub :new, mock do
+      post "/projects/#{@project.id}/issues", headers: { Authorization: @token.result }, params: {
+        issue: {
+          "name": "New Issue",
+          "body": "New Body"
+        }
+      }
+
+      assert response.parsed_body["issues_infos"][0]["name"] == "New Issue"
+      assert response.parsed_body["issues_infos"][0]["body"] == "New Body"
+      assert_response :created
+    end
+  end
+
+  test "should not create issues if user is not loged in" do
+
+    mock = Minitest::Mock.new
+
+    def mock.user()
+      Sawyer::Resource.new(Sawyer::Agent.new("/issues_test"), login: "username_test")
+    end
+
+    def mock.create_issue(path, name, body, options)
+      Sawyer::Resource.new(Sawyer::Agent.new("/issues_test"), title: name, body: body, number: "3")
+    end
+
+
+    Octokit::Client.stub :new, mock do
+      post "/projects/#{@project.id}/issues", params: {
+        issue: {
+          "name": "New Issue",
+          "body": "New Body"
+        }
+      }
+
+      assert_response :unauthorized
+    end
+  end
+
+  test "should update issues if user is loged in" do
+
+    mock = Minitest::Mock.new
+
+    def mock.user()
+      Sawyer::Resource.new(Sawyer::Agent.new("/issues_test"), login: "username_test")
+    end
+
+    def mock.update_issue(path, number, name, body, options)
+      Sawyer::Resource.new(Sawyer::Agent.new("/issues_test"), title: name, body: body, number: "3")
+    end
+
+
+    Octokit::Client.stub :new, mock do
+      put "/projects/#{@project.id}/issues", headers: { Authorization: @token.result }, params: {
+        issue: {
+          "number": "3",
+          "name": "Updated Issue",
+          "body": "Updated Body"
+        }
+      }
+      assert response.parsed_body["issues_infos"][0]["name"] == "Updated Issue"
+      assert response.parsed_body["issues_infos"][0]["body"] == "Updated Body"
+      assert_response :success
+    end
+  end
+
+  test "should not update issues if user is not loged in" do
+
+    mock = Minitest::Mock.new
+
+    def mock.user()
+      Sawyer::Resource.new(Sawyer::Agent.new("/issues_test"), login: "username_test")
+    end
+
+    def mock.update_issue(path, number, name, body, options)
+      Sawyer::Resource.new(Sawyer::Agent.new("/issues_test"), title: name, body: body, number: "3")
+    end
+
+
+    Octokit::Client.stub :new, mock do
+      put "/projects/#{@project.id}/issues", params: {
+        issue: {
+          "number": "3",
+          "name": "Updated Issue",
+          "body": "Updated Body"
+        }
+      }
+
+      assert_response :unauthorized
+    end
+  end
+
+  test "should close issue if user is loged in" do
+
+    mock = Minitest::Mock.new
+
+    def mock.user()
+      Sawyer::Resource.new(Sawyer::Agent.new("/issues_test"), login: "username_test")
+    end
+
+    def mock.close_issue(path, number)
+      Sawyer::Resource.new(Sawyer::Agent.new("/issues_test"), number: "3")
+    end
+
+
+    Octokit::Client.stub :new, mock do
+      delete "/projects/#{@project.id}/issues", headers: { Authorization: @token.result }, params: {
+        issue: {
+          "number": "3"
+        }
+      }
+
+      assert_response :success
+    end
+  end
+
+  test "should not close issue if user is not loged in" do
+
+    mock = Minitest::Mock.new
+
+    def mock.user()
+      Sawyer::Resource.new(Sawyer::Agent.new("/issues_test"), login: "username_test")
+    end
+
+    def mock.close_issue(path, number)
+      Sawyer::Resource.new(Sawyer::Agent.new("/issues_test"), number: "3")
+    end
+
+
+    Octokit::Client.stub :new, mock do
+      delete "/projects/#{@project.id}/issues", params: {
+        issue: {
+          "number": "3"
+        }
+      }
+
+      assert_response :unauthorized
+    end
+  end
+end

test/fixtures/issues.yml

@@ -0,0 +1,17 @@
+# Read about fixtures at http://api.rubyonrails.org/classes/ActiveRecord/FixtureSet.html
+
+one:
+  title: MyString
+  body: MyText
+  assignee: MyString
+  milestone: 1
+  labels: 
+  assignees: 
+
+two:
+  title: MyString
+  body: MyText
+  assignee: MyString
+  milestone: 1
+  labels: 
+  assignees: 

test/models/issue_test.rb

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