minor test changes to updating_a_lesson_spec

Author: Scott Adams

app/models/project.rb

@@ -80,8 +80,8 @@ def user_has_a_role_within_the_school
   end
 
   def user_is_a_member_or_the_owner_of_the_lesson
-    return if !lesson || user_id == lesson.user_id || lesson.school_class.members.exists?(student_id: user_id)
-
+    return if !lesson || user_id == lesson.user_id || (lesson.school_class && lesson.school_class.members.exists?(student_id: user_id))
+  
     errors.add(:user, "'#{user_id}' is not the owner or a member of the lesson '#{lesson_id}'")
   end
 end

spec/features/lesson/updating_a_lesson_spec.rb

@@ -9,10 +9,16 @@
   end
 
   let(:headers) { { Authorization: UserProfileMock::TOKEN } }
-  let!(:lesson) { create(:lesson, name: 'Test Lesson', user_id: owner.id) }
+  let(:lesson) { build(:lesson, name: 'Test Lesson', user_id: owner.id) }
   let(:owner) { create(:owner, school:, name: 'School Owner') }
   let(:teacher) { create(:teacher, school:) }
   let(:school) { create(:school) }
+  let(:project) { create(:project) }
+
+  before do
+    lesson
+    project.update!(lesson:, school:, user_id: lesson.user_id, identifier: 'something')
+  end
 
   let(:params) do
     {
@@ -60,7 +66,11 @@
 
   context 'when the lesson is associated with a school (library)' do
     let(:school) { create(:school) }
-    let!(:lesson) { create(:lesson, school:, name: 'Test Lesson', visibility: 'teachers', user_id: teacher.id) }
+    let(:lesson) { build(:lesson, school:, name: 'Test Lesson', visibility: 'teachers', user_id: teacher.id) }
+
+    before do
+      lesson
+    end
 
     it 'responds 200 OK when the user is a school-owner' do
       put("/api/lessons/#{lesson.id}", headers:, params:)