remove question no longer needed and associated test file

app/models/form/sales/questions/other_nationality1.rb

@@ -1,10 +0,0 @@
-class Form::Sales::Questions::OtherNationality1 < ::Form::Question
-  def initialize(id, hsh, page)
-    super
-    @id = "othernational"
-    @check_answer_label = "Buyer 1’s nationality"
-    @header = "Nationality"
-    @type = "text"
-    @check_answers_card_number = 1
-  end
-end

spec/models/form/sales/questions/other_nationality1_spec.rb

@@ -1,41 +0,0 @@
-require "rails_helper"
-
-RSpec.describe Form::Sales::Questions::OtherNationality1, type: :model do
-  subject(:question) { described_class.new(question_id, question_definition, page) }
-
-  let(:question_id) { nil }
-  let(:question_definition) { nil }
-  let(:page) { instance_double(Form::Page) }
-
-  it "has correct page" do
-    expect(question.page).to eq(page)
-  end
-
-  it "has the correct id" do
-    expect(question.id).to eq("othernational")
-  end
-
-  it "has the correct header" do
-    expect(question.header).to eq("Nationality")
-  end
-
-  it "has the correct check_answer_label" do
-    expect(question.check_answer_label).to eq("Buyer 1’s nationality")
-  end
-
-  it "has the correct type" do
-    expect(question.type).to eq("text")
-  end
-
-  it "is not marked as derived" do
-    expect(question.derived?).to be false
-  end
-
-  it "has the correct hint" do
-    expect(question.hint_text).to be_nil
-  end
-
-  it "has the correct check_answers_card_number" do
-    expect(question.check_answers_card_number).to eq(1)
-  end
-end