Merge pull request #1059 from fastlane/create-account

Add new API endpoint to register user

app/features-json/repos_json_controller.rb

@@ -24,7 +24,7 @@ class RepositoryJSONController < APIController
       github_client = Octokit::Client.new(access_token: params[:token])
 
       begin
-        #  Note: This fails if the user.email scope is missing from token
+        # Note: This fails if the user.email scope is missing from token
         email = github_client.emails.find(&:primary).email
       rescue Octokit::NotFound
         json_error!(

app/features-json/user_json_controller.rb

@@ -0,0 +1,53 @@
+require_relative "../services/user_service"
+require_relative "api_controller"
+require_relative "json_params"
+
+module FastlaneCI
+  # Controller responsible for handling users
+  class UserJSONController < APIController
+    disable(:authentication)
+
+    HOME = "/api/user"
+
+    post HOME.to_s do
+      # fetch email based on the API token instead
+      github_client = Octokit::Client.new(access_token: params[:github_token])
+
+      begin
+        # Note: This fails if the user.email scope is missing from token
+        email = github_client.emails.find(&:primary).email
+      rescue Octokit::NotFound
+        json_error!(
+          error_message: "Provided API token needs user email scope",
+          error_key: "User.Token.MissingEmailScope",
+          error_code: 400
+        )
+      rescue Octokit::Unauthorized
+        json_error!(
+          error_message: "Provided API token is invalid",
+          error_key: "User.Token.Invalid",
+          error_code: 403
+        )
+      end
+
+      user = Services.user_service.create_user!(
+        email: email,
+        password: params[:password]
+      )
+
+      if user
+        Services.user_service.create_provider_credential!(
+          user_id: user.id,
+          email: user.email,
+          api_token: params[:github_token]
+        )
+        return json({ status: :success })
+      else
+        json_error!(
+          error_message: "Error creating new user",
+          error_key: "User.Error"
+        )
+      end
+    end
+  end
+end

app/shared/models/github_provider_credential.rb

@@ -18,6 +18,7 @@ class GitHubProviderCredential < ProviderCredential
     # @return [String]
     attr_reader :provider_name
 
+    # TODO: Document what this value is used for
     # @return [String]
     attr_reader :full_name
 

launch.rb

@@ -149,6 +149,7 @@ def self.register_available_controllers
       require_relative "app/features-json/project_json_controller"
       require_relative "app/features-json/repos_json_controller"
       require_relative "app/features-json/login_json_controller"
+      require_relative "app/features-json/user_json_controller"
       require_relative "app/features-json/build_json_controller"
       require_relative "app/features-json/artifact_json_controller"
       require_relative "app/features-json/setup_json_controller"
@@ -161,6 +162,7 @@ def self.register_available_controllers
       FastlaneCI::FastlaneApp.use(FastlaneCI::ArtifactJSONController)
       FastlaneCI::FastlaneApp.use(FastlaneCI::SetupJSONController)
       FastlaneCI::FastlaneApp.use(FastlaneCI::SettingJSONController)
+      FastlaneCI::FastlaneApp.use(FastlaneCI::UserJSONController)
     end
 
     def self.start_github_workers

spec/features-json/user_json_spec.rb

@@ -0,0 +1,45 @@
+require "spec_helper"
+require "app/features-json/user_json_controller"
+require "app/services/user_service"
+
+describe FastlaneCI::UserJSONController do
+  def app
+    described_class
+  end
+  let(:json) { JSON.parse(last_response.body) }
+
+  before do
+    allow(FastlaneCI.dot_keys).to receive(:encryption_key).and_return("test")
+  end
+
+  describe "/api/user" do
+    let(:fake_email) { "email@email.com" }
+    before do
+      github_client = "github_client"
+      expect(Octokit::Client).to receive(:new).and_return(github_client)
+
+      email_entry = "email_entry"
+      expect(email_entry).to receive(:primary).and_return(true)
+      expect(email_entry).to receive(:email).and_return(fake_email)
+
+      expect(github_client).to receive(:emails).and_return([email_entry])
+    end
+
+    it "creates a new user and attach the provider credentials" do
+      allow(FastlaneCI::Services.user_service.user_data_source).to receive(:user_exist?).with({ email: fake_email }).and_return(false)
+
+      post "/api/user", { github_token: "github_token", password: "password" }.to_json, { "CONTENT_TYPE" => "application/json" }
+      expect(last_response).to be_ok
+      expect(json["status"]).to eq("success")
+    end
+
+    it "returns an error if the user already exists" do
+      allow(FastlaneCI::Services.user_service.user_data_source).to receive(:user_exist?).with({ email: fake_email }).and_return(true)
+
+      post "/api/user", { github_token: "github_token", password: "password" }.to_json, { "CONTENT_TYPE" => "application/json" }
+      expect(last_response.status).to eq(400)
+      expect(json["key"]).to eq("User.Error")
+      expect(json["message"]).to eq("Error creating new user")
+    end
+  end
+end