BED-4947: Fix Last Analysis Run At Bug

[superlinkx]

Description

Fixes a logic bug that was causing last_analysis_run_at to run every time the datapipe status changed

Motivation and Context

This PR addresses: BED-4947

Why is this change required? What problem does it solve?

This bug was causing scheduled analysis to never run

How Has This Been Tested?

• Existing test has been updated to account for the correct, intended behavior

Screenshots (optional):

Types of changes

• Bug fix (non-breaking change which fixes an issue)

Checklist:

• I have met the contributing prerequisites
  • Assigned myself to this PR
  • Added the appropriate labels
  • Associated an issue: Issues and Pull Requests README #672
  • Read the Contributing guide: https:/SpecterOps/BloodHound/wiki/Contributing
• I have ensured that related documentation is up-to-date
  • Open API docs
  • Code comments (GoDocs / JSDocs)
• I have followed proper test practices
  • Added/updated tests to cover my changes
  • All new and existing tests passed

[superlinkx]

Looks like we're not getting statuses correctly for some reason in one of our tests. Needs more testing and validation.