Process completed with exit code 1.

Plates Heron API #create when providing events when missing required subjects in the events part displays the error Failure/Error: event.errors.each { |key, value| errors.add(key, value) } ActiveSupport::DeprecationException: DEPRECATION WARNING: Enumerating ActiveModel::Errors as a hash has been deprecated. In Rails 6.1, `errors` is an array of Error objects, therefore it should be accessed by a block with a single block parameter like this: person.errors.each do |error| attribute = error.attribute message = error.message end You are passing a block expecting two parameters, so the old hash behavior is simulated. As this is deprecated, this will result in an ArgumentError in Rails 7.0. (called from add_all_errors_from_event at /home/runner/work/sequencescape/sequencescape/app/heron/factories/concerns/eventful.rb:26)

Plates Heron API #create when providing events when missing required subjects in the events part behaves like a failed plate creation does not create a plate Failure/Error: event.errors.each { |key, value| errors.add(key, value) } ActiveSupport::DeprecationException: DEPRECATION WARNING: Enumerating ActiveModel::Errors as a hash has been deprecated. In Rails 6.1, `errors` is an array of Error objects, therefore it should be accessed by a block with a single block parameter like this: person.errors.each do |error| attribute = error.attribute message = error.message end You are passing a block expecting two parameters, so the old hash behavior is simulated. As this is deprecated, this will result in an ArgumentError in Rails 7.0. (called from add_all_errors_from_event at /home/runner/work/sequencescape/sequencescape/app/heron/factories/concerns/eventful.rb:26) Shared Example Group: "a failed plate creation" called from ./spec/requests/api/v2/heron/plates_spec.rb:254

Plates Heron API #create when providing events when missing required subjects in the events part behaves like a failed plate creation returns 422 Failure/Error: event.errors.each { |key, value| errors.add(key, value) } ActiveSupport::DeprecationException: DEPRECATION WARNING: Enumerating ActiveModel::Errors as a hash has been deprecated. In Rails 6.1, `errors` is an array of Error objects, therefore it should be accessed by a block with a single block parameter like this: person.errors.each do |error| attribute = error.attribute message = error.message end You are passing a block expecting two parameters, so the old hash behavior is simulated. As this is deprecated, this will result in an ArgumentError in Rails 7.0. (called from add_all_errors_from_event at /home/runner/work/sequencescape/sequencescape/app/heron/factories/concerns/eventful.rb:26) Shared Example Group: "a failed plate creation" called from ./spec/requests/api/v2/heron/plates_spec.rb:254

SampleManifest::Uploader will not be valid without some configuration Failure/Error: upload.errors.each { |key, value| errors.add key, value } ActiveSupport::DeprecationException: DEPRECATION WARNING: Enumerating ActiveModel::Errors as a hash has been deprecated. In Rails 6.1, `errors` is an array of Error objects, therefore it should be accessed by a block with a single block parameter like this: person.errors.each do |error| attribute = error.attribute message = error.message end You are passing a block expecting two parameters, so the old hash behavior is simulated. As this is deprecated, this will result in an ArgumentError in Rails 7.0. (called from extract_errors at /home/runner/work/sequencescape/sequencescape/app/models/sample_manifest/uploader.rb:69)

SampleManifest::Uploader will not be valid without a filename Failure/Error: upload.errors.each { |key, value| errors.add key, value } ActiveSupport::DeprecationException: DEPRECATION WARNING: Enumerating ActiveModel::Errors as a hash has been deprecated. In Rails 6.1, `errors` is an array of Error objects, therefore it should be accessed by a block with a single block parameter like this: person.errors.each do |error| attribute = error.attribute message = error.message end You are passing a block expecting two parameters, so the old hash behavior is simulated. As this is deprecated, this will result in an ArgumentError in Rails 7.0. (called from extract_errors at /home/runner/work/sequencescape/sequencescape/app/models/sample_manifest/uploader.rb:69)

