diff --git a/lib/passageidentity/client.rb b/lib/passageidentity/client.rb index 58ee817..786fec4 100644 --- a/lib/passageidentity/client.rb +++ b/lib/passageidentity/client.rb @@ -1,7 +1,7 @@ # frozen_string_literal: true require_relative 'auth' -require_relative 'user_api' +require_relative 'user' require_relative 'version' module Passage @@ -30,7 +30,7 @@ def initialize(app_id:, api_key:) req_opts[:debug_auth_names] = ['header'] @auth = Passage::Auth.new(app_id: app_id, req_opts: req_opts) - @user = Passage::UserAPI.new(app_id: app_id, req_opts: req_opts) + @user = Passage::User.new(app_id: app_id, req_opts: req_opts) end end end diff --git a/lib/passageidentity/user_api.rb b/lib/passageidentity/user.rb similarity index 95% rename from lib/passageidentity/user_api.rb rename to lib/passageidentity/user.rb index 72d4ef7..8ac8c1a 100644 --- a/lib/passageidentity/user_api.rb +++ b/lib/passageidentity/user.rb @@ -3,8 +3,8 @@ require_relative '../openapi_client' module Passage - # The UserAPI class provides methods for interacting with Passage Users - class UserAPI + # The User class provides methods for interacting with Passage Users + class User def initialize(app_id:, req_opts:) @app_id = app_id @req_opts = req_opts @@ -28,11 +28,11 @@ def get(user_id:) end end - def get_by_identifier(user_identifier:) - raise ArgumentError, 'identifier is required.' unless user_identifier && !user_identifier.empty? + def get_by_identifier(identifier:) + raise ArgumentError, 'identifier is required.' unless identifier && !identifier.empty? begin - req_opts = set_get_by_identifier_query_params(identifier: user_identifier) + req_opts = set_get_by_identifier_query_params(identifier: identifier) response = @user_client.list_paginated_users(@app_id, req_opts) rescue Faraday::Error => e raise PassageError.new( diff --git a/tests/user_api_test.rb b/tests/user_test.rb similarity index 86% rename from tests/user_api_test.rb rename to tests/user_test.rb index b98cd6d..a5ca6ee 100644 --- a/tests/user_api_test.rb +++ b/tests/user_test.rb @@ -7,8 +7,8 @@ Dotenv.load('.env') -# This is a test suite for the Passage User API using the Test::Unit framework. -class TestUserAPI < Test::Unit::TestCase +# This is a test suite for the Passage User using the Test::Unit framework. +class TestUser < Test::Unit::TestCase PassageClient = Passage::Client.new(app_id: ENV['APP_ID'], api_key: ENV['API_KEY']) def setup @@ -45,7 +45,7 @@ def test_get_user_by_identifier user = PassageClient.user.get(user_id: @test_user.id) assert_equal @test_user.id, user.id - user_by_identifier = PassageClient.user.get_by_identifier(user_identifier: @test_user.email) + user_by_identifier = PassageClient.user.get_by_identifier(identifier: @test_user.email) assert_equal @test_user.id, user_by_identifier.id assert_equal user, user_by_identifier @@ -55,7 +55,7 @@ def test_get_user_by_identifier_upper_case user = PassageClient.user.get(user_id: @test_user.id) assert_equal @test_user.id, user.id - user_by_identifier = PassageClient.user.get_by_identifier(user_identifier: @test_user.email.upcase) + user_by_identifier = PassageClient.user.get_by_identifier(identifier: @test_user.email.upcase) assert_equal @test_user.id, user_by_identifier.id assert_equal user, user_by_identifier @@ -70,7 +70,7 @@ def test_get_user_by_identifier_phone user = PassageClient.user.get(user_id: create_user.id) assert_equal create_user.id, user.id - user_by_identifier = PassageClient.user.get_by_identifier(user_identifier: phone) + user_by_identifier = PassageClient.user.get_by_identifier(identifier: phone) assert_equal create_user.id, user_by_identifier.id assert_equal user, user_by_identifier @@ -81,7 +81,7 @@ def test_invalid_get_by_identifier assert_equal @test_user.id, user.id assert_raise Passage::PassageError do - PassageClient.user.get_by_identifier(user_identifier: 'error@passage.id') + PassageClient.user.get_by_identifier(identifier: 'error@passage.id') end end