Process completed with exit code 1.

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#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 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 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#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

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

Sample manifest with tag sequences multiplexed tube sample manifest with tag sequences valid cgap foreign barcodes upload 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)

Sample manifest with tag sequences multiplexed tube sample manifest with tag sequences valid upload and reupload with override Failure/Error: let!(:user) { create :admin } ActiveRecord::RecordInvalid: Validation failed: Login has already been taken

Sample manifest with tag sequences plate sample manifest invalid unrecognised cgap foreign barcodes validation errors Failure/Error: let!(:user) { create :admin } ActiveRecord::RecordInvalid: Validation failed: Login has already been taken

Sample manifest with tag sequences plate sample manifest invalid no file no file Failure/Error: let!(:user) { create :admin } ActiveRecord::RecordInvalid: Validation failed: Login has already been taken

The job was canceled because "_3_1" failed.

SampleManifest::Uploader when checking uploads will generate sample accessions 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 1d tube sample manifest Failure/Error: Study.find_by(name: new_study) || FactoryBot.create(:study, name: new_study) ActiveRecord::RecordInvalid: Validation failed: Login has already been taken

SampleManifest::Uploader when checking uploads will upload a valid multiplexed library tube with tag groups and indexes sample manifest Failure/Error: Study.find_by(name: new_study) || FactoryBot.create(:study, name: new_study) ActiveRecord::RecordInvalid: Validation failed: Login has already been taken

SampleManifest::Uploader when checking uploads will upload a valid partial multiplexed library tube with tag sequences sample manifest Failure/Error: Study.find_by(name: new_study) || FactoryBot.create(:study, name: new_study) ActiveRecord::RecordInvalid: Validation failed: Login has already been taken

SampleManifest::Uploader when checking uploads will not upload an invalid 1d tube sample manifest Failure/Error: Study.find_by(name: new_study) || FactoryBot.create(:study, name: new_study) 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: Study.find_by(name: new_study) || FactoryBot.create(:study, name: new_study) ActiveRecord::RecordInvalid: Validation failed: Login has already been taken

SampleManifest::Uploader when checking uploads will not upload an invalid library tube with tag sequences sample manifest Failure/Error: Study.find_by(name: new_study) || FactoryBot.create(:study, name: new_study) ActiveRecord::RecordInvalid: Validation failed: Login has already been taken

The job was canceled because "_3_1" failed.

SampleManifest::Uploader when checking uploads will upload a valid library tube with tag sequences sample manifest Failure/Error: Study.find_by(name: new_study) || FactoryBot.create(:study, name: new_study) ActiveRecord::RecordInvalid: Validation failed: Login has already been taken

SampleManifest::Uploader when checking uploads will upload a valid partial 1d tube sample manifest Failure/Error: Study.find_by(name: new_study) || FactoryBot.create(:study, name: new_study) ActiveRecord::RecordInvalid: Validation failed: Login has already been taken

SampleManifest::Uploader when checking uploads will not upload an invalid multiplexed library tube with tag sequences sample manifest Failure/Error: Study.find_by(name: new_study) || FactoryBot.create(:study, name: new_study) ActiveRecord::RecordInvalid: Validation failed: Login has already been taken

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

Process completed with exit code 1.

The job was canceled because "_2_1" failed.

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.

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 creates two 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 sets attributes on the created records 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 no ancestors with library requests is passed Skipped: Temporarily skipped with xit

PlatePurpose::AdditionalInput#state_of with no 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 ancestors with library 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/.