Adding schema tests (#895)

Also making sure the schema is updated to the normal schema before the suite and after any schema tests
fixes #864

spec/models/mediaflux/schema_fields_create_request_spec.rb

@@ -0,0 +1,146 @@
+# frozen_string_literal: true
+require "rails_helper"
+
+RSpec.describe Mediaflux::SchemaFieldsCreateRequest, type: :model, connect_to_mediaflux: true do
+  let(:user) { FactoryBot.create :user }
+  let(:required_indexed_field) { { name: "code", type: "string", index: true, "min-occurs" => 1, "max-occurs" => 1, label: "desc" } }
+  let(:required_not_indexed_field) { { name: "title", type: "string", index: false, "min-occurs" => 1, "max-occurs" => 1, label: "desc" } }
+  let(:optional_field) do
+    {
+      name: "data_users", type: "string", index: false, "min-occurs" => 0, label: "label", description: "element description",
+      instructions: "do this, not that",
+      attributes: [{ name: "read_only", type: "boolean", index: false, "min-occurs" => 0, description: "desc" }]
+    }
+  end
+  let(:required_many_field) { { name: "departments", type: "string", index: true, "min-occurs" => 1, label: "desc" } }
+  let(:date_field) { { name: "created_on", type: "date", label: "desc", index: false, "min-occurs" => 1, "max-occurs" => 1 } }
+  let(:optional_field2) { { name: "updated_on", type: "date", label: "desc", index: false, "min-occurs" => 0, "max-occurs" => 1 } }
+  let(:nested_field) do
+    requested_by = { name: "RequestedBy", type: "string", index: false, "min-occurs" => 1, "max-occurs" => 1, label: "Requested By",
+                     description: "The person who made the request",
+                     instructions: "The person who made the request, given as a locally unique user." }
+    requested_date = { name: "RequestDateTime", type: "date", index: false, "min-occurs" => 1, "max-occurs" => 1, label: "Request Date-Time", description: "The date and time the request was made",
+                       instructions: "The date and time the request was made, following ISO 8601 standards for timestamps." }
+    note_by = { name: "NoteBy", type: "string", index: false, "min-occurs" => 1, "max-occurs" => 1, label: "Note By", description: "The person making the note." }
+    note_date = { name: "NoteDateTime", type: "date", index: false, "min-occurs" => 1, "max-occurs" => 1, label: "Note Date-Time", description: "The date and time the note was made" }
+    note_type = { name: "EventType", type: "string", index: false, "min-occurs" => 1, "max-occurs" => 1, label: "Event Type", description: "A general category label for the event note" }
+    message = { name: "Message", type: "string", index: false, "min-occurs" => 1, "max-occurs" => 1, label: "Message", description: "The plain-language message contents of the event note." }
+    event_note = { name: "EventlNote", type: "document", index: false, "min-occurs" => 0, label: "Event Note(s)", description: "A supplementary record for a provenance event",
+                   instructions: "A supplementary record of noteworthy details for a given provenance event (e.g., quota decisions, storage tier assignments, revisions to submitted " \
+                                 "metadata, explanations of extenuating circumstances, etc.)",
+                   sub_elements: [note_by, note_date, note_type, message] }
+    { name: "Submission", type: "document", index: false, "min-occurs" => 1, "max-occurs" => 1, label: "Submission", description: "A record of a project’s initial submission",
+      instructions: "A record of a project’s initial submission, including the request to create a new project and the approval or denial by system administrators.",
+      sub_elements: [requested_by, requested_date, event_note] }
+  end
+
+  after do
+    Rake::Task["schema:create"].reenable
+    Rake::Task["schema:create"].invoke
+  end
+
+  describe "required indexed field" do
+    it "submits the proper request" do
+      request = described_class.new(
+        schema_name: "tigerdata",
+        document: "project",
+        description: "test document schema",
+        fields: [required_indexed_field],
+        session_token: user.mediaflux_session
+      )
+      request.resolve
+
+      expect(request.response_xml.to_s).to include "changed=\"true\""
+    end
+  end
+
+  describe "indexed not required field" do
+    it "submits the proper request" do
+      request = described_class.new(
+        schema_name: "tigerdata",
+        document: "project",
+        description: "test document schema",
+        fields: [required_not_indexed_field],
+        session_token: user.mediaflux_session
+      )
+      request.resolve
+
+      expect(request.response_xml.to_s).to include "changed=\"true\""
+    end
+  end
+
+  describe "optional field" do
+    it "submits the proper request" do
+      request = described_class.new(
+        schema_name: "tigerdata",
+        document: "project",
+        description: "test document schema",
+        fields: [optional_field],
+        session_token: user.mediaflux_session
+      )
+      request.resolve
+
+      expect(request.response_xml.to_s).to include "changed=\"true\""
+    end
+  end
+
+  describe "required many values field" do
+    it "submits the proper request" do
+      request = described_class.new(
+        schema_name: "tigerdata",
+        document: "project",
+        description: "test document schema",
+        fields: [required_many_field],
+        session_token: user.mediaflux_session
+      )
+      request.resolve
+
+      expect(request.response_xml.to_s).to include "changed=\"true\""
+    end
+  end
+
+  describe "a date field" do
+    it "submits the proper request" do
+      request = described_class.new(
+        schema_name: "tigerdata",
+        document: "project",
+        description: "test document schema",
+        fields: [date_field],
+        session_token: user.mediaflux_session
+      )
+      request.resolve
+
+      expect(request.response_xml.to_s).to include "changed=\"true\""
+    end
+  end
+
+  describe "updated field" do
+    it "submits the proper request" do
+      request = described_class.new(
+        schema_name: "tigerdata",
+        document: "project",
+        description: "test document schema",
+        fields: [optional_field2],
+        session_token: user.mediaflux_session
+      )
+      request.resolve
+
+      expect(request.response_xml.to_s).to include "changed=\"true\""
+    end
+  end
+
+  describe "nested fields" do
+    it "submits the proper request" do
+      request = described_class.new(
+        schema_name: "tigerdata",
+        document: "project",
+        description: "test document schema",
+        fields: [nested_field],
+        session_token: user.mediaflux_session
+      )
+      request.resolve
+
+      expect(request.response_xml.to_s).to include "changed=\"true\""
+    end
+  end
+end

spec/support/connect_to_mediaflux.rb

@@ -1,4 +1,6 @@
 # frozen_string_literal: true
+require "rake"
+Rails.application.load_tasks
 
 def reset_mediaflux_root
   # Clean out the namespace before running tests to avoid collisions
@@ -42,11 +44,6 @@ def create_test_root_namespace(user)
 RSpec.configure do |config|
   config.before(:each) do |ex|
     if ex.metadata[:connect_to_mediaflux]
-      require "rake"
-      Rails.application.load_tasks
-
-      # Ensure the latest mediaflux schema has been loaded before running the tests
-      Rake::Task["schema:create"].invoke
       reset_mediaflux_root
     end
   rescue StandardError => namespace_error
@@ -55,6 +52,12 @@ def create_test_root_namespace(user)
     Rails.logger.error(message)
   end
 
+  config.before(:suite) do |_ex|
+    # Ensure the latest mediaflux schema has been loaded before running the tests
+    Rake::Task["schema:create"].reenable
+    Rake::Task["schema:create"].invoke
+  end
+
   config.after(:suite) do |_ex|
     reset_mediaflux_root
   end