SampleManifest::Uploader when checking uploads will upload a valid library tube with tag sequences sample manifest Failure/Error: ActiveRecord::Base.transaction do return false unless valid? # rubocop:todo Rails/TransactionExitStatement return true if process_upload_and_callbacks # rubocop:todo Rails/TransactionExitStatement # One of our post processing checks failed, something went wrong, so we # roll everything back raise ActiveRecord::Rollback end ActiveSupport::DeprecationException: DEPRECATION WARNING: Using `return`, `break` or `throw` to exit a transaction block is deprecated without replacement. If the `throw` came from `Timeout.timeout(duration)`, pass an exception class as a second argument so it doesn't use `throw` to abort its block. This results in the transaction being committed, but in the next release of Rails it will rollback. (called from run! at /home/runner/work/sequencescape/sequencescape/app/models/sample_manifest/uploader.rb:31)

SampleManifest::Uploader when checking uploads will upload a valid partial multiplexed library tube with tag sequences sample manifest Failure/Error: FactoryBot.create(:empty_library_tube) ActiveRecord::RecordInvalid: Validation failed: Name has already been taken

SampleManifest::Uploader when checking uploads will generate sample accessions Failure/Error: study: create(:open_study, accession_number: 'acc') ActiveRecord::RecordInvalid: Validation failed: Login has already been taken

SampleManifest::Uploader when checking uploads will upload a valid partial multiplexed library tube with tag groups and indexes Failure/Error: FactoryBot.create(:empty_library_tube) ActiveRecord::RecordInvalid: Validation failed: Name has already been taken

SampleManifest::Uploader when checking uploads will not upload an invalid multiplexed library tube with tag groups and indexes sample manifest Failure/Error: FactoryBot.create(:empty_library_tube) ActiveRecord::RecordInvalid: Validation failed: Name has already been taken

The job was canceled because "_3_0" failed.

Sample#consent_withdrawn when true behaves like it reports information elsewhere we generate a warehouse message Failure/Error: value = helper.target(object) NoMethodError: undefined method `target' for :sample_metadata:Symbol Shared Example Group: "it reports information elsewhere" called from ./spec/features/patient_consent_withdrawl_spec.rb:118

Sample#consent_withdrawn when false behaves like it reports information elsewhere we generate a warehouse message Failure/Error: value = helper.target(object) NoMethodError: undefined method `target' for :sample_metadata:Symbol Shared Example Group: "it reports information elsewhere" called from ./spec/features/patient_consent_withdrawl_spec.rb:141

