Skip to content

Add projections - #60

Open
alexanderkarpetc wants to merge 69 commits into
masterfrom
projections
Open

Add projections#60
alexanderkarpetc wants to merge 69 commits into
masterfrom
projections

Conversation

@alexanderkarpetc

Copy link
Copy Markdown
Contributor

In this PR:

  • Refactor proto model according to UI needs
  • Add projections and repositories for projections
  • Add unit tests for projections and integrational test

In projections.proto add lacking IDs and fields for UI. In events.proto merge ImportMenu and SetDateRange because menu creation was very complicated and unclear.

Previously integrational test didn't include all flows. Now it is enriched with lack commands and checks projections' routings.

Code coverage total increase is 1.69%
Total coverage 98.58%

@codecov

codecovBot commented May 4, 2018

Copy link
Copy Markdown

Codecov Report

Merging #60 into master will increase coverage by 1.71%.
The diff coverage is 99.85%.

@@ Coverage Diff @@## master #60 +/- ##
=========================================
+ Coverage 96.89% 98.6% +1.71% 
=========================================
Files 15 34 +19 Lines 547 1219 +672 Branches 30 41 +11 =========================================
+ Hits 530 1202 +672 
Misses 15 15 Partials 2 2

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@alexanderkarpetc@armiol@PaulAgeyev