meibo 0.16.0 → 0.17.0

data/lib/meibo/factory_bot/role.rb

@@ -1,10 +1,10 @@
 # frozen_string_literal: true
 
-require 'factory_bot'
-require 'securerandom'
+require "factory_bot"
+require "securerandom"
 
 FactoryBot.define do
-  factory :meibo_role, class: 'Meibo::Role' do
+  factory :meibo_role, class: "Meibo::Role" do
     initialize_with { new(**attributes) }
 
     transient do

data/lib/meibo/factory_bot/roster.rb

@@ -1,9 +1,9 @@
 # frozen_string_literal: true
 
-require 'factory_bot'
+require "factory_bot"
 
 FactoryBot.define do
-  factory :meibo_roster, class: 'Meibo::Roster' do
+  factory :meibo_roster, class: "Meibo::Roster" do
     initialize_with { new(**attributes) }
   end
 end

data/lib/meibo/factory_bot/user.rb

@@ -1,10 +1,10 @@
 # frozen_string_literal: true
 
-require 'factory_bot'
-require 'securerandom'
+require "factory_bot"
+require "securerandom"
 
 FactoryBot.define do
-  factory :meibo_user, class: 'Meibo::User' do
+  factory :meibo_user, class: "Meibo::User" do
     initialize_with { new(**attributes) }
 
     transient do
@@ -13,9 +13,9 @@ FactoryBot.define do
     end
 
     sourced_id { SecureRandom.uuid }
-    sequence(:username) {|n| "user#{n}@example.com" }
-    sequence(:given_name) {|n| "John#{n}" }
-    sequence(:family_name) {|n| "Doe#{n}" }
+    sequence(:username) { |n| "user#{n}@example.com" }
+    sequence(:given_name) { |n| "John#{n}" }
+    sequence(:family_name) { |n| "Doe#{n}" }
     agent_sourced_ids { agents&.map(&:sourced_id) }
     primary_org_sourced_id { primary_organization&.sourced_id }
 

data/lib/meibo/factory_bot/user_profile.rb

@@ -1,10 +1,10 @@
 # frozen_string_literal: true
 
-require 'factory_bot'
-require 'securerandom'
+require "factory_bot"
+require "securerandom"
 
 FactoryBot.define do
-  factory :meibo_user_profile, class: 'Meibo::UserProfile' do
+  factory :meibo_user_profile, class: "Meibo::UserProfile" do
     initialize_with { new(**attributes) }
 
     transient do
@@ -13,10 +13,10 @@ FactoryBot.define do
 
     sourced_id { SecureRandom.uuid }
     user_sourced_id { user&.sourced_id }
-    profile_type { 'meibo test' }
-    vendor_id { 'meibo' }
-    credential_type { 'plain' }
-    sequence(:username) {|n| "user#{n}"}
+    profile_type { "meibo test" }
+    vendor_id { "meibo" }
+    credential_type { "plain" }
+    sequence(:username) { |n| "user#{n}" }
 
     trait :jp do
       initialize_with { Meibo::JapanProfile::UserProfile.new(**attributes) }

data/lib/meibo/japan_profile/academic_session.rb

@@ -3,7 +3,7 @@
 module Meibo
   class JapanProfile < ::Meibo::Profile
     class AcademicSession < ::Meibo::AcademicSession