Batches controller reordering requests Failure/Error: values = helper.target(object) NoMethodError: undefined method `target' for :requests:Symbol

NpgActions::AssetsController#pass when posting invalid XML to change qc state on an asset. NPG did the same action before renders the exception page Failure/Error: expect(response.body).to match(regexp) expected "<error><message>param is missing or the value is empty: qc_information\nDid you mean? action\n unknown_attribute\n controller\n asset_id</message></error>" to match /<error><message>param is missing or the value is empty: qc_information<\/message><\/error>/m Diff: @@ -1,5 +1,9 @@ -/<error><message>param is missing or the value is empty: qc_information<\/message><\/error>/m +<error><message>param is missing or the value is empty: qc_information +Did you mean? action + unknown_attribute + controller + asset_id</message></error>

NpgActions::AssetsController#pass with both an active and an additional cancelled request behaves like a passed state change renders and creates events Failure/Error: format.html { render template: 'assets/show.xml.builder' } ActiveSupport::DeprecationException: DEPRECATION WARNING: Rendering actions with '.' in the name is deprecated: assets/show.xml.builder (called from block (3 levels) in action_for_qc_state at /home/runner/work/sequencescape/sequencescape/app/controllers/npg_actions/assets_controller.rb:41) Shared Example Group: "a passed state change" called from ./spec/controllers/npg_actions/assets_controller_spec.rb:194

NpgActions::AssetsController#pass when changing qc state on an asset after NPG did the same action before renders and but does not recreate the events Failure/Error: format.html { render template: 'assets/show.xml.builder' } ActiveSupport::DeprecationException: DEPRECATION WARNING: Rendering actions with '.' in the name is deprecated: assets/show.xml.builder (called from block (3 levels) in action_for_qc_state at /home/runner/work/sequencescape/sequencescape/app/controllers/npg_actions/assets_controller.rb:41)

NpgActions::AssetsController#pass with valid parameters behaves like a passed state change renders and creates events Failure/Error: format.html { render template: 'assets/show.xml.builder' } ActiveSupport::DeprecationException: DEPRECATION WARNING: Rendering actions with '.' in the name is deprecated: assets/show.xml.builder (called from block (3 levels) in action_for_qc_state at /home/runner/work/sequencescape/sequencescape/app/controllers/npg_actions/assets_controller.rb:41) Shared Example Group: "a passed state change" called from ./spec/controllers/npg_actions/assets_controller_spec.rb:125

NpgActions::AssetsController#fail with valid parameters behaves like a failed state change renders and creates events Failure/Error: format.html { render template: 'assets/show.xml.builder' } ActiveSupport::DeprecationException: DEPRECATION WARNING: Rendering actions with '.' in the name is deprecated: assets/show.xml.builder (called from block (3 levels) in action_for_qc_state at /home/runner/work/sequencescape/sequencescape/app/controllers/npg_actions/assets_controller.rb:41) Shared Example Group: "a failed state change" called from ./spec/controllers/npg_actions/assets_controller_spec.rb:352

NpgActions::AssetsController#fail with both a valid and an additional cancelled request behaves like a failed state change renders and creates events Failure/Error: format.html { render template: 'assets/show.xml.builder' } ActiveSupport::DeprecationException: DEPRECATION WARNING: Rendering actions with '.' in the name is deprecated: assets/show.xml.builder (called from block (3 levels) in action_for_qc_state at /home/runner/work/sequencescape/sequencescape/app/controllers/npg_actions/assets_controller.rb:41) Shared Example Group: "a failed state change" called from ./spec/controllers/npg_actions/assets_controller_spec.rb:368

Sample manifest with tag sequences library tube sample manifest with tag sequences valid standard reupload and override Failure/Error: ActiveRecord::Base.transaction do return false unless valid? # rubocop:todo Rails/TransactionExitStatement return true if process_upload_and_callbacks # rubocop:todo Rails/TransactionExitStatement # One of our post processing checks failed, something went wrong, so we # roll everything back raise ActiveRecord::Rollback end ActiveSupport::DeprecationException: DEPRECATION WARNING: Using `return`, `break` or `throw` to exit a transaction block is deprecated without replacement. If the `throw` came from `Timeout.timeout(duration)`, pass an exception class as a second argument so it doesn't use `throw` to abort its block. This results in the transaction being committed, but in the next release of Rails it will rollback. (called from run! at /home/runner/work/sequencescape/sequencescape/app/models/sample_manifest/uploader.rb:31)

The job was canceled because "_3_0" failed.

TubeRacks Heron API when there is a tube rack message when there is some data missing/incorrect when some tubes do not have a barcode behaves like a failed tube rack creation returns a 422 status code Failure/Error: recipient.errors.each { |k, v| errors.add("Recipient at #{coordinate} #{k}", v) } ActiveSupport::DeprecationException: DEPRECATION WARNING: Enumerating ActiveModel::Errors as a hash has been deprecated. In Rails 6.1, `errors` is an array of Error objects, therefore it should be accessed by a block with a single block parameter like this: person.errors.each do |error| attribute = error.attribute message = error.message end You are passing a block expecting two parameters, so the old hash behavior is simulated. As this is deprecated, this will result in an ArgumentError in Rails 7.0. (called from block in check_recipients at /home/runner/work/sequencescape/sequencescape/app/heron/factories/concerns/recipients.rb:52) Shared Example Group: "a failed tube rack creation" called from ./spec/requests/api/v2/heron/tube_racks_spec.rb:186

TubeRacks Heron API when there is a tube rack message when there is some data missing/incorrect when some tubes do not have a barcode behaves like a failed tube rack creation does not create a tube rack Failure/Error: recipient.errors.each { |k, v| errors.add("Recipient at #{coordinate} #{k}", v) } ActiveSupport::DeprecationException: DEPRECATION WARNING: Enumerating ActiveModel::Errors as a hash has been deprecated. In Rails 6.1, `errors` is an array of Error objects, therefore it should be accessed by a block with a single block parameter like this: person.errors.each do |error| attribute = error.attribute message = error.message end You are passing a block expecting two parameters, so the old hash behavior is simulated. As this is deprecated, this will result in an ArgumentError in Rails 7.0. (called from block in check_recipients at /home/runner/work/sequencescape/sequencescape/app/heron/factories/concerns/recipients.rb:52) Shared Example Group: "a failed tube rack creation" called from ./spec/requests/api/v2/heron/tube_racks_spec.rb:186

TubeRacks Heron API when there is a tube rack message when there is some data missing/incorrect when some tubes do not have a barcode behaves like a failed tube rack creation contains errors in the response Failure/Error: recipient.errors.each { |k, v| errors.add("Recipient at #{coordinate} #{k}", v) } ActiveSupport::DeprecationException: DEPRECATION WARNING: Enumerating ActiveModel::Errors as a hash has been deprecated. In Rails 6.1, `errors` is an array of Error objects, therefore it should be accessed by a block with a single block parameter like this: person.errors.each do |error| attribute = error.attribute message = error.message end You are passing a block expecting two parameters, so the old hash behavior is simulated. As this is deprecated, this will result in an ArgumentError in Rails 7.0. (called from block in check_recipients at /home/runner/work/sequencescape/sequencescape/app/heron/factories/concerns/recipients.rb:52) Shared Example Group: "a failed tube rack creation" called from ./spec/requests/api/v2/heron/tube_racks_spec.rb:186

Location reports by selection without a start date selected Failure/Error: <%= f.date_field :start_date %> ActionView::Template::Error: undefined method `strftime' for "":String Did you mean? strip

