Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Use Xoom Designer to regenerate the project #13

Draft
wants to merge 100 commits into
base: master
Choose a base branch
from
Draft

Use Xoom Designer to regenerate the project #13

wants to merge 100 commits into from

Conversation

jakzal
Copy link
Member

@jakzal jakzal commented Sep 28, 2021 •
edited
Loading

  • Generate xoom-auth with xoom-designer
  • Make existing resource tests pass
    • TenantResourceTest
    • GroupResourceTest (9/11 remaining)
    • PermissionResourceTest
    • RoleResourceTest
    • UserResourceTest (1/6 remaining)
  • Remove the legacy model and resources
  • Remove the generated code that's not used

jakzal added 26 commits September 28, 2021 10:39
@jakzal
Generate xoom-auth with xoom-designer
90d9174 
Signed-off-by: Jakub Zalas <[email protected]>
@jakzal
Fix problems with aggregate definitions
24b715c 
Signed-off-by: Jakub Zalas <[email protected]>
@jakzal
Disable code that did not compile
35d36af 
Signed-off-by: Jakub Zalas <[email protected]>
@jakzal
Regenerate the project with latest snapshot of Designer
7078320 
Signed-off-by: Jakub Zalas <[email protected]>
@jakzal
Regenerate the project with latest snapshot of Designer
8d34d38 
Signed-off-by: Jakub Zalas <[email protected]>
@jakzal
Move resource tests to the infrastructure package
c5ba6cc 
Signed-off-by: Jakub Zalas <[email protected]>
@jakzal
Compile TenantResourceTest
c1dfc73 
Signed-off-by: Jakub Zalas <[email protected]>
@jakzal
Remove the exchange package
bbc5b9d 
Signed-off-by: Jakub Zalas <[email protected]>
@jakzal
Use the latest xoom SNAPSHOT and bump rest-assured
7dc8d44 
Signed-off-by: Jakub Zalas <[email protected]>
@jakzal
Make route paths for queries unique
b609549 
Signed-off-by: Jakub Zalas <[email protected]>
@jakzal
Pass TenantResourceTest.testThatTenantSubscribes
cc4b0cb 
Signed-off-by: Jakub Zalas <[email protected]>
@jakzal
Pass TenantResourceTest.testThatTenantDeactivatesAndActivates
1a7dad3 
Signed-off-by: Jakub Zalas <[email protected]>
@jakzal
Pass TenantResourceTest.testThatTenantChangesDescription
7fadaad 
Signed-off-by: Jakub Zalas <[email protected]>
@jakzal
Pass TenantResourceTest.testThatTenantChangesName
f669b0d 
Signed-off-by: Jakub Zalas <[email protected]>
@jakzal
Cleanup global state after each test
12bfae5 
Signed-off-by: Jakub Zalas <[email protected]>
@jakzal
Fix the order of tenant name and description arguments to make most o…
5c3b6b3 
…f remaining TenantResourceTest tests pass