-      TYPES = { school_year: 'schoolYear' }.freeze
+      TYPES = { school_year: "schoolYear" }.freeze
 
       DataModel.define(
         self,
@@ -18,7 +18,9 @@ module Meibo
       #   - typeはschoolYear固定
       #   - start_dateは対象年度の開始日固定
       #   - end_dateは対象年度の終了日固定
-      def initialize(school_year:, title: "#{school_year}年度", type: TYPES[:school_year], start_date: Date.new(school_year, 4, 1), end_date: Date.new(school_year + 1, 3, 31), **other_fields)
+      def initialize(school_year:, title: "#{school_year}年度", type: TYPES[:school_year],
+                     start_date: Date.new(school_year, 4,
+                                          1), end_date: Date.new(school_year + 1, 3, 31), **other_fields)
         super(school_year: school_year, title: title, type: type, start_date: start_date, end_date: end_date, **other_fields)
       end
     end

data/lib/meibo/japan_profile/classroom.rb

@@ -6,7 +6,7 @@ module Meibo
       DataModel.define(
         self,
         attribute_name_to_header_field_map: superclass.attribute_name_to_header_field_map.merge(
-          special_needs: 'metadata.jp.specialNeeds'
+          special_needs: "metadata.jp.specialNeeds"
         ).freeze,
         converters: superclass.converters.merge(
           boolean: [:special_needs].freeze,

data/lib/meibo/japan_profile/course.rb

@@ -6,11 +6,11 @@ module Meibo
       DataModel.define(
         self,
         attribute_name_to_header_field_map: superclass.attribute_name_to_header_field_map,
-        converters: superclass.converters.merge(enum: { course_code: [''].freeze }.freeze)
+        converters: superclass.converters.merge(enum: { course_code: [""].freeze }.freeze)
       )
 
       # NOTE: courseCodeは空文字固定
-      def initialize(course_code: '', **other_fields)
+      def initialize(course_code: "", **other_fields)
         super(course_code: course_code, **other_fields)
       end
     end

data/lib/meibo/japan_profile/enrollment.rb

@@ -4,17 +4,17 @@ module Meibo
   class JapanProfile < ::Meibo::Profile
     class Enrollment < ::Meibo::Enrollment
       ROLES = {
-        student: 'student',
-        teacher: 'teacher',
-        administrator: 'administrator',
-        guardian: 'guardian'
+        student: "student",
+        teacher: "teacher",
+        administrator: "administrator",
+        guardian: "guardian"
       }.freeze
 
       DataModel.define(
         self,
-          attribute_name_to_header_field_map: superclass.attribute_name_to_header_field_map.merge(
-          shusseki_no: 'metadata.jp.ShussekiNo',
-          public_flg: 'metadata.jp.PublicFlg'
+        attribute_name_to_header_field_map: superclass.attribute_name_to_header_field_map.merge(
+          shusseki_no: "metadata.jp.ShussekiNo",
+          public_flg: "metadata.jp.PublicFlg"
         ).freeze,
         converters: superclass.converters.merge(
           boolean: [*superclass.converters[:boolean], :public_flg].freeze,
@@ -25,7 +25,8 @@ module Meibo
 
       # NOTE: 児童生徒の場合primaryはfalse固定
       # MEMO: 保護者の場合もそうでは?
-      def initialize(shusseki_no: nil, public_flg: nil, role:, primary: (role == ROLES[:student] ? false : nil), **other_fields)
+      def initialize(role:, shusseki_no: nil, public_flg: nil, primary: (role == ROLES[:student] ? false : nil),
+                     **other_fields)
         super(role: role, primary: primary, **other_fields)
         @shusseki_no = shusseki_no
         @public_flg = public_flg

data/lib/meibo/japan_profile/organization.rb

@@ -4,8 +4,8 @@ module Meibo
   class JapanProfile < ::Meibo::Profile
     class Organization < ::Meibo::Organization
       TYPES = {
-        district: 'district',
-        school: 'school'
+        district: "district",
+        school: "school"
       }.freeze
 
       DataModel.define(

data/lib/meibo/japan_profile/role.rb

@@ -9,9 +9,9 @@ module Meibo
       #   - 保護者の場合guardian
       # MEMO: enrollments.csvの方ではadministratorとguardianも許可されているがズレてないか
       ROLES = {
-        teacher: 'teacher',
-        student: 'student',
-        guardian: 'guardian'
+        teacher: "teacher",
+        student: "student",
+        guardian: "guardian"
       }.freeze
 
       DataModel.define(

data/lib/meibo/japan_profile/user.rb

@@ -6,16 +6,17 @@ module Meibo
       DataModel.define(
         self,
         attribute_name_to_header_field_map: superclass.attribute_name_to_header_field_map.merge(
-          kana_given_name: 'metadata.jp.kanaGivenName',
-          kana_family_name: 'metadata.jp.kanaFamilyName',
-          kana_middle_name: 'metadata.jp.kanaMiddleName',
-          home_class: 'metadata.jp.homeClass'
+          kana_given_name: "metadata.jp.kanaGivenName",
+          kana_family_name: "metadata.jp.kanaFamilyName",
+          kana_middle_name: "metadata.jp.kanaMiddleName",
+          home_class: "metadata.jp.homeClass"
         ).freeze,
         converters: superclass.converters
       )
 
       # NOTE: enabled_userは必須ではないが固定
-      def initialize(enabled_user: true, kana_given_name: nil, kana_family_name: nil, kana_middle_name: nil, home_class: nil, **other_fields)
+      def initialize(enabled_user: true, kana_given_name: nil, kana_family_name: nil, kana_middle_name: nil,
+                     home_class: nil, **other_fields)
         super(enabled_user: enabled_user, **other_fields)
         @kana_given_name = kana_given_name
         @kana_family_name = kana_family_name

data/lib/meibo/japan_profile/user_set.rb

@@ -6,9 +6,7 @@ module Meibo
       def check_semantically_consistent
         super
         each do |user|
-          if user.home_class
-            roster.classes.find(user.home_class)
-          end
+          roster.classes.find(user.home_class) if user.home_class
         end
       end
     end

data/lib/meibo/japan_profile.rb

@@ -12,7 +12,7 @@ module Meibo
       role: Builder::RoleBuilder.create(Role),
       user: Builder::UserBuilder.create(User),
       user_profile: Builder::UserProfileBuilder.create(UserProfile)
-    }
+    }.freeze
 
     DATA_MODELS = {
       file_academic_sessions: AcademicSession,
@@ -24,7 +24,7 @@ module Meibo
       file_roles: Role,
       file_user_profiles: UserProfile,
       file_users: User
-    }
+    }.freeze
 
     DATA_SET = ::Meibo::Profile::DATA_SET.merge(
       users: UserSet

data/lib/meibo/manifest/processing_mode.rb

@@ -11,21 +11,22 @@ module Meibo
       end
 
       def absent?
-        @mode == 'absent'
+        @mode == "absent"
       end
 
       def bulk?
-        @mode == 'bulk'
+        @mode == "bulk"
       end
 
       def delta?
-        @mode == 'delta'
+        @mode == "delta"
       end
 
       def ==(other)
-        if other.is_a?(ProcessingMode)
+        case other
+        when ProcessingMode
           @mode == other.mode
-        elsif other.is_a?(String)
+        when String
           to_s == other
         else
           false

data/lib/meibo/manifest.rb

@@ -2,43 +2,45 @@
 
 module Meibo
   class Manifest
-    MANIFEST_VERSION = '1.0'
-    ONEROSTER_VERSION = '1.2'
+    MANIFEST_VERSION = "1.0"
+    ONEROSTER_VERSION = "1.2"
 
     PROCESSING_MODES = {
-      absent: ProcessingMode.new('absent'),
-      bulk: ProcessingMode.new('bulk'),
-      delta: ProcessingMode.new('delta')
+      absent: ProcessingMode.new("absent"),
+      bulk: ProcessingMode.new("bulk"),
+      delta: ProcessingMode.new("delta")
     }.freeze
 
     PROPERTY_NAME_TO_ATTRIBUTE_MAP = {
-      'manifest.version' => :manifest_version,
-      'oneroster.version' => :oneroster_version,
-      'file.academicSessions' => :file_academic_sessions,
-      'file.categories' => :file_categories,
-      'file.classes' => :file_classes,
-      'file.classResources' => :file_class_resources,
-      'file.courses' => :file_courses,
-      'file.courseResources' => :file_course_resources,
-      'file.demographics' => :file_demographics,
-      'file.enrollments' => :file_enrollments,
-      'file.lineItemLearningObjectiveIds' => :file_line_item_learning_objective_ids,
-      'file.lineItems' => :file_line_items,
-      'file.lineItemScoreScales' => :file_line_item_score_scales,
-      'file.orgs' => :file_orgs,
-      'file.resources' => :file_resources,
-      'file.resultLearningObjectiveIds' => :file_result_learning_objective_ids,
-      'file.results' => :file_results,
-      'file.resultScoreScales' => :file_result_score_scales,
-      'file.roles' => :file_roles,
-      'file.scoreScales' => :file_score_scales,
-      'file.userProfiles' => :file_user_profiles,
-      'file.userResources' => :file_user_resources,
-      'file.users' => :file_users,
-      'source.systemName' => :source_system_name,
-      'source.systemCode' => :source_system_code
+      "manifest.version" => :manifest_version,
+      "oneroster.version" => :oneroster_version,
+      "file.academicSessions" => :file_academic_sessions,
+      "file.categories" => :file_categories,
+      "file.classes" => :file_classes,
+      "file.classResources" => :file_class_resources,
+      "file.courses" => :file_courses,
+      "file.courseResources" => :file_course_resources,
+      "file.demographics" => :file_demographics,
+      "file.enrollments" => :file_enrollments,
+      "file.lineItemLearningObjectiveIds" => :file_line_item_learning_objective_ids,
+      "file.lineItems" => :file_line_items,
+      "file.lineItemScoreScales" => :file_line_item_score_scales,
+      "file.orgs" => :file_orgs,
+      "file.resources" => :file_resources,
+      "file.resultLearningObjectiveIds" => :file_result_learning_objective_ids,
+      "file.results" => :file_results,
+      "file.resultScoreScales" => :file_result_score_scales,
+      "file.roles" => :file_roles,
+      "file.scoreScales" => :file_score_scales,
+      "file.userProfiles" => :file_user_profiles,
+      "file.userResources" => :file_user_resources,
+      "file.users" => :file_users,
+      "source.systemName" => :source_system_name,
+      "source.systemCode" => :source_system_code
     }.freeze
-    ATTRIBUTE_TO_PROPERTY_NAME_MAP = PROPERTY_NAME_TO_ATTRIBUTE_MAP.to_h {|property_name, attribute| [attribute, property_name] }.freeze
+    ATTRIBUTE_TO_PROPERTY_NAME_MAP = PROPERTY_NAME_TO_ATTRIBUTE_MAP.to_h do |property_name, attribute|
+      [attribute, property_name]
+    end.freeze
 
     # NOTE: 想定値
     DEFAULT_VALUES = {
@@ -67,19 +69,20 @@ module Meibo
       file_users: PROCESSING_MODES[:bulk]
     }.freeze
 
-    attr_reader :manifest_version, :oneroster_version, :file_academic_sessions, :file_categories, :file_classes, :file_class_resources, :file_courses, :file_course_resources, :file_demographics, :file_enrollments, :file_line_item_learning_objective_ids, :file_line_items, :file_line_item_score_scales, :file_orgs, :file_resources, :file_result_learning_objective_ids, :file_results, :file_result_score_scales, :file_roles, :file_score_scales, :file_user_profiles, :file_user_resources, :file_users, :source_system_name, :source_system_code
+    attr_reader :manifest_version, :oneroster_version, :file_academic_sessions, :file_categories, :file_classes,
+                :file_class_resources, :file_courses, :file_course_resources, :file_demographics, :file_enrollments, :file_line_item_learning_objective_ids, :file_line_items, :file_line_item_score_scales, :file_orgs, :file_resources, :file_result_learning_objective_ids, :file_results, :file_result_score_scales, :file_roles, :file_score_scales, :file_user_profiles, :file_user_resources, :file_users, :source_system_name, :source_system_code
 
     def self.build_from_default(**custom_properties)
       new(**Manifest::DEFAULT_VALUES.merge(custom_properties))
     end
 
     def self.filename
-      'manifest.csv'
+      "manifest.csv"
     end
 
     attribute_name_to_header_field_map = {
-      property_name: 'propertyName',
-      value: 'value'
+      property_name: "propertyName",
+      value: "value"
     }
     header_fields = attribute_name_to_header_field_map.values
     define_singleton_method(:header_fields) { header_fields }
@@ -92,17 +95,20 @@ module Meibo
     def self.filename_for(attribute)
       property_name = attribute_to_property_name(attribute)
 
-      raise Meibo::Error, "#{property_name}はファイルのプロパティではありません" unless property_name.start_with?('file.')
+      raise Meibo::Error, "#{property_name}はファイルのプロパティではありません" unless property_name.start_with?("file.")
 
-      property_name.split('file.', 2).last + '.csv'
+      "#{property_name.split("file.", 2).last}.csv"
     end
 
     def self.parse(csv)
       properties = CSV.parse(csv, encoding: Meibo::CSV_ENCODING, headers: true, header_converters: header_converters)
-      new(**properties.to_h {|property| [PROPERTY_NAME_TO_ATTRIBUTE_MAP.fetch(property[:property_name]), property[:value]] })
+      new(**properties.to_h do |property|
+              [PROPERTY_NAME_TO_ATTRIBUTE_MAP.fetch(property[:property_name]), property[:value]]
+            end)
     end
 
-    def initialize(manifest_version:, oneroster_version:, file_academic_sessions:, file_categories:, file_classes:, file_class_resources:, file_courses:, file_course_resources:, file_demographics:, file_enrollments:, file_line_item_learning_objective_ids:, file_line_items:, file_line_item_score_scales:, file_orgs:, file_resources:, file_result_learning_objective_ids:, file_results:, file_result_score_scales:, file_roles:, file_score_scales:, file_user_profiles:, file_user_resources:, file_users:, source_system_name: nil, source_system_code: nil)
+    def initialize(manifest_version:, oneroster_version:, file_academic_sessions:, file_categories:, file_classes:,
+                   file_class_resources:, file_courses:, file_course_resources:, file_demographics:, file_enrollments:, file_line_item_learning_objective_ids:, file_line_items:, file_line_item_score_scales:, file_orgs:, file_resources:, file_result_learning_objective_ids:, file_results:, file_result_score_scales:, file_roles:, file_score_scales:, file_user_profiles:, file_user_resources:, file_users:, source_system_name: nil, source_system_code: nil)
       @manifest_version = manifest_version
       @oneroster_version = oneroster_version
       @file_academic_sessions = file_academic_sessions
@@ -138,7 +144,7 @@ module Meibo
 
     def file_attributes(processing_mode:)
       PROPERTY_NAME_TO_ATTRIBUTE_MAP.values.filter_map do |attribute|
-        attribute.start_with?('file_') && public_send(attribute) == processing_mode.to_s && attribute
+        attribute.start_with?("file_") && public_send(attribute) == processing_mode.to_s && attribute
       end
     end
 
@@ -157,7 +163,7 @@ module Meibo
     end
 
     def to_h
-      PROPERTY_NAME_TO_ATTRIBUTE_MAP.values.to_h {|attribute| [attribute, public_send(attribute)] }
+      PROPERTY_NAME_TO_ATTRIBUTE_MAP.values.to_h { |attribute| [attribute, public_send(attribute)] }
     end
   end
 end

data/lib/meibo/organization.rb

@@ -3,34 +3,35 @@
 module Meibo
   class Organization
     TYPES = {
-      department: 'department',
-      school: 'school',
-      district: 'district',
-      local: 'local',
-      state: 'state',
-      national: 'national'
+      department: "department",
+      school: "school",
+      district: "district",
+      local: "local",
+      state: "state",
+      national: "national"
     }.freeze
 
     DataModel.define(
       self,
       attribute_name_to_header_field_map: {
-        sourced_id: 'sourcedId',
-        status: 'status',
-        date_last_modified: 'dateLastModified',
-        name: 'name',
-        type: 'type',
-        identifier: 'identifier',
-        parent_sourced_id: 'parentSourcedId'
+        sourced_id: "sourcedId",
+        status: "status",
+        date_last_modified: "dateLastModified",
+        name: "name",
+        type: "type",
+        identifier: "identifier",
+        parent_sourced_id: "parentSourcedId"
       },
       converters: {
         datetime: [:date_last_modified],
         enum: { type: [*TYPES.values, ENUM_EXT_PATTERN].freeze },
-        required: [:sourced_id, :name, :type],
+        required: %i[sourced_id name type],
         status: [:status]
       }
     )
 
-    def initialize(sourced_id:, status: nil, date_last_modified: nil, name:, type:, identifier: nil, parent_sourced_id: (type == TYPES[:district] ? 'NULL' : nil), **extension_fields)
+    def initialize(sourced_id:, name:, type:, status: nil, date_last_modified: nil, identifier: nil,
+                   parent_sourced_id: (type == TYPES[:district] ? "NULL" : nil), **extension_fields)
       @sourced_id = sourced_id
       @status = status
       @date_last_modified = date_last_modified

data/lib/meibo/profile.rb

@@ -12,7 +12,7 @@ module Meibo
       role: Builder::RoleBuilder.create(Role),
       user: Builder::UserBuilder.create(User),
       user_profile: Builder::UserProfileBuilder.create(UserProfile)
-    }
+    }.freeze
 
     DATA_MODELS = {
       file_academic_sessions: Meibo::AcademicSession,
@@ -24,7 +24,7 @@ module Meibo
       file_roles: Meibo::Role,
       file_user_profiles: Meibo::UserProfile,
       file_users: Meibo::User
-    }
+    }.freeze
 
     DATA_SET = {
       academic_sessions: Meibo::AcademicSessionSet,
@@ -36,7 +36,7 @@ module Meibo
       roles: Meibo::RoleSet,
       user_profiles: Meibo::UserProfileSet,
       users: Meibo::UserSet
-    }
+    }.freeze
 
     attr_reader :builders, :data_models, :data_set
 

data/lib/meibo/reader.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
-require 'csv'
-require 'zip'
+require "csv"
+require "zip"
 
 module Meibo
   class Reader
@@ -11,6 +11,12 @@ module Meibo
       end
     end
 
+    def self.open_buffer(io, profile: Meibo.default_profile)
+      Zip::File.open_buffer(io) do |zipfile|
+        yield new(zipfile: zipfile, profile: profile)
+      end
+    end
+
     attr_reader :profile, :zipfile
 
     def initialize(zipfile:, profile:)
@@ -112,11 +118,9 @@ module Meibo
         file_roles: :roles,
         file_users: :users,
         file_user_profiles: :user_profiles
-      }.filter_map {|file_attribute, data_method|
-        if bulk_file_attributes.include?(file_attribute)
-          [data_method, public_send(data_method).to_a]
-        end
-      }.to_h
+      }.filter_map do |file_attribute, data_method|
+        [data_method, public_send(data_method).to_a] if bulk_file_attributes.include?(file_attribute)
+      end.to_h
     end
 
     def file_entry?(filename)

