search

samvera / hyrax

Hyrax is a Ruby on Rails Engine built by the Samvera community. Hyrax provides a foundation for creating many different digital repository applications.
http://hyrax.samvera.org/
Apache License 2.0
184 stars 124 forks source link

RepositoryAuditService audits not showing results in the UI #188

Closed mjgiarlo closed 7 years ago

mjgiarlo commented 7 years ago

Issue by electrickite Tuesday Nov 15, 2016 at 14:42 GMT Originally opened as https://github.com/projecthydra/sufia/issues/2886

Descriptive summary

When running audits using Sufia::RepositoryAuditService.audit_everything, the results are not shown in the UI.

Observed in Sufia 7.0.0, but I can't see anything in later versions that would have changed this behavior.

Expected behavior

Running audits on a FileSet will show the results of the audit on the file set show page.

Actual behavior

The audit status remains 'Audits have not yet been run on this file.' even if audits have been run.

Steps to reproduce the behavior

  1. Create at least one Work with an attached FileSet
  2. From a Rails console, run Sufia::RepositoryAuditService.audit_everything
  3. If needed, wait for background audit jobs to complete.
  4. From a Rails console, run ChecksumAuditLog.all and observe that audit log entries have been created.
  5. Navigate to the FileSet "show" page and observe the audit status is still 'Audits have not yet been run on this file.'
mjgiarlo commented 7 years ago

Comment by mjgiarlo Tuesday Nov 15, 2016 at 15:04 GMT

Thanks, @electrickite! Do you have cycles to troubleshoot?

mjgiarlo commented 7 years ago

Comment by mjgiarlo Tuesday Nov 15, 2016 at 15:05 GMT

Related to https://github.com/projecthydra/curation_concerns/issues/1091

mjgiarlo commented 7 years ago

Comment by electrickite Tuesday Nov 15, 2016 at 15:08 GMT

@mjgiarlo I should be able to track this down a little more later today or tomorrow. I would not be surprised to find out this is bubbling up from curation concerns, but I figured we should track it in Sufia since it affects Sufia components.

mjgiarlo commented 7 years ago

Comment by mjgiarlo Friday Dec 16, 2016 at 23:27 GMT

@electrickite have you had a chance to look into this?

mjgiarlo commented 7 years ago

Comment by mjgiarlo Tuesday Dec 20, 2016 at 21:49 GMT

I suggest moving this work to Hyrax vs. it blocking the 7.3.0 release. If someone is inclined to put in a patch before the 7.3.0 release is cut, I am happy to include this in the release.