Implement aggregation guidelines

Description

Using RMB-provided functionality. One potential approach is that the "join" functionality will be broken out into a separate RMB component.

Environment

None

Potential Workaround

None

Checklist

hide

TestRail: Results

Activity

Show:

shale99 June 27, 2017 at 3:16 PM

sure , thanks

Kurt Nordstrom June 27, 2017 at 3:12 PM

I will review today and perhaps we can compare notes tomorrow when I get into work?

shale99 June 27, 2017 at 10:36 AM

i have created a pull request with the mod-users-bl rewrite. All endpoints have been implemented. this includes the /users endpoint which uses the rmb join functionality to return a list of composite users - with each composite user containing perms / creds / groups associated with the user. this incorporates the 'includes' trait.
- can you review?
i have not completed the testing - started looking into mocking via raml briefly, but am currently just rewriting marcs fakes

Jakub Skoczen June 20, 2017 at 2:09 PM

As discussed on the call: it would be good to include unit test for the new aggregation functionality. They may require mock/fake services, and Marc has examples of such services in mod-inv/mod-circ.

Done

Details

Assignee

Reporter

Priority

Fix versions

TestRail: Cases

Open TestRail: Cases

TestRail: Runs

Open TestRail: Runs
Created June 6, 2017 at 1:21 PM
Updated August 28, 2017 at 2:29 PM
Resolved July 16, 2017 at 9:28 AM
TestRail: Cases
TestRail: Runs