data/lib/meibo/role.rb

@@ -3,43 +3,43 @@
 module Meibo
   class Role
     TYPES = {
-      primary: 'primary',
-      secondary: 'secondary'
+      primary: "primary",
+      secondary: "secondary"
     }.freeze
 
     ROLES = {
-      aide: 'aide',
-      counselor: 'counselor',
-      district_administrator: 'districtAdministrator',
-      guardian: 'guardian',
-      parent: 'parent',
-      principal: 'principal',
-      proctor: 'proctor',
-      relative: 'relative',
-      site_administrator: 'siteAdministrator',
-      student: 'student',
-      system_administrator: 'systemAdministrator',
-      teacher: 'teacher'
+      aide: "aide",
+      counselor: "counselor",
+      district_administrator: "districtAdministrator",
+      guardian: "guardian",
+      parent: "parent",
+      principal: "principal",
+      proctor: "proctor",
+      relative: "relative",
+      site_administrator: "siteAdministrator",
+      student: "student",
+      system_administrator: "systemAdministrator",
+      teacher: "teacher"
     }.freeze
 
     DataModel.define(
       self,
       attribute_name_to_header_field_map: {
-        sourced_id: 'sourcedId',
-        status: 'status',
-        date_last_modified: 'dateLastModified',
-        user_sourced_id: 'userSourcedId',
-        role_type: 'roleType',
-        role: 'role',
-        begin_date: 'beginDate',
-        end_date: 'endDate',
-        org_sourced_id: 'orgSourcedId',
-        user_profile_sourced_id: 'userProfileSourcedId'
+        sourced_id: "sourcedId",
+        status: "status",
+        date_last_modified: "dateLastModified",
+        user_sourced_id: "userSourcedId",
+        role_type: "roleType",
+        role: "role",
+        begin_date: "beginDate",
+        end_date: "endDate",
+        org_sourced_id: "orgSourcedId",
+        user_profile_sourced_id: "userProfileSourcedId"
       }.freeze,
       converters: {
-        date: [:begin_date, :end_date].freeze,
+        date: %i[begin_date end_date].freeze,
         datetime: [:date_last_modified].freeze,
-        required: [:sourced_id, :user_sourced_id, :role_type, :role, :org_sourced_id].freeze,
+        required: %i[sourced_id user_sourced_id role_type role org_sourced_id].freeze,
         enum: {
           role: [*ROLES.values, ENUM_EXT_PATTERN].freeze,
           role_type: TYPES.values.freeze
@@ -48,7 +48,8 @@ module Meibo
       }.freeze
     )
 
-    def initialize(sourced_id:, status: nil, date_last_modified: nil, user_sourced_id:, role_type:, role:, begin_date: nil, end_date: nil, org_sourced_id:, user_profile_sourced_id: nil, **extension_fields)
+    def initialize(sourced_id:, user_sourced_id:, role_type:, role:, org_sourced_id:, status: nil, date_last_modified: nil,
+                   begin_date: nil, end_date: nil, user_profile_sourced_id: nil, **extension_fields)
       @sourced_id = sourced_id
       @status = status
       @date_last_modified = date_last_modified

data/lib/meibo/role_set.rb

@@ -9,9 +9,7 @@ module Meibo
         roster.organizations.find(role.org_sourced_id)
         roster.users.find(role.user_sourced_id)
 
-        if role.user_profile_sourced_id
-          roster.user_profiles.find(role.user_profile_sourced_id)
-        end
+        roster.user_profiles.find(role.user_profile_sourced_id) if role.user_profile_sourced_id
       end
     end