Add LiquidAssignsService

[aapomm]

Spec

Right now, we're creating liquid_assigns in a controller concern, and when we need liquid_assigns outside of a controller, we're manually creating the hash.

Proposed solution
Implement a LiquidAssignsService class to build a hash for liquid_assigns based on the given project.

Other Information

If there's anything else that's important and relevant to your pull
request, mention that information here. This could include
benchmarks, or other information.

Thanks for contributing to Dradis!

Copyright assignment

Collaboration is difficult with commercial closed source but we want
to keep as much of the OSS ethos as possible available to users
who want to fix it themselves.

In order to unambiguously own and sell Dradis Framework commercial
products, we must have the copyright associated with the entire
codebase. Any code you create which is merged must be owned by us.
That's not us trying to be a jerks, that's just the way it works.

Please review the CONTRIBUTING.md
file for the details.

You can delete this section, but the following sentence needs to
remain in the PR's description:

I assign all rights, including copyright, to any future Dradis
work by myself to Security Roots.

Check List

• Added a CHANGELOG entry

[caitmich]

What about passing in kwargs instead?
Ex.
@team = args.fetch(:team, nil) in the initializer, and
result['team'] = TeamDrop.new(project.team) if team
on L19
to remove the need for the Dradis::Pro check

[aapomm]

I like that better than the Dradis::Pro check but listing all the args sort of defeats the convenience this brings for the caller. If a caller has to instantiate the class with the following:

LiquidAssignsService.new(
  project: project,
  team: project.team,
  document_properties: project.content_library.properties
)

Then it starts to look like:

result['project'] = ...
result['document_properties'] = ...
result['team'] = ...

just without the drop?

[etdsoft]

I'd extract this to a #assigns_pro that only gets called if defined? as we do in other places.

[aapomm]

A bit unrelated to the task but it makes this specific spec not dependent on the other specs that create records by counting the records in the trash instead of expecting it to be empty.