Location reports by selection with selection criteria that find no results Failure/Error: <%= f.date_field :start_date %> ActionView::Template::Error: undefined method `strftime' for "01/01/2017":String Did you mean? strip

QcResultFactory multiple resources plate with blank wells will save all of the assets with valid wells Failure/Error: qc_result.errors.each { |k, v| errors.add(k, v) unless k == :asset && blank_well? } ActiveSupport::DeprecationException: DEPRECATION WARNING: Enumerating ActiveModel::Errors as a hash has been deprecated. In Rails 6.1, `errors` is an array of Error objects, therefore it should be accessed by a block with a single block parameter like this: person.errors.each do |error| attribute = error.attribute message = error.message end You are passing a block expecting two parameters, so the old hash behavior is simulated. As this is deprecated, this will result in an ArgumentError in Rails 7.0. (called from check_qc_result at /home/runner/work/sequencescape/sequencescape/app/models/qc_result_factory.rb:159)

QcResultFactory multiple resources passed as an array #save does not save any of the resources unless they are all valid Failure/Error: qc_result.errors.each { |k, v| errors.add(k, v) unless k == :asset && blank_well? } ActiveSupport::DeprecationException: DEPRECATION WARNING: Enumerating ActiveModel::Errors as a hash has been deprecated. In Rails 6.1, `errors` is an array of Error objects, therefore it should be accessed by a block with a single block parameter like this: person.errors.each do |error| attribute = error.attribute message = error.message end You are passing a block expecting two parameters, so the old hash behavior is simulated. As this is deprecated, this will result in an ArgumentError in Rails 7.0. (called from check_qc_result at /home/runner/work/sequencescape/sequencescape/app/models/qc_result_factory.rb:159)

QcResultFactory multiple resources passed as an array produces sensible error messages if the resource is not valid Failure/Error: qc_result.errors.each { |k, v| errors.add(k, v) unless k == :asset && blank_well? } ActiveSupport::DeprecationException: DEPRECATION WARNING: Enumerating ActiveModel::Errors as a hash has been deprecated. In Rails 6.1, `errors` is an array of Error objects, therefore it should be accessed by a block with a single block parameter like this: person.errors.each do |error| attribute = error.attribute message = error.message end You are passing a block expecting two parameters, so the old hash behavior is simulated. As this is deprecated, this will result in an ArgumentError in Rails 7.0. (called from check_qc_result at /home/runner/work/sequencescape/sequencescape/app/models/qc_result_factory.rb:159)

QcResultFactory QcResultFactory::Resource Barcode will not create a valid resource with an invalid barcode Failure/Error: qc_result.errors.each { |k, v| errors.add(k, v) unless k == :asset && blank_well? } ActiveSupport::DeprecationException: DEPRECATION WARNING: Enumerating ActiveModel::Errors as a hash has been deprecated. In Rails 6.1, `errors` is an array of Error objects, therefore it should be accessed by a block with a single block parameter like this: person.errors.each do |error| attribute = error.attribute message = error.message end You are passing a block expecting two parameters, so the old hash behavior is simulated. As this is deprecated, this will result in an ArgumentError in Rails 7.0. (called from check_qc_result at /home/runner/work/sequencescape/sequencescape/app/models/qc_result_factory.rb:159)

QcResultFactory QcResultFactory::Resource Asset #save should create a qc_result record if valid Failure/Error: qc_result.errors.each { |k, v| errors.add(k, v) unless k == :asset && blank_well? } ActiveSupport::DeprecationException: DEPRECATION WARNING: Enumerating ActiveModel::Errors as a hash has been deprecated. In Rails 6.1, `errors` is an array of Error objects, therefore it should be accessed by a block with a single block parameter like this: person.errors.each do |error| attribute = error.attribute message = error.message end You are passing a block expecting two parameters, so the old hash behavior is simulated. As this is deprecated, this will result in an ArgumentError in Rails 7.0. (called from check_qc_result at /home/runner/work/sequencescape/sequencescape/app/models/qc_result_factory.rb:159)

A submission should error if you set an incorrect request option on construction failed: Expected: 422

A submission should not error if you create it without required options, but does if you build it failed: Expected: 422

POST XML to change qc_state on a asset failed: Expected: 200

Process completed with exit code 1.

The job was canceled because "_2_1" failed.

Trying to update invalid request options failed: Expected: 422

Trying to update invalid request options failed: Expected: 422

Trying to update invalid request options failed: Expected: 422

Trying to update invalid request options failed: Expected: 422

Trying to update invalid request options failed: Expected: 422

The operation was canceled.

Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/checkout@v2. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.

The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/

Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/checkout@v2, actions/setup-node@v2, actions/upload-artifact@v2. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.

The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, actions/setup-node@v2, actions/upload-artifact@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/

RecordLoader::RequestTypeValidatorsLoader with two_entry_example selected sets attributes on the created records Skipped: Temporarily skipped with xit

RecordLoader::RequestTypeValidatorsLoader with two_entry_example selected is idempotent Skipped: Temporarily skipped with xit

RecordLoader::RequestTypeValidatorsLoader with two_entry_example selected creates two records Skipped: Temporarily skipped with xit

PlatePurpose::AdditionalInput#state_of with no ancestors with library requests is passed Skipped: Temporarily skipped with xit

PlatePurpose::AdditionalInput#state_of with no ancestors with no library requests is pending Skipped: Temporarily skipped with xit

PlatePurpose::AdditionalInput#state_of with ancestors with library requests is pending Skipped: Temporarily skipped with xit

PlatePurpose::AdditionalInput#state_of with ancestors with no library requests is pending Skipped: Temporarily skipped with xit

PlatePurpose::AdditionalInput#state_of with no requests is pending Skipped: Temporarily skipped with xit

Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/checkout@v2, actions/setup-node@v3, actions/upload-artifact@v2. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.