Reverting rubocop styles to pass the tests initially #5360
Annotations
11 errors and 5 warnings
|
Run rspec tests:
spec/models/sample_manifest/uploader_spec.rb#L164
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)
|
Run rspec tests:
spec/models/sample_manifest/uploader_spec.rb#L64
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
|
Run rspec tests:
spec/models/sample_manifest/uploader_spec.rb#L132
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
|
Run rspec tests:
spec/models/sample_manifest/uploader_spec.rb#L269
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
|
Run rspec tests:
spec/models/sample_manifest/uploader_spec.rb#L181
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
|
Run rspec tests:
spec/models/sample_manifest/uploader_spec.rb#L283
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
|
Run rspec tests:
spec/models/sample_manifest/uploader_spec.rb#L193
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
|
Run rspec tests:
spec/models/sample_manifest/uploader_spec.rb#L81
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
|
Run rspec tests:
spec/models/sample_manifest/uploader_spec.rb#L241
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
|
Run rspec tests:
spec/models/sample_manifest/uploader_spec.rb#L205
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
|
Run rspec tests:
spec/models/plate_purpose/additional_input_spec.rb#L53
PlatePurpose::AdditionalInput#state_of with no ancestors with no library requests is pending
Skipped: Temporarily skipped with xit
|
Run rspec tests:
spec/models/plate_purpose/additional_input_spec.rb#L64
PlatePurpose::AdditionalInput#state_of with no ancestors with library requests is passed
Skipped: Temporarily skipped with xit
|
Run rspec tests:
spec/models/plate_purpose/additional_input_spec.rb#L22
PlatePurpose::AdditionalInput#state_of with no requests is pending
Skipped: Temporarily skipped with xit
|
Run rspec tests:
spec/models/plate_purpose/additional_input_spec.rb#L32
PlatePurpose::AdditionalInput#state_of with ancestors with no library requests is pending
Skipped: Temporarily skipped with xit
|
Run rspec tests:
spec/models/plate_purpose/additional_input_spec.rb#L43
PlatePurpose::AdditionalInput#state_of with ancestors with library requests is pending
Skipped: Temporarily skipped with xit
|
Loading