Signed-off-by: Jakub Zalas <[email protected]>
@jakzal
Remove redundant state changes
d386025 
Signed-off-by: Jakub Zalas <[email protected]>
@jakzal
Pass user credentials to user registration entities, state, and events
3377b47 
Signed-off-by: Jakub Zalas <[email protected]>
@jakzal
Compile GroupResourceTest
0d6ec76 
Signed-off-by: Jakub Zalas <[email protected]>
@jakzal
Compile PermissionResourceTest
7f7f1c4 
Signed-off-by: Jakub Zalas <[email protected]>
@jakzal
Compile RoleResourceTest
1ddbfde 
Signed-off-by: Jakub Zalas <[email protected]>
@jakzal
Compile UserResourceTest
abf9e1d 
Signed-off-by: Jakub Zalas <[email protected]>
@jakzal
Remove the unused TestResponseChannelConsumer
04ecd4c 
Signed-off-by: Jakub Zalas <[email protected]>
@jakzal
Use JUnit 5 for resource tests
21fb70d 
Signed-off-by: Jakub Zalas <[email protected]>
@jakzal
Allow for both JUnit 4 and 5
782304a 
Signed-off-by: Jakub Zalas <[email protected]>
@jakzal
Convert the payload for group description update to json
75caea8 
Signed-off-by: Jakub Zalas <[email protected]>
@jakzal jakzal force-pushed the rewrite/event-sourced branch from aa2a36e to 75caea8 Compare September 28, 2021 09:40
@jakzal jakzal force-pushed the rewrite/event-sourced branch from 2263379 to 0e61084 Compare October 22, 2021 15:16
jakzal added 10 commits October 22, 2021 17:20
@jakzal
Remove generated tests that are not covering any real behaviour
eeeb99c 
Signed-off-by: Jakub Zalas <[email protected]>
@jakzal
Merge branch 'master' into rewrite/event-sourced
31a20c0
@jakzal
Decouple Group post data from thew view
7a32e81 
Signed-off-by: Jakub Zalas <[email protected]>
@jakzal
Decouple Role post data from thew view
06cb20a 
Signed-off-by: Jakub Zalas <[email protected]>
@jakzal
Persist Role relations in RoleView
dfa6a02 
Signed-off-by: Jakub Zalas <[email protected]>
@jakzal
Update the role assign group path to match the old path
2c3786c 
Signed-off-by: Jakub Zalas <[email protected]>
@jakzal
Decouple Permission post data from the view
c048afd 
Signed-off-by: Jakub Zalas <[email protected]>
@jakzal
Pass RoleResourceTest testThatRolePermissionAttaches, testThatRolePer…
18215d9 
…missionDetaches, testThatRolePermissionQueries

Signed-off-by: Jakub Zalas <[email protected]>
@jakzal
Match the RoleResource.assignGroup response with the expected one
e8926e2 
Signed-off-by: Jakub Zalas <[email protected]>
@jakzal
Pass RoleResourceTest testThatRoleGroupIsAssigned, testThatRoleGroupI…
265afbc 
…sUnassigned, testThatRoleGroupQueries

Signed-off-by: Jakub Zalas <[email protected]>
@jakzal jakzal force-pushed the rewrite/event-sourced branch from be78596 to 265afbc Compare November 10, 2021 15:41
@jakzal
Store references to group relations instead of only identifiers
f29131d 
Signed-off-by: Jakub Zalas <[email protected]>
@jakzal
Store references to user relations instead of only identifiers
b3f748b 
Signed-off-by: Jakub Zalas <[email protected]>
@jakzal
Decouple User post data from the view
49dbddc 
Signed-off-by: Jakub Zalas <[email protected]>
@jakzal
Pass RoleResourceTest testThatRoleUserIsAssigned testThatRoleUserIsUn…
54e02ba 
…assigned testThatRoleUserQueries

Signed-off-by: Jakub Zalas <[email protected]>
@jakzal
Pass UserResourceTest.testThatUserIsInRole
4053938 
Signed-off-by: Jakub Zalas <[email protected]>
@jakzal
Add the missing endpoint for user permissions
2ca3f80 
Signed-off-by: Jakub Zalas <[email protected]>
@jakzal
Start the test world in projection tests
63c80a1 
Signed-off-by: Jakub Zalas <[email protected]>
@jakzal
Clear the component registry after projection tests
c358662 
Signed-off-by: Jakub Zalas <[email protected]>
@jakzal
Use the store dispatcher to dispatch events instead of calling projec…
9412c8f 
…tions directly

The benefit is that if the event is not configured for given projection, it will not be handled by the projection.

Signed-off-by: Jakub Zalas <[email protected]>
@jakzal jakzal force-pushed the rewrite/event-sourced branch from 761edb0 to 9412c8f Compare November 23, 2021 17:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@jakzal