No need for completed

All information must be filled out (even if not applicable)

spec/factories/users.rb

@@ -19,11 +19,6 @@
       terms_and_conditions_agreed_at { Date.new(2000, 0o1, 0o1) }
     end
 
-    trait :completed do
-      registered
-      ofsted_number { 'EY123456' }
-    end
-
     trait :name do
       first_name { Faker::Name.first_name }
       last_name { Faker::Name.last_name }

spec/support/shared/with_events.rb

@@ -1,7 +1,7 @@
 RSpec.shared_context 'with events' do
-  let(:user) { create(:user, :completed) }
-  let(:user1) { create(:user, :completed) }
-  let(:user2) { create(:user, :completed) }
+  let(:user) { create(:user, :registered) }
+  let(:user1) { create(:user, :registered) }
+  let(:user2) { create(:user, :registered) }
 
   let(:events) do
     Ahoy::Event.where(user_id: user.id).where_properties(training_module_id: module_name)

spec/system/whats_new_page_spec.rb

@@ -4,7 +4,7 @@
   include_context 'with user'
 
   context 'when existing user' do
-    let(:user) { create :user, :completed, :display_whats_new }
+    let(:user) { create :user, :registered, :display_whats_new }
 
     context "and 'whats new' page has not been viewed" do
       it "visits what's new page after sign in" do
@@ -13,7 +13,7 @@
     end
 
     context "and 'whats new' page has been viewed" do
-      let(:user) { create :user, :completed, :display_whats_new }
+      let(:user) { create :user, :registered, :display_whats_new }
 
       it "does not visit what's new page after sign in" do
         click_on 'Sign out'