Add some tests for workers endpoints

test/gateways/rest/api/organizations/create_test.rb

@@ -15,7 +15,7 @@ class REST::API::Organizations::CreateTest < ActionDispatch::IntegrationTest
     name = Faker::Company.name
 
     assert_new_organization { execute(name) }
-    assert_equal saved_organization.name, name
+    assert_equal name, saved_organization.name
   end
 
   test "not creating a new organization with an empty name" do

test/gateways/rest/api/organizations/update_test.rb

@@ -17,7 +17,7 @@ class REST::API::Organizations::UpdateTest < ActionDispatch::IntegrationTest
     new_name = Faker::Company.name
 
     assert_no_new_organizations { execute(organization.external_id, new_name) }
-    assert_equal organization.reload.name, new_name
+    assert_equal new_name, organization.reload.name
   end
 
   test "not updating an organization name to the already existing one" do
@@ -27,7 +27,7 @@ class REST::API::Organizations::UpdateTest < ActionDispatch::IntegrationTest
     create(:organization, name: new_name)
 
     assert_no_new_organizations { execute(organization.external_id, new_name) }
-    assert_equal organization.reload.name, old_name
+    assert_equal old_name, organization.reload.name
   end
 
   private

test/gateways/rest/api/workers/create_test.rb

@@ -0,0 +1,49 @@
+# frozen_string_literal: true
+
+require "test_helper"
+
+class REST::API::Workers::CreateTest < ActionDispatch::IntegrationTest
+  include Tests::Helpers::API
+  include Tests::Helpers::Authentication
+  include Tests::Helpers::Workers
+
+  setup do
+    @organization = create(:organization)
+    @manager = create(:worker, :manager, organization: @organization)
+    @credentials = create_credentials(owner: @manager)
+  end
+
+  test "creating a new worker" do
+    attributes = valid_worker_attributes
+
+    assert_new_worker(@organization) { execute(@organization.external_id, attributes) }
+
+    attributes => { first_name:, last_name:, role:, }
+    assert_equal first_name, saved_worker.first_name
+    assert_equal last_name, saved_worker.last_name
+    assert_equal role, saved_worker.role
+    assert_equal @organization, saved_worker.organization
+  end
+
+  test "not creating a worker with a blank first name" do
+    attributes = valid_worker_attributes.merge(first_name: "")
+
+    assert_no_new_workers(@organization) { execute(@organization.external_id, attributes) }
+  end
+
+  test "not creating a worker with a blank last name" do
+    attributes = valid_worker_attributes.merge(last_name: "")
+
+    assert_no_new_workers(@organization) { execute(@organization.external_id, attributes) }
+  end
+
+  private
+
+  def execute(organization_id, attributes)
+    post(
+      "/api/workers",
+      params: { organization_id:, attributes: },
+      headers: headers_with_auth(@credentials)
+    )
+  end
+end

test/gateways/rest/api/workers/delete_test.rb

@@ -0,0 +1,32 @@
+# frozen_string_literal: true
+
+require "test_helper"
+
+class REST::API::Workers::DeleteTest < ActionDispatch::IntegrationTest
+  include Tests::Helpers::API
+  include Tests::Helpers::Authentication
+  include Tests::Helpers::Workers
+
+  setup do
+    @organization = create(:organization)
+    @manager = create(:worker, :manager, organization: @organization)
+    @credentials = create_credentials(owner: @manager)
+  end
+
+  test "deleting an existing new worker" do
+    worker = create(:worker, organization: @organization)
+
+    execute(worker.external_id)
+
+    assert_nil ::Worker.find_by(id: worker.id)
+  end
+
+  private
+
+  def execute(worker_id)
+    delete(
+      "/api/workers/#{worker_id}",
+      headers: headers_with_auth(@credentials)
+    )
+  end
+end

test/gateways/rest/api/workers/update_test.rb

@@ -0,0 +1,38 @@
+# frozen_string_literal: true
+
+require "test_helper"
+
+class REST::API::Workers::UpdateTest < ActionDispatch::IntegrationTest
+  include Tests::Helpers::API
+  include Tests::Helpers::Authentication
+  include Tests::Helpers::Workers
+
+  setup do
+    @organization = create(:organization)
+    @manager = create(:worker, :manager, organization: @organization)
+    @credentials = create_credentials(owner: @manager)
+  end
+
+  test "updating an existing new worker" do
+    worker = create(:worker, organization: @organization)
+    attributes = valid_worker_attributes
+
+    assert_no_new_workers(@organization) { execute(worker.external_id, attributes) }
+
+    attributes => { first_name:, last_name:, role:, }
+    worker.reload
+    assert_equal first_name, worker.first_name
+    assert_equal last_name, worker.last_name
+    assert_equal role, worker.role
+  end
+
+  private
+
+  def execute(worker_id, attributes)
+    put(
+      "/api/workers/#{worker_id}",
+      params: { attributes: },
+      headers: headers_with_auth(@credentials)
+    )
+  end
+end

test/helpers.rb

@@ -7,4 +7,5 @@ module Helpers; end
 require_relative "helpers/authentication"
 require_relative "helpers/shifts"
 require_relative "helpers/organizations"
+require_relative "helpers/workers"
 require_relative "helpers/api"

test/helpers/workers.rb

@@ -0,0 +1,40 @@
+# frozen_string_literal: true
+
+module Tests
+  module Helpers
+    module Workers
+      def valid_worker_attributes
+        first_name = Faker::Name.first_name
+        last_name = Faker::Name.last_name
+        role = ::Worker::ROLES.to_a.sample
+        time_zone = TZInfo::Timezone.all.sample.name
+
+        { first_name:, last_name:, role:, time_zone: }
+      end
+
+      def assert_no_new_workers(organization, &)
+        assert_no_changes(
+          -> { workers_count(organization) },
+          &
+        )
+      end
+
+      def assert_new_worker(organization, &)
+        assert_changes(
+          -> { workers_count(organization) },
+          from: workers_count(organization),
+          to: workers_count(organization) + 1,
+          &
+        )
+      end
+
+      def saved_worker
+        ::Worker.find_by(external_id: response_body.fetch(:id))
+      end
+
+      def workers_count(organization)
+        ::Worker.where(organization:).count
+      end
+    end
+  